[OpenSER-Devel] [ openser-Bugs-1819335 ] [lcr] bug in doc in "rpid_avp" param

[SourceForge.net]

Bugs item #1819335, was opened at 2007-10-24 17:12
Message generated for change (Settings changed) made by juhe
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743020&aid=1819335&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: Closed
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Iñaki Baz (ibc_sf)
>Assigned to: Juha Heinanen (juhe)
Summary: [lcr] bug in doc in "rpid_avp" param

Initial Comment:
Devel doc says:

1.3.18. rpid_avp (AVP string)
modparam("^auth$|domain|lcr", "rpid_avp", "$avp(i:302)")

But domain module has not "rpid_avp" parameter so OpenSer gives an error.


Let me a question:

I've noted since devel (1.3) version lot of parameters in this lcr module are mandatory (in 1.2 they weren't).

In my case I just want load_contacts() and next_contacts() functions, and not gateway related. But I need to declare:

modparam("lcr", "gw_uri_avp", "$avp(i:703)")
modparam("lcr", "ruri_user_avp", "$avp(i:704)")
modparam("lcr", "rpid_avp", "$avp(i:705)")
modparam("lcr", "dm_flag", 25)

All of then parameters I don't need.

Couldn't be possible to "separate" gateway related things and contacts "q" related things? or at least those parameters not to be mandatory?

Regards and congratulations for this module, serial forwarding for contacts works very well :)

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

Comment By: Iñaki Baz (ibc_sf)
Date: 2007-10-24 19:25

Message:
Logged In: YES 
user_id=1844020
Originator: YES

Ok, I understand.

Anyway, as I said in my report, it's very "extrange" for me to define all
these parameters when I'm not using them since I just want load_contacts()
and next_contacts() functions.

Anyway I understand the issue.

Thanks.

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

Comment By: Ovidiu Sas (osas)
Date: 2007-10-24 17:31

Message:
Logged In: YES 
user_id=1395524
Originator: NO

> I've noted since devel (1.3) version lot of parameters in this lcr
module
> are mandatory (in 1.2 they weren't).

The idea is to have no default avps.
Imagine that in your script you will set some avps that are having the
same value as the default value of some modules.
The results will be unpredictable and very difficult to debug.

Please check the following thread on the mailing list:
http://openser.org/pipermail/users/2007-March/009494.html

Regards,
Ovidiu Sas

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

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