Module: kamailio
Branch: 5.4
Commit: 2c501f4d7b9678f0cfad3afec5e3095fd61e1763
URL: https://github.com/kamailio/kamailio/commit/2c501f4d7b9678f0cfad3afec5e3095…
Author: Olle E. Johansson <oej(a)edvina.net>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2021-06-01T11:10:43+02:00
snmpstats: Fix typo in MIB
Reported by arkadiam on Github
(cherry picked from commit 7f017707c91659641ce9ca87ee4149b2e39379f7)
(cherry picked from commit dbd8e3899335eb8795763fd52b1f91aebfad1c92)
---
Modified: src/modules/snmpstats/mibs/KAMAILIO-SIP-SERVER-MIB
---
Diff: https://github.com/kamailio/kamailio/commit/2c501f4d7b9678f0cfad3afec5e3095…
Patch: https://github.com/kamailio/kamailio/commit/2c501f4d7b9678f0cfad3afec5e3095…
---
diff --git a/src/modules/snmpstats/mibs/KAMAILIO-SIP-SERVER-MIB b/src/modules/snmpstats/mibs/KAMAILIO-SIP-SERVER-MIB
index 3351a9e424..2aaddf01ca 100644
--- a/src/modules/snmpstats/mibs/KAMAILIO-SIP-SERVER-MIB
+++ b/src/modules/snmpstats/mibs/KAMAILIO-SIP-SERVER-MIB
@@ -562,7 +562,7 @@ KAMAILIO-SIP-SERVER-MIB DEFINITIONS ::= BEGIN
set the row-status to createAndGo.
Then the client can do an SNMP GET on this table with the
- aquired index. If the row-status is 'active', then the
+ acquired index. If the row-status is 'active', then the
kamailioSIPRegUserIndex is the index into the
kamailioSIPRegUserTable that has the specified URI (0 zero
indicates no such URI). The client should then set the
Module: kamailio
Branch: 5.5
Commit: 19901d0cdb6cf06a84315c19a27986bca7a05025
URL: https://github.com/kamailio/kamailio/commit/19901d0cdb6cf06a84315c19a27986b…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2021-06-01T10:58:18+02:00
core: parse via - fixed whitespacing
(cherry picked from commit 01664d1d82dcfe952fce29033b4fcd5e44cf2319)
---
Modified: src/core/parser/parse_via.h
---
Diff: https://github.com/kamailio/kamailio/commit/19901d0cdb6cf06a84315c19a27986b…
Patch: https://github.com/kamailio/kamailio/commit/19901d0cdb6cf06a84315c19a27986b…
---
diff --git a/src/core/parser/parse_via.h b/src/core/parser/parse_via.h
index fca486e358..e4957961f1 100644
--- a/src/core/parser/parse_via.h
+++ b/src/core/parser/parse_via.h
@@ -13,8 +13,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
*/
@@ -32,11 +32,11 @@
struct sip_msg;
/* via param types
- * WARNING: keep in sync with parse_via.c FIN_HIDDEN...
+ * WARNING: keep in sync with parse_via.c FIN_HIDDEN...
* and with tm/sip_msg.c via_body_cloner
*/
enum {
- PARAM_HIDDEN=230, PARAM_TTL, PARAM_BRANCH,
+ PARAM_HIDDEN=230, PARAM_TTL, PARAM_BRANCH,
PARAM_MADDR, PARAM_RECEIVED, PARAM_RPORT, PARAM_I, PARAM_ALIAS,
#ifdef USE_COMP
PARAM_COMP,
@@ -60,12 +60,12 @@ typedef struct via_param {
/* Format: name/version/transport host:port;params comment */
- /* WARNING: keep in sync with tm/sip_msg.c via_body_cloner */
-typedef struct via_body {
+/* WARNING: keep in sync with tm/sip_msg.c via_body_cloner */
+typedef struct via_body {
int error;
str hdr; /* Contains "Via" or "v" */
str name;
- str version;
+ str version;
str transport;
str host;
short proto; /* transport */
@@ -80,7 +80,7 @@ typedef struct via_body {
struct via_param* param_lst; /* list of parameters*/
struct via_param* last_param; /*last via parameter, internal use*/
- /* shortcuts to "important" params*/
+ /* shortcuts to "important" params*/
struct via_param* branch;
str tid; /* transaction id, part of branch */
struct via_param* received;
@@ -90,8 +90,8 @@ typedef struct via_body {
#ifdef USE_COMP
struct via_param* comp; /* see rfc3486 */
#endif
- struct via_body* next; /* pointer to next via body string if
- compact via or null */
+ struct via_body* next; /* pointer to next via body string
+ if compact via or null */
} via_body_t;
Module: kamailio
Branch: master
Commit: bf4ae4b5f4f4b75c534681c520fa1060f234a18e
URL: https://github.com/kamailio/kamailio/commit/bf4ae4b5f4f4b75c534681c520fa106…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2021-06-01T09:31:20+02:00
modules: readme files regenerated - nathelper ... [skip ci]
---
Modified: src/modules/nathelper/README
---
Diff: https://github.com/kamailio/kamailio/commit/bf4ae4b5f4f4b75c534681c520fa106…
Patch: https://github.com/kamailio/kamailio/commit/bf4ae4b5f4f4b75c534681c520fa106…
---
diff --git a/src/modules/nathelper/README b/src/modules/nathelper/README
index 0fa395da20..8e5c790d58 100644
--- a/src/modules/nathelper/README
+++ b/src/modules/nathelper/README
@@ -66,7 +66,7 @@ Ovidiu Sas
5.5. nat_uac_test(flags)
5.6. is_rfc1918(ip_address)
5.7. add_contact_alias([ip_addr, port, proto])
- 5.8. handle_ruri_alias()
+ 5.8. handle_ruri_alias([mode])
5.9. set_contact_alias([trim])
5.10. set_alias_to_pv(target_avp)
@@ -157,7 +157,7 @@ Chapter 1. Admin Guide
5.5. nat_uac_test(flags)
5.6. is_rfc1918(ip_address)
5.7. add_contact_alias([ip_addr, port, proto])
- 5.8. handle_ruri_alias()
+ 5.8. handle_ruri_alias([mode])
5.9. set_contact_alias([trim])
5.10. set_alias_to_pv(target_avp)
@@ -497,7 +497,7 @@ modparam("nathelper", "nat_addr_mode", 0)
5.5. nat_uac_test(flags)
5.6. is_rfc1918(ip_address)
5.7. add_contact_alias([ip_addr, port, proto])
- 5.8. handle_ruri_alias()
+ 5.8. handle_ruri_alias([mode])
5.9. set_contact_alias([trim])
5.10. set_alias_to_pv(target_avp)
@@ -668,7 +668,7 @@ if(is_rfc1918("$rd")) {
};
...
-5.8. handle_ruri_alias()
+5.8. handle_ruri_alias([mode])
Checks if the Request URI has an “alias” parameter and if so, removes
it and sets the “$du” based on its value. Note that this means that
@@ -677,6 +677,10 @@ if(is_rfc1918("$rd")) {
on a request, make sure that you screen the alias parameter value of
Request URI the same way as you would screen the Request URI itself.
+ The optional parameter mode can be 0 to consume first alias parameter,
+ otherwise it consumes the last alias parameter. If the parameter mode
+ is not provided, it consumes the first parameter.
+
Returns 1 if “;alias” parameter was found and “$du” was set and the
“$ru” rewritten, 2 if the alias parameter was not found and nothing was
done, or -1 in case of error.
Module: kamailio
Branch: master
Commit: 9c060a66f83a42c46767f5b28b6a3640764ebb87
URL: https://github.com/kamailio/kamailio/commit/9c060a66f83a42c46767f5b28b6a364…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2021-06-01T09:29:33+02:00
nathelper: docs for mode parameter for handle_ruri_alias()
---
Modified: src/modules/nathelper/doc/nathelper_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/9c060a66f83a42c46767f5b28b6a364…
Patch: https://github.com/kamailio/kamailio/commit/9c060a66f83a42c46767f5b28b6a364…
---
diff --git a/src/modules/nathelper/doc/nathelper_admin.xml b/src/modules/nathelper/doc/nathelper_admin.xml
index 5096a676d4..39e61e6856 100644
--- a/src/modules/nathelper/doc/nathelper_admin.xml
+++ b/src/modules/nathelper/doc/nathelper_admin.xml
@@ -794,17 +794,21 @@ if(is_rfc1918("$rd")) {
<section id="nathelper.f.handle_ruri_alias">
<title>
- <function moreinfo="none">handle_ruri_alias()</function>
+ <function moreinfo="none">handle_ruri_alias([mode])</function>
</title>
<para>
Checks if the Request URI has an <quote>alias</quote>
parameter and if so, removes it and sets the <quote>$du</quote> based
- on its value. Note that this means that routing of request is based on
- <quote>;alias</quote> parameter value of the Request URI rather than
+ on its value. Note that this means that routing of request is based on
+ <quote>;alias</quote> parameter value of the Request URI rather than
the Request URI itself. If you call <function>handle_ruri_alias()</function>
on a request, make sure that you screen the alias parameter value of
- Request URI the same way as you would screen the
- Request URI itself.
+ Request URI the same way as you would screen the Request URI itself.
+ </para>
+ <para>
+ The optional parameter mode can be 0 to consume first alias parameter,
+ otherwise it consumes the last alias parameter. If the parameter mode is
+ not provided, it consumes the first parameter.
</para>
<para>
Returns <emphasis>1</emphasis> if <quote>;alias</quote> parameter was
#### Pre-Submission Checklist
- [x] Commit message has the format required by CONTRIBUTING guide
- [x] Commits are split per component (core, individual modules, libs, utils, ...)
- [x] Each component has a single commit (if not, squash them into one commit)
- [x] No commits to README files for modules (changes must be done to docbook files
in `doc/` subfolder, the README file is autogenerated)
#### Type Of Change
- [x] Small bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)
#### Checklist:
- [x] PR should be backported to stable branches
- [ ] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
fixed `4sipdump(src_port)` handling
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/2753
-- Commit Summary --
* sipdump: fixed typo
-- File Changes --
M src/modules/sipdump/sipdump_mod.c (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2753.patchhttps://github.com/kamailio/kamailio/pull/2753.diff
--
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/pull/2753
### Description
<!--
On every startup of Kamailio I'm getting an error im my Log when Kamailio is going to load the globalblacklist table from MySQL Database
I tested this with version 5.4.1 build from git source.
With version 5.3.6 there are no errors like this. I diffed the source of userblacklist.c and saw some changes in the code. So there may be a bug.
-->
### Troubleshooting
#### Reproduction
<!--
Create a simple config an use check_blacklist()
Use db_mysql as database backend
-->
#### Debugging Data
#### Log Messages
```
Sep 17 07:56:36 voip-lab-proxy03 /usr/sbin/kamailio[553]: DEBUG: <core> [core/route.c:885]: fix_actions(): fixing check_blacklist()
Sep 17 07:56:36 voip-lab-proxy03 /usr/sbin/kamailio[553]: DEBUG: userblacklist [userblacklist.c:472]: add_source(): add table globalblacklist
Sep 17 07:56:36 voip-lab-proxy03 /usr/sbin/kamailio[553]: DEBUG: <core> [dtrie.c:57]: dtrie_init(): allocate 16 bytes for root at 0x7fc29efa4ab0
Sep 17 07:56:36 voip-lab-proxy03 /usr/sbin/kamailio[553]: DEBUG: <core> [dtrie.c:67]: dtrie_init(): allocate 80 bytes for 10 root children pointer at 0x7fc29efa4b28
Sep 17 07:56:36 voip-lab-proxy03 /usr/sbin/kamailio[553]: ERROR: <core> [db.c:481]: db_use_table(): invalid parameter value
Sep 17 07:56:36 voip-lab-proxy03 /usr/sbin/kamailio[553]: ERROR: userblacklist [db.c:123]: db_reload_source(): cannot use db table 'globalblacklist'
Sep 17 07:56:36 voip-lab-proxy03 /usr/sbin/kamailio[553]: ERROR: userblacklist [userblacklist.c:425]: load_source(): cannot load source from 'globalblacklist'
```
### Additional Information
```
Kamailio 5.4.1
```
* **Operating System**:
```
PRETTY_NAME="Debian GNU/Linux 10 (buster)"
NAME="Debian GNU/Linux"
VERSION_ID="10"
VERSION="10 (buster)"
VERSION_CODENAME=buster
ID=debian
HOME_URL="https://www.debian.org/"
SUPPORT_URL="https://www.debian.org/support"
BUG_REPORT_URL="https://bugs.debian.org/"
Linux voip-lab-proxy03 4.19.0-10-amd64 #1 SMP Debian 4.19.132-1 (2020-07-24) 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/2480
### Description
For some reason `From` header cannot be parsed.
#### Reproduction
Tested config
```
listen=udp:127.0.0.1:5060
loadmodule "pv.so"
loadmodule "http_client.so"
loadmodule "lost.so"
modparam("http_client", "httpcon", "http://example.com=>http://example.com");
request_route {
lost_held_query("http://example.com", "$var(pidf)", "$var(geo_url)", "$var(err)");
}
```
Client sipp
```
sipp -m 1 -sn uac -p 2345 127.0.0.1
```
#### Log Messages
Kamailio output
````
1(7643) WARNING: lost [functions.c:252]: lost_held_function(): P-A-I header not found, trying From header ...
1(7643) ERROR: lost [utilities.c:644]: lost_get_from_header(): From header not found
1(7643) ERROR: lost [functions.c:259]: lost_held_function(): no device id found
2(7644) WARNING: lost [functions.c:252]: lost_held_function(): P-A-I header not found, trying From header ...
2(7644) ERROR: lost [utilities.c:644]: lost_get_from_header(): From header not found
2(7644) ERROR: lost [functions.c:259]: lost_held_function(): no device id found
3(7645) WARNING: lost [functions.c:252]: lost_held_function(): P-A-I header not found, trying From header ...
3(7645) ERROR: lost [utilities.c:644]: lost_get_from_header(): From header not found
3(7645) ERROR: lost [functions.c:259]: lost_held_function(): no device id found
````
#### SIP Traffic
```
INVITE sip:service@127.0.0.1:5060 SIP/2.0
Via: SIP/2.0/UDP 127.0.0.1:2345;branch=z9hG4bK-7655-1-0
From: sipp <sip:sipp@127.0.0.1:2345>;tag=7655SIPpTag001
To: service <sip:service@127.0.0.1:5060>
Call-ID: 1-7655(a)127.0.0.1
CSeq: 1 INVITE
Contact: sip:sipp@127.0.0.1:2345
Max-Forwards: 70
Subject: Performance Test
Content-Type: application/sdp
Content-Length: 129
v=0
o=user1 53655765 2353687637 IN IP4 127.0.0.1
s=-
c=IN IP4 127.0.0.1
t=0 0
m=audio 6000 RTP/AVP 0
a=rtpmap:0 PCMU/8000
```
### Additional Information
current master
* **Operating System**:
CentOS 8 x86_64
--
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/2723