[sr-dev] git:master:8a38f369: sdpops: Add note about sdp_remove_media() and RFC compliance

Module: kamailio Branch: master Commit: 8a38f369b01a642b92704cc0e6efeb6e108f8a02 URL: https://github.com/kamailio/kamailio/commit/8a38f369b01a642b92704cc0e6efeb6… Author: Carsten Bock &lt;carsten(a)ng-voice.com&gt; Committer: Carsten Bock &lt;carsten(a)ng-voice.com&gt; Date: 2017-04-07T11:09:14+02:00 sdpops: Add note about sdp_remove_media() and RFC compliance --- Modified: src/modules/sdpops/doc/sdpops_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/8a38f369b01a642b92704cc0e6efeb6… Patch: https://github.com/kamailio/kamailio/commit/8a38f369b01a642b92704cc0e6efeb6… --- diff --git a/src/modules/sdpops/doc/sdpops_admin.xml b/src/modules/sdpops/doc/sdpops_admin.xml index a4bf378..53a2552 100644 --- a/src/modules/sdpops/doc/sdpops_admin.xml +++ b/src/modules/sdpops/doc/sdpops_admin.xml @@ -278,6 +278,31 @@ if(sdp_with_active_media("video")) Remove the streams that match on 'm=type ...' line. The parameter can be static string or variable holding the media type. </para> + <para> + Note: If this is executed on a request, the callee should reply + without the Media too. However, RFC 3264 mandates, that the reply + should contain the exact same number of "m=" lines as the + request. + </para> +<blockquote><attribution>RFC 3264</attribution> + <para> +6 Generating the Answer + </para> + <para> + [...] + </para> + <para> + For each "m=" line in the offer, there MUST be a corresponding "m=" + line in the answer. The answer MUST contain exactly the same number + of "m=" lines as the offer. This allows for streams to be matched up + based on their order. This implies that if the offer contained zero + "m=" lines, the answer MUST contain zero "m=" lines. + </para> +</blockquote> + <para> + So this may not work with all Endpoints, especially if they + follow RFC 3264 precisely (e.g. JSSIP). + </para> <para> This function can be used from ANY_ROUTE. </para>