[SR-Users] Dispatcher: ds_is_from_list does not longer populate grp_avp

2012-08-15 Thread DanB
Hey Guys, Something I noticed after upgrading Kamailio from 3.3.0 to 3.3.1. Having the following code: if ds_is_from_list() { xlog(Request coming from SCloud); xlog(Checking grp_internal, having: $avp(s:dsgrp)); } will produce the following output to syslog (dsgrp null instead of group

Re: [SR-Users] Dispatcher: ds_is_from_list does not longer populate grp_avp

2012-08-15 Thread Daniel-Constantin Mierla
Hello, there was no change in dispatcher module related to this function since 3.3.0. The value of the group id is set to the variable specified by the parameter setid_pvar: http://kamailio.org/docs/modules/stable/modules_k/dispatcher.html#id2495800 Very likely you are using the wrong

Re: [SR-Users] Dispatcher: ds_is_from_list does not longer populate grp_avp

2012-08-15 Thread DanB
Hey Daniel, Thanks for so fast feedback. I have doublecked again the config: If it helps, here are my complete settings: # - dispatcher params - modparam(dispatcher, list_file, /etc/sbalancer/dispatcher.list) modparam(dispatcher, flags, 2) modparam(dispatcher, dst_avp, $avp(dsdst))

Re: [SR-Users] Dispatcher: ds_is_from_list does not longer populate grp_avp

2012-08-15 Thread DanB
Daniel, My bad, I was reading through the lines :(. I now understand that setid_pvar is different than group pvar. I got the false understanding based on documentation saying: Parameter groupid is optional, when it is missing, then the matching will be done against all addresses in all

Re: [SR-Users] Dispatcher: ds_is_from_list does not longer populate grp_avp

2012-08-15 Thread Daniel-Constantin Mierla
Hello, right, the readme was mistaken. Probably grp_avp was used initially but changed later because it messes up when ds_is_from_list() and ds_select_*() are used for same requests. I committed an update to the readme. Cheers, Daniel On 8/15/12 12:40 PM, DanB wrote: Daniel, My bad, I

Re: [SR-Users] Dispatcher: ds_is_from_list does not longer populate grp_avp

2012-08-15 Thread Daniel-Constantin Mierla
Hello, grp_avp is set only when calling the destination/domain select functions. You have to set the parameter setid_pvar: http://kamailio.org/docs/modules/stable/modules_k/dispatcher.html#id2495800 And use the variable you set for this parameter after ds_is_from_list(). So, there are two

Re: [SR-Users] Dispatcher: ds_is_from_list does not longer populate grp_avp

2012-08-15 Thread DanB
Daniel, Since we are still in connected state, one more (hopefully last) question on this subject: If I set the setid_pvar the configuration does not longer pass checking (just upgraded to latest nightly 3.3.1): root@PrxyDev:/etc/sbalancer# kamailio -c -f kamailio.cfg loading modules under

Re: [SR-Users] Dispatcher: ds_is_from_list does not longer populate grp_avp

2012-08-15 Thread Daniel-Constantin Mierla
Another mismatch of sources and docs, the parameter is expected to be 'setid_pvname' instead of 'setid_pvar'. I'll update docs. Cheers, Daniel On 8/15/12 12:56 PM, DanB wrote: Daniel, Since we are still in connected state, one more (hopefully last) question on this subject: If I set the

Re: [SR-Users] Dispatcher: ds_is_from_list does not longer populate grp_avp

2012-08-15 Thread DanB
Daniel, Nice catch! All smooth now. Thanks and enjoy your free time ;). DanB On 08/15/2012 01:00 PM, Daniel-Constantin Mierla wrote: Another mismatch of sources and docs, the parameter is expected to be 'setid_pvname' instead of 'setid_pvar'. I'll update docs. Cheers, Daniel On 8/15/12