Module: kamailio
Branch: master
Commit: bbf51ef65ecee0b6c8e30abb3738da3436b85692
URL: https://github.com/kamailio/kamailio/commit/bbf51ef65ecee0b6c8e30abb3738da3…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2017-02-09T11:32:45+01:00
path: note in docs that some registrars check for supported header
---
Modified: src/modules/path/doc/path_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/bbf51ef65ecee0b6c8e30abb3738da3…
Patch: https://github.com/kamailio/kamailio/commit/bbf51ef65ecee0b6c8e30abb3738da3…
---
diff --git a/src/modules/path/doc/path_admin.xml b/src/modules/path/doc/path_admin.xml
index 946dfdc..31b8a57 100644
--- a/src/modules/path/doc/path_admin.xml
+++ b/src/modules/path/doc/path_admin.xml
@@ -10,9 +10,9 @@
<!-- Module User's Guide -->
<chapter>
-
+
<title>&adminguide;</title>
-
+
<section>
<title>Overview</title>
<para>
@@ -22,9 +22,9 @@
for evaluating this parameter in subsequent requests and to set the destination &uri; according to it.
</para>
<section>
- <title>Path insertion for registrations</title>
+ <title>Path Insertion For Registrations</title>
<para>
- For registrations in a scenario like <quote>[UAC] -> [P1] -> [REG]</quote>,
+ For registrations in a scenario like <quote>[UAC] -> [P1] -> [REG]</quote>,
the "path" module can be used at the intermediate proxy P1 to insert a Path
header into the message before forwarding it to the registrar REG. Two functions
can be used to achieve this:
@@ -46,7 +46,7 @@
<para>
<emphasis>add_path_received(...)</emphasis> also add a Path header in the
same form as above, but also adds a parameter indicating the received-&uri;
- of the message, like
+ of the message, like
<quote>Path: <sip:1.2.3.4;received=sip:2.3.4.5:1234;lr></quote>. This
is especially useful if the proxy does NAT detection and wants to pass
the NAT'ed address to the registrar.
@@ -58,7 +58,17 @@
</itemizedlist>
</para>
<para>
+ Note that some SIP registrars may check if header Supported includes 'path'. It
+ can be added in &kamailio;.cfg using append_hf() from textops module.
</para>
+ <example>
+ <title>Add Supported header</title>
+ <programlisting format="linespecific">
+...
+append_hf("Supported: path\r\n");
+...
+</programlisting>
+ </example>
</section>
<section>
<title>Outbound routing to NAT'ed UACs</title>
Module: kamailio
Branch: master
Commit: e53bc78966990156dbe3b879db1d4924e9f22e56
URL: https://github.com/kamailio/kamailio/commit/e53bc78966990156dbe3b879db1d492…
Author: Victor Seva <linuxmaniac(a)torreviejawireless.org>
Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org>
Date: 2017-02-09T09:53:43+01:00
pkg/kamailio/deb: version set to 5.0.0-pre2
---
Modified: pkg/kamailio/deb/debian/changelog
Modified: pkg/kamailio/deb/jessie/changelog
Modified: pkg/kamailio/deb/precise/changelog
Modified: pkg/kamailio/deb/sid/changelog
Modified: pkg/kamailio/deb/squeeze/changelog
Modified: pkg/kamailio/deb/stretch/changelog
Modified: pkg/kamailio/deb/trusty/changelog
Modified: pkg/kamailio/deb/wheezy/changelog
Modified: pkg/kamailio/deb/xenial/changelog
---
Diff: https://github.com/kamailio/kamailio/commit/e53bc78966990156dbe3b879db1d492…
Patch: https://github.com/kamailio/kamailio/commit/e53bc78966990156dbe3b879db1d492…
---
diff --git a/pkg/kamailio/deb/debian/changelog b/pkg/kamailio/deb/debian/changelog
index cb31fb5..6445f89 100644
--- a/pkg/kamailio/deb/debian/changelog
+++ b/pkg/kamailio/deb/debian/changelog
@@ -1,3 +1,9 @@
+kamailio (5.0.0~pre2) unstable; urgency=medium
+
+ * version set to 5.0.0-pre2
+
+ -- Victor Seva <vseva(a)debian.org> Thu, 09 Feb 2017 09:53:10 +0100
+
kamailio (5.0.0~pre1) unstable; urgency=medium
* version set to 5.0.0-pre1
diff --git a/pkg/kamailio/deb/jessie/changelog b/pkg/kamailio/deb/jessie/changelog
index cb31fb5..6445f89 100644
--- a/pkg/kamailio/deb/jessie/changelog
+++ b/pkg/kamailio/deb/jessie/changelog
@@ -1,3 +1,9 @@
+kamailio (5.0.0~pre2) unstable; urgency=medium
+
+ * version set to 5.0.0-pre2
+
+ -- Victor Seva <vseva(a)debian.org> Thu, 09 Feb 2017 09:53:10 +0100
+
kamailio (5.0.0~pre1) unstable; urgency=medium
* version set to 5.0.0-pre1
diff --git a/pkg/kamailio/deb/precise/changelog b/pkg/kamailio/deb/precise/changelog
index cb31fb5..6445f89 100644
--- a/pkg/kamailio/deb/precise/changelog
+++ b/pkg/kamailio/deb/precise/changelog
@@ -1,3 +1,9 @@
+kamailio (5.0.0~pre2) unstable; urgency=medium
+
+ * version set to 5.0.0-pre2
+
+ -- Victor Seva <vseva(a)debian.org> Thu, 09 Feb 2017 09:53:10 +0100
+
kamailio (5.0.0~pre1) unstable; urgency=medium
* version set to 5.0.0-pre1
diff --git a/pkg/kamailio/deb/sid/changelog b/pkg/kamailio/deb/sid/changelog
index cb31fb5..6445f89 100644
--- a/pkg/kamailio/deb/sid/changelog
+++ b/pkg/kamailio/deb/sid/changelog
@@ -1,3 +1,9 @@
+kamailio (5.0.0~pre2) unstable; urgency=medium
+
+ * version set to 5.0.0-pre2
+
+ -- Victor Seva <vseva(a)debian.org> Thu, 09 Feb 2017 09:53:10 +0100
+
kamailio (5.0.0~pre1) unstable; urgency=medium
* version set to 5.0.0-pre1
diff --git a/pkg/kamailio/deb/squeeze/changelog b/pkg/kamailio/deb/squeeze/changelog
index cb31fb5..6445f89 100644
--- a/pkg/kamailio/deb/squeeze/changelog
+++ b/pkg/kamailio/deb/squeeze/changelog
@@ -1,3 +1,9 @@
+kamailio (5.0.0~pre2) unstable; urgency=medium
+
+ * version set to 5.0.0-pre2
+
+ -- Victor Seva <vseva(a)debian.org> Thu, 09 Feb 2017 09:53:10 +0100
+
kamailio (5.0.0~pre1) unstable; urgency=medium
* version set to 5.0.0-pre1
diff --git a/pkg/kamailio/deb/stretch/changelog b/pkg/kamailio/deb/stretch/changelog
index cb31fb5..6445f89 100644
--- a/pkg/kamailio/deb/stretch/changelog
+++ b/pkg/kamailio/deb/stretch/changelog
@@ -1,3 +1,9 @@
+kamailio (5.0.0~pre2) unstable; urgency=medium
+
+ * version set to 5.0.0-pre2
+
+ -- Victor Seva <vseva(a)debian.org> Thu, 09 Feb 2017 09:53:10 +0100
+
kamailio (5.0.0~pre1) unstable; urgency=medium
* version set to 5.0.0-pre1
diff --git a/pkg/kamailio/deb/trusty/changelog b/pkg/kamailio/deb/trusty/changelog
index cb31fb5..6445f89 100644
--- a/pkg/kamailio/deb/trusty/changelog
+++ b/pkg/kamailio/deb/trusty/changelog
@@ -1,3 +1,9 @@
+kamailio (5.0.0~pre2) unstable; urgency=medium
+
+ * version set to 5.0.0-pre2
+
+ -- Victor Seva <vseva(a)debian.org> Thu, 09 Feb 2017 09:53:10 +0100
+
kamailio (5.0.0~pre1) unstable; urgency=medium
* version set to 5.0.0-pre1
diff --git a/pkg/kamailio/deb/wheezy/changelog b/pkg/kamailio/deb/wheezy/changelog
index cb31fb5..6445f89 100644
--- a/pkg/kamailio/deb/wheezy/changelog
+++ b/pkg/kamailio/deb/wheezy/changelog
@@ -1,3 +1,9 @@
+kamailio (5.0.0~pre2) unstable; urgency=medium
+
+ * version set to 5.0.0-pre2
+
+ -- Victor Seva <vseva(a)debian.org> Thu, 09 Feb 2017 09:53:10 +0100
+
kamailio (5.0.0~pre1) unstable; urgency=medium
* version set to 5.0.0-pre1
diff --git a/pkg/kamailio/deb/xenial/changelog b/pkg/kamailio/deb/xenial/changelog
index cb31fb5..6445f89 100644
--- a/pkg/kamailio/deb/xenial/changelog
+++ b/pkg/kamailio/deb/xenial/changelog
@@ -1,3 +1,9 @@
+kamailio (5.0.0~pre2) unstable; urgency=medium
+
+ * version set to 5.0.0-pre2
+
+ -- Victor Seva <vseva(a)debian.org> Thu, 09 Feb 2017 09:53:10 +0100
+
kamailio (5.0.0~pre1) unstable; urgency=medium
* version set to 5.0.0-pre1
Hello,
following discussion on mailing lists and at Fosdem over the last
weekend, I am considering to create the git branch 5.0 later this week,
on Thursday, Feb 9, 2017. Afterwards, the master branch can take
commits with new features. The branch 5.0 will be tested for 1-2 more
weeks and if all is fine, the v5.0.0 can be released.
Should anyone have other suggestions, jump into discussion.
Cheers,
Daniel
--
Daniel-Constantin Mierla
www.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio Advanced Training - Mar 6-8 (Europe) and Mar 20-22 (USA) - www.asipto.com
Kamailio World Conference - May 8-10, 2017 - www.kamailioworld.com
### Description
fix_nated_contact() and removing and appending a contact will malform the SIP packet.
I read somewhere that all of the *_hf functions are applied only when the packet is sent out again. Perhaps things are working as intended and I am just asking too much of these functions.
#### Reproduction
In my onreply_route for 200 OK with SDP:
fix_nated_contact();
#dlg_setflag(1); # I want the contact in the dialog table to be the fixed one
#Then I want to replace that contact with a contact to Kamailio's public IP and route it to Asterisk
#for ACK response. This contact must be here for Asterisk to relay this back to me.
remove_hf("Contact");
append_hf("Contact: <sip:$sel(cfg_get.global.publicIP):5060>\r\n"); # insert_hf same problem
### Troubleshooting
This results in two contacts, but does not malform. remove_hf after the nat_fix_contact is the issue.
fix_nated_contact();
insert_hf("Contact: <sip:$sel(cfg_get.global.publicIP):5060>\r\n");
#### SIP Traffic
The inbound 200 OK
CSeq: 9865 INVITE
Contact: <sip:424242143@devicePrivateIP:5060>
Content-Type: application/sdp
After fix_nated_contact(); remove_hf("Contact"); append_hf("Contact: <sip:serverIP:5060>\r\n");
CSeq: 9865 INVITE
sip:424242143@devicePublicIP:1026Content-Type: application/sdp
### Possible Solutions
I managed to get the functionality I wanted with either of these, but they result in two contact headers.
I have also noticed that a remove->append->remove->append ignores the remove between appends.
Workaround 1
remove_hf("Contact");
insert_hf("Contact: <sip:$tU@$si:$sp>\r\n");
dlg_setflag(1);
insert_hf("Contact: <sip:$sel(cfg_get.global.publicIP):5060>\r\n");
Workaround 2
fix_nated_contact();
dlg_setflag(1);
insert_hf("Contact: <sip:$sel(cfg_get.global.publicIP):5060>\r\n");
### Additional Information
* **Kamailio Version**: kamailio 4.4.5 (x86_64/linux) f98162
* **Operating System**: Linux 3.10.0-514.2.2.el7.x86_64 # 1 SMP Tue Dec 6 23:06:41 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/977