Hi,
i am using Kamailio as an outbound proxy for Translating between TLS
(public network) and UDP (internal network). When testing with a
client, that uses a SIPS scheme, i've noticed the following behaviour,
which broke my setup:
Due to the sips:-URI in the Request URI and the two different
interfaces (UDP and TLS), the RR module adds two new Record-Route
headers like this:
Record-Route: <sips:mt@46.101.100.26:5061;transport=tls;r2=on;lr=on;ftag=7BABD31B-59F04202000E785C-AD96E700;rm=1;did=f81.f88>
Record-Route: <sips:mt@46.101.100.26;r2=on;lr=on;ftag=7BABD31B-59F04202000E785C-AD96E700;rm=1;did=f81.f88>
This however breaks the internal routing, as the server is internally
only reachable via UDP and not via TLS, as the internal network
components only have UDP interfaces.
Is this a bug? I've temporarily created a workaround, so RR will
always use "sip" instead of "sips" (which solved my issue), but I
wonder if this in general should be considered a bug???
Thanks,
Carsten
--
Carsten Bock
CEO (Geschäftsführer)
ng-voice GmbH
Millerntorplatz 1
20359 Hamburg / Germany
http://www.ng-voice.com
mailto:carsten@ng-voice.com
Office +49 40 5247593-40
Fax +49 40 5247593-99
Sitz der Gesellschaft: Hamburg
Registergericht: Amtsgericht Hamburg, HRB 120189
Geschäftsführer: Carsten Bock
Ust-ID: DE279344284
Hier finden Sie unsere handelsrechtlichen Pflichtangaben:
http://www.ng-voice.com/imprint/
Module: kamailio
Branch: 5.0
Commit: 49d4af49682ea189321875c7132b3987d55a5519
URL: https://github.com/kamailio/kamailio/commit/49d4af49682ea189321875c7132b398…
Author: Victor Seva <linuxmaniac(a)torreviejawireless.org>
Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org>
Date: 2017-10-25T10:49:57+02:00
pkg/kamailio/deb: set version to 5.0.4 [skip ci]
---
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/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/49d4af49682ea189321875c7132b398…
Patch: https://github.com/kamailio/kamailio/commit/49d4af49682ea189321875c7132b398…
---
diff --git a/pkg/kamailio/deb/debian/changelog b/pkg/kamailio/deb/debian/changelog
index 08f8f4a163..3d61d4dfae 100644
--- a/pkg/kamailio/deb/debian/changelog
+++ b/pkg/kamailio/deb/debian/changelog
@@ -1,3 +1,9 @@
+kamailio (5.0.4) unstable; urgency=medium
+
+ * version set to 5.0.4
+
+ -- Victor Seva <vseva(a)debian.org> Wed, 25 Oct 2017 10:49:00 +0200
+
kamailio (5.0.3) unstable; urgency=medium
* version set to 5.0.3
diff --git a/pkg/kamailio/deb/jessie/changelog b/pkg/kamailio/deb/jessie/changelog
index 08f8f4a163..3d61d4dfae 100644
--- a/pkg/kamailio/deb/jessie/changelog
+++ b/pkg/kamailio/deb/jessie/changelog
@@ -1,3 +1,9 @@
+kamailio (5.0.4) unstable; urgency=medium
+
+ * version set to 5.0.4
+
+ -- Victor Seva <vseva(a)debian.org> Wed, 25 Oct 2017 10:49:00 +0200
+
kamailio (5.0.3) unstable; urgency=medium
* version set to 5.0.3
diff --git a/pkg/kamailio/deb/precise/changelog b/pkg/kamailio/deb/precise/changelog
index 08f8f4a163..3d61d4dfae 100644
--- a/pkg/kamailio/deb/precise/changelog
+++ b/pkg/kamailio/deb/precise/changelog
@@ -1,3 +1,9 @@
+kamailio (5.0.4) unstable; urgency=medium
+
+ * version set to 5.0.4
+
+ -- Victor Seva <vseva(a)debian.org> Wed, 25 Oct 2017 10:49:00 +0200
+
kamailio (5.0.3) unstable; urgency=medium
* version set to 5.0.3
diff --git a/pkg/kamailio/deb/sid/changelog b/pkg/kamailio/deb/sid/changelog
index 08f8f4a163..3d61d4dfae 100644
--- a/pkg/kamailio/deb/sid/changelog
+++ b/pkg/kamailio/deb/sid/changelog
@@ -1,3 +1,9 @@
+kamailio (5.0.4) unstable; urgency=medium
+
+ * version set to 5.0.4
+
+ -- Victor Seva <vseva(a)debian.org> Wed, 25 Oct 2017 10:49:00 +0200
+
kamailio (5.0.3) unstable; urgency=medium
* version set to 5.0.3
diff --git a/pkg/kamailio/deb/stretch/changelog b/pkg/kamailio/deb/stretch/changelog
index 08f8f4a163..3d61d4dfae 100644
--- a/pkg/kamailio/deb/stretch/changelog
+++ b/pkg/kamailio/deb/stretch/changelog
@@ -1,3 +1,9 @@
+kamailio (5.0.4) unstable; urgency=medium
+
+ * version set to 5.0.4
+
+ -- Victor Seva <vseva(a)debian.org> Wed, 25 Oct 2017 10:49:00 +0200
+
kamailio (5.0.3) unstable; urgency=medium
* version set to 5.0.3
diff --git a/pkg/kamailio/deb/trusty/changelog b/pkg/kamailio/deb/trusty/changelog
index 08f8f4a163..3d61d4dfae 100644
--- a/pkg/kamailio/deb/trusty/changelog
+++ b/pkg/kamailio/deb/trusty/changelog
@@ -1,3 +1,9 @@
+kamailio (5.0.4) unstable; urgency=medium
+
+ * version set to 5.0.4
+
+ -- Victor Seva <vseva(a)debian.org> Wed, 25 Oct 2017 10:49:00 +0200
+
kamailio (5.0.3) unstable; urgency=medium
* version set to 5.0.3
diff --git a/pkg/kamailio/deb/wheezy/changelog b/pkg/kamailio/deb/wheezy/changelog
index 08f8f4a163..3d61d4dfae 100644
--- a/pkg/kamailio/deb/wheezy/changelog
+++ b/pkg/kamailio/deb/wheezy/changelog
@@ -1,3 +1,9 @@
+kamailio (5.0.4) unstable; urgency=medium
+
+ * version set to 5.0.4
+
+ -- Victor Seva <vseva(a)debian.org> Wed, 25 Oct 2017 10:49:00 +0200
+
kamailio (5.0.3) unstable; urgency=medium
* version set to 5.0.3
diff --git a/pkg/kamailio/deb/xenial/changelog b/pkg/kamailio/deb/xenial/changelog
index 08f8f4a163..3d61d4dfae 100644
--- a/pkg/kamailio/deb/xenial/changelog
+++ b/pkg/kamailio/deb/xenial/changelog
@@ -1,3 +1,9 @@
+kamailio (5.0.4) unstable; urgency=medium
+
+ * version set to 5.0.4
+
+ -- Victor Seva <vseva(a)debian.org> Wed, 25 Oct 2017 10:49:00 +0200
+
kamailio (5.0.3) unstable; urgency=medium
* version set to 5.0.3
Module: kamailio
Branch: 5.0
Commit: 5cb020ac9ed53f8520b9070a1945329abad08029
URL: https://github.com/kamailio/kamailio/commit/5cb020ac9ed53f8520b9070a1945329…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2017-10-25T10:20:34+02:00
core: parser - typedef'ed msg_start struct to msg_start_t
(cherry picked from commit a21ea97800c7d7d3b082a9b1d57887e125d83d68)
---
Modified: src/core/parser/parse_fline.h
---
Diff: https://github.com/kamailio/kamailio/commit/5cb020ac9ed53f8520b9070a1945329…
Patch: https://github.com/kamailio/kamailio/commit/5cb020ac9ed53f8520b9070a1945329…
---
diff --git a/src/core/parser/parse_fline.h b/src/core/parser/parse_fline.h
index 04f96b63f4..b10c02bc83 100644
--- a/src/core/parser/parse_fline.h
+++ b/src/core/parser/parse_fline.h
@@ -18,8 +18,8 @@
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
*/
@@ -68,31 +68,28 @@
#define REFER_LEN 5
#define PUBLISH_LEN 7
-struct msg_start {
+typedef struct msg_start {
short type; /*!< Type of the message - request/response */
short flags; /*!< First line flags */
int len; /*!< length including delimiter */
union {
struct {
- str method; /*!< Method string */
- str uri; /*!< Request URI */
- str version; /*!< SIP version */
+ str method; /*!< Method string */
+ str uri; /*!< Request URI */
+ str version; /*!< SIP version */
int method_value;
} request;
struct {
- str version; /*!< SIP version */
- str status; /*!< Reply status */
- str reason; /*!< Reply reason phrase */
- unsigned int /*!< statusclass,*/ statuscode;
+ str version; /*!< SIP version */
+ str status; /*!< Reply status */
+ str reason; /*!< Reply reason phrase */
+ unsigned int statuscode; /*!< Reply status code */
} reply;
}u;
-};
-
-
+} msg_start_t;
char* parse_first_line(char* buffer, unsigned int len, struct msg_start * fl);
char* parse_fline(char* buffer, char* end, struct msg_start* fl);
-
#endif /* PARSE_FLINE_H */
Module: kamailio
Branch: master
Commit: a21ea97800c7d7d3b082a9b1d57887e125d83d68
URL: https://github.com/kamailio/kamailio/commit/a21ea97800c7d7d3b082a9b1d57887e…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2017-10-25T10:16:43+02:00
core: parser - typedef'ed msg_start struct to msg_start_t
---
Modified: src/core/parser/parse_fline.h
---
Diff: https://github.com/kamailio/kamailio/commit/a21ea97800c7d7d3b082a9b1d57887e…
Patch: https://github.com/kamailio/kamailio/commit/a21ea97800c7d7d3b082a9b1d57887e…
---
diff --git a/src/core/parser/parse_fline.h b/src/core/parser/parse_fline.h
index 04f96b63f4..b10c02bc83 100644
--- a/src/core/parser/parse_fline.h
+++ b/src/core/parser/parse_fline.h
@@ -18,8 +18,8 @@
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
*/
@@ -68,31 +68,28 @@
#define REFER_LEN 5
#define PUBLISH_LEN 7
-struct msg_start {
+typedef struct msg_start {
short type; /*!< Type of the message - request/response */
short flags; /*!< First line flags */
int len; /*!< length including delimiter */
union {
struct {
- str method; /*!< Method string */
- str uri; /*!< Request URI */
- str version; /*!< SIP version */
+ str method; /*!< Method string */
+ str uri; /*!< Request URI */
+ str version; /*!< SIP version */
int method_value;
} request;
struct {
- str version; /*!< SIP version */
- str status; /*!< Reply status */
- str reason; /*!< Reply reason phrase */
- unsigned int /*!< statusclass,*/ statuscode;
+ str version; /*!< SIP version */
+ str status; /*!< Reply status */
+ str reason; /*!< Reply reason phrase */
+ unsigned int statuscode; /*!< Reply status code */
} reply;
}u;
-};
-
-
+} msg_start_t;
char* parse_first_line(char* buffer, unsigned int len, struct msg_start * fl);
char* parse_fline(char* buffer, char* end, struct msg_start* fl);
-
#endif /* PARSE_FLINE_H */