rtpproxy: add rtpp_bind_local support for the offer/answer/engage.#3794
rtpproxy: add rtpp_bind_local support for the offer/answer/engage.#3794sobomax wants to merge 1 commit intoOpenSIPS:masterfrom
Conversation
When this AVP is set, its value would be provided to the rtpproxy with "l" modifier, allowing proper address to be selected for the session.
|
@sobomax I love the idea of passing the local socket to rtpproxy, but we're trying to avoid using AVPs for passing information like that, as it might confuse people (happened too many times). That's why we'd rather explicitely using the argument as a flag to rtpproxy_offer/answer/engage(). That's actually why we like to pass all the other flags explicitely (see Furthermore, not sure if the protocol supports, but it would be nice to have some sort of interface alias/tag - that's useful when using multiple rtpproxy instances in a set, where you have different IPs configuration, and simply choose internal or external interface. Your example shows an IP, so I presumed that currently only IPs are allowed. |
Summary
This PR adds more flexible control over socket that RTPProxy picks for a new or existing media session. The existing mechanism with I/E designators is a bit crude and limiting.
Details
When
rtpp_bind_localAVP is set, its value would be provided to the rtpproxy with "l" modifier, allowing proper address to be selected for the session.Solution
This can be used either globally by setting the rtpproxy module parameter:
Or on logic level by setting up the AVP:
Compatibility
Should be none, it's a new API.
Closing issues
None