We think we have found a potential bug in the path module.

When we call add_path_received() we get the following for WSS connections:

Path: <sip:217.10.77.242;lr;received=sip:217.10.69.93:42612%3Btransport%3Dws>

We think there should be a transport%3Dwss appended instead of ws.

We have looked a bit into the code and think that this is caused by the following definition:

in modules/path/path.c


const static char *proto_strings[] = {
                [PROTO_TCP] = "%3Btransport%3Dtcp",
                [PROTO_TLS] = "%3Btransport%3Dtls",
                [PROTO_SCTP] = "%3Btransport%3Dsctp",
                [PROTO_WS] = "%3Btransport%3Dws",
                [PROTO_WSS] = "%3Btransport%3Dws",
};

We think that on the last line, it should read
[PROTO_WSS] = "%3Btransport%3Dwss",


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <kamailio/kamailio/issues/4168@github.com>

ChristianBergerSipgateChristianBergerSipgate created an issue (kamailio/kamailio#4168)

We think we have found a potential bug in the path module.

When we call add_path_received() we get the following for WSS connections:

Path: <sip:217.10.77.242;lr;received=sip:217.10.69.93:42612%3Btransport%3Dws>

We think there should be a transport%3Dwss appended instead of ws.

We have looked a bit into the code and think that this is caused by the following definition:

in modules/path/path.c


const static char *proto_strings[] = {
                [PROTO_TCP] = "%3Btransport%3Dtcp",
                [PROTO_TLS] = "%3Btransport%3Dtls",
                [PROTO_SCTP] = "%3Btransport%3Dsctp",
                [PROTO_WS] = "%3Btransport%3Dws",
                [PROTO_WSS] = "%3Btransport%3Dws",
};

We think that on the last line, it should read
[PROTO_WSS] = "%3Btransport%3Dwss",


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <kamailio/kamailio/issues/4168@github.com>