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

[SourceForge.net]

Patches item #3606986, was opened at 2013-03-05 16:07
Message generated for change (Comment added) made by razvancrainea
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1086412&aid=3606986&group_id=232389

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")

engage_rtp_proxy("froc");
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: Razvan Crainea (razvancrainea)
Date: 2013-03-12 04:13

Message:
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,
Răzvan

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

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1086412&aid=3606986&group_id=232389