[OpenSIPS-Devel] [ opensips-Patches-3606986 ] Add pvar support for rtp proxy used

SourceForge.net noreply at sourceforge.net
Thu Mar 14 18:53:49 CET 2013

Patches item #3606986, was opened at 2013-03-05 16:07
Message generated for change (Comment added) made by rrb3942
You can respond by visiting: 

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: trunk
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Ryan Bullock (rrb3942)
Assigned to: Razvan Crainea (razvancrainea)
Summary: Add pvar support for rtp proxy used

Initial Comment:
Adds new module parameter "rtpp_sock_pvar" which is a pvar to be set with the url of the rtpproxy used on the call.

Example usage:
modparam("rtpproxy", "rtpp_sock_pvar", "$avp(rtp_sock)")
#Load balancing rtpproxies
modparam("rtpproxy", "rtpproxy_sock",  "udp:localhost:12221 udp:localhost:12222")

xlog("Used rtpproxy $avp(rtp_sock) for callid $ci");

In the log:
"Used rtpproxy udp:localhost:12222 for callid abc at 123.com"

Useful for troubleshooting quality issues related to a single node or instance or rtpproxy.

I have done some limited tested in a lab setup for this patch.


>Comment By: Ryan Bullock (rrb3942)
Date: 2013-03-14 10:53

Hey Răzvan,

I tested your patch and it works well, but doesn't set the node if only a
single rtpproxy is configured. I attached an updated patch that sets it in
that case as well (for consistency, hopefully you know what node the call
went to if you only have 1 :) .

Tested with a single rtpproxy, a set of two, and in the scenario where one
rtpproxy does not respond. All worked well.

Thanks for the help, let me know if anything else is required.




Comment By: Razvan Crainea (razvancrainea)
Date: 2013-03-12 04:13

Hi, Ryan!

Thank you for the patch you provided. The only problem I can see is that
you only set the variable for the first server. However, if that server
fails, and a failover succeeds, you will have a bogus server set. Therefore
I moved the code that sets the variable in the select_rtpp_node() function,
in order to have a more general approach.
I attached a new patch for this. Can you five it a try and let me know if
it works as desired?

Best regards,


You can respond by visiting: 

More information about the Devel mailing list