[OpenSER-Devel] [ openser-Bugs-1890852 ] [presence] Change "totag_avpid" to allow any AVP

[SourceForge.net]

Bugs item #1890852, was opened at 2008-02-11 01:59
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743020&aid=1890852&group_id=139143

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: modules
Group: ver devel
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Iñaki Baz (ibc_sf)
Assigned to: Nobody/Anonymous (nobody)
Summary: [presence] Change "totag_avpid" to allow any AVP

Initial Comment:
Actually (if I'm not wrong) "presence" module has a parameter "totag_avpid" that is the only parameter in any module that askes for an AVP numeric id.

This is:
  modparam("presence", "totag_avpid", 28)
means that "presence" will use AVP(i:28) for its purposes.

Any other module using AVP's allows choosing any kind of AVP, not just numeric. Examples:
  modparam("tm", "fr_inv_timer_avp", "$avp(i:39)")
  modparam("auth", "rpid_avp", "$avp(s:rpid)")
  modparam("registrar", "received_avp", "$avp(i:66)")
  modparam("cpl-c","timer_avp","$avp(i:39)")

Why should "presence" be different?

I propose to change "totag_avpid" to be like the other similar parameters:
  modparam("presence", "totag_avpid", "$avp(i:28)")

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743020&aid=1890852&group_id=139143