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

[Carsten Bock]

Module: kamailio
Branch: master
Commit: 8a38f369b01a642b92704cc0e6efeb6e108f8a02
URL: https://github.com/kamailio/kamailio/commit/8a38f369b01a642b92704cc0e6efeb6e108f8a02

Author: Carsten Bock <carsten at ng-voice.com>
Committer: Carsten Bock <carsten at ng-voice.com>
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/8a38f369b01a642b92704cc0e6efeb6e108f8a02.diff
Patch: https://github.com/kamailio/kamailio/commit/8a38f369b01a642b92704cc0e6efeb6e108f8a02.patch

---

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>