Module: kamailio
Branch: 5.5
Commit: dbd8e3899335eb8795763fd52b1f91aebfad1c92
URL: https://github.com/kamailio/kamailio/commit/dbd8e3899335eb8795763fd52b1f91a…
Author: Olle E. Johansson <oej(a)edvina.net>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2021-05-26T08:14:04+02:00
snmpstats: Fix typo in MIB
Reported by arkadiam on Github
(cherry picked from commit 7f017707c91659641ce9ca87ee4149b2e39379f7)
---
Modified: src/modules/snmpstats/mibs/KAMAILIO-SIP-SERVER-MIB
---
Diff: https://github.com/kamailio/kamailio/commit/dbd8e3899335eb8795763fd52b1f91a…
Patch: https://github.com/kamailio/kamailio/commit/dbd8e3899335eb8795763fd52b1f91a…
---
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
#### 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
- [x] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
There's an regression in 5.5 on `pike.so`. After c9dc0336a33a0ecfe776975be7fbfab8f3c91b48 the module has stopped to work. `mark_node()` had returned a NULL on every IP-address.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/2744
-- Commit Summary --
* pike: fixed regression
-- File Changes --
M src/modules/pike/ip_tree.c (4)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2744.patchhttps://github.com/kamailio/kamailio/pull/2744.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/2744
#### 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
- [x] Tested changes locally
- [ ] Related to issue #XXXX
#### Description
There's an regression after 42228552b72267786561704f120e3da3aac5fd89
`pike.so` was affected by the commit. RPC `pike.top` and `pike.list` commands didn't work.
`str_casesearch()` had returned NULL while text and needle were equal.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/2743
-- Commit Summary --
* core: ut - fixed str_casesearch()
-- File Changes --
M src/core/ut.c (4)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2743.patchhttps://github.com/kamailio/kamailio/pull/2743.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/2743
- add aes-cbc, des-ede3-cbc ipsec encryption algoriyhms
- improve nlmsg_seq choice for concurrent multi UEs Registrations at
same time
- before sending replies over IPsec check the existing
of opened TCP sockets
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [v] Commit message has the format required by CONTRIBUTING guide
- [v] Commits are split per component (core, individual modules, libs, utils, ...)
- [v] Each component has a single commit (if not, squash them into one commit)
- [v] 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
- [v] Small bug fix (non-breaking change which fixes an issue)
- [v] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)
#### Checklist:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [ ] PR should be backported to stable branches
- [v] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/2731
-- Commit Summary --
* ims_ipsec_pcscf: more algorithms, SA improvements
-- File Changes --
M src/modules/ims_ipsec_pcscf/cmd.c (29)
M src/modules/ims_ipsec_pcscf/ipsec.c (34)
M src/modules/ims_ipsec_pcscf/ipsec.h (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2731.patchhttps://github.com/kamailio/kamailio/pull/2731.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/2731
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [ ] Commit message has the format required by CONTRIBUTING guide
- [ ] Commits are split per component (core, individual modules, libs, utils, ...)
- [ ] Each component has a single commit (if not, squash them into one commit)
- [ ] 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
- [ ] 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:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [ ] PR should be backported to stable branches
- [ ] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/2741
-- Commit Summary --
* Fix typo in openserSIPRegUserLookupTable description
-- File Changes --
M src/modules/snmpstats/mibs/KAMAILIO-SIP-SERVER-MIB (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2741.patchhttps://github.com/kamailio/kamailio/pull/2741.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/2741
Module: kamailio
Branch: master
Commit: 7f017707c91659641ce9ca87ee4149b2e39379f7
URL: https://github.com/kamailio/kamailio/commit/7f017707c91659641ce9ca87ee4149b…
Author: Olle E. Johansson <oej(a)edvina.net>
Committer: Olle E. Johansson <oej(a)edvina.net>
Date: 2021-05-24T08:27:22+02:00
snmpstats: Fix typo in MIB
Reported by arkadiam on Github
---
Modified: src/modules/snmpstats/mibs/KAMAILIO-SIP-SERVER-MIB
---
Diff: https://github.com/kamailio/kamailio/commit/7f017707c91659641ce9ca87ee4149b…
Patch: https://github.com/kamailio/kamailio/commit/7f017707c91659641ce9ca87ee4149b…
---
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.3
Commit: bc5970811dc23ea0a01cb437688da731e44cc57f
URL: https://github.com/kamailio/kamailio/commit/bc5970811dc23ea0a01cb437688da73…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2021-05-18T21:31:46+02:00
modules: readme files regenerated - modules ... [skip ci]
---
Modified: src/modules/xhttp_prom/README
---
Diff: https://github.com/kamailio/kamailio/commit/bc5970811dc23ea0a01cb437688da73…
Patch: https://github.com/kamailio/kamailio/commit/bc5970811dc23ea0a01cb437688da73…
---
diff --git a/src/modules/xhttp_prom/README b/src/modules/xhttp_prom/README
index da9ddbabcb..c6e2908a57 100644
--- a/src/modules/xhttp_prom/README
+++ b/src/modules/xhttp_prom/README
@@ -417,7 +417,7 @@ prom_gauge_set("gg01", "-12.5");
...
# Definition of gg02 gauge with two labels method and IP
-modparam("xhttp_prom", "prom_gauge", "name=cnt02; label=method:IP;");
+modparam("xhttp_prom", "prom_gauge", "name=gg02; label=method:IP;");
...
# Assign 2.8 to value of gg02 gauge with labels method and IP. It creates the ga
uge if it does not exist.
Module: kamailio
Branch: 5.4
Commit: 0ef00531926d77351f1202d18b5ffe1c071aef12
URL: https://github.com/kamailio/kamailio/commit/0ef00531926d77351f1202d18b5ffe1…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2021-05-18T21:31:33+02:00
modules: readme files regenerated - modules ... [skip ci]
---
Modified: src/modules/xhttp_prom/README
---
Diff: https://github.com/kamailio/kamailio/commit/0ef00531926d77351f1202d18b5ffe1…
Patch: https://github.com/kamailio/kamailio/commit/0ef00531926d77351f1202d18b5ffe1…
---
diff --git a/src/modules/xhttp_prom/README b/src/modules/xhttp_prom/README
index da9ddbabcb..c6e2908a57 100644
--- a/src/modules/xhttp_prom/README
+++ b/src/modules/xhttp_prom/README
@@ -417,7 +417,7 @@ prom_gauge_set("gg01", "-12.5");
...
# Definition of gg02 gauge with two labels method and IP
-modparam("xhttp_prom", "prom_gauge", "name=cnt02; label=method:IP;");
+modparam("xhttp_prom", "prom_gauge", "name=gg02; label=method:IP;");
...
# Assign 2.8 to value of gg02 gauge with labels method and IP. It creates the ga
uge if it does not exist.
Module: kamailio
Branch: master
Commit: 6ad49854e23e7642dbda01a74378e9e8d8eef622
URL: https://github.com/kamailio/kamailio/commit/6ad49854e23e7642dbda01a74378e9e…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2021-05-18T21:16:17+02:00
modules: readme files regenerated - xhttp_prom ... [skip ci]
---
Modified: src/modules/xhttp_prom/README
---
Diff: https://github.com/kamailio/kamailio/commit/6ad49854e23e7642dbda01a74378e9e…
Patch: https://github.com/kamailio/kamailio/commit/6ad49854e23e7642dbda01a74378e9e…
---
diff --git a/src/modules/xhttp_prom/README b/src/modules/xhttp_prom/README
index 4ccdd4bd06..280d79caa8 100644
--- a/src/modules/xhttp_prom/README
+++ b/src/modules/xhttp_prom/README
@@ -512,7 +512,7 @@ prom_gauge_set("gg01", "-12.5");
...
# Definition of gg02 gauge with two labels method and IP
-modparam("xhttp_prom", "prom_gauge", "name=cnt02; label=method:IP;");
+modparam("xhttp_prom", "prom_gauge", "name=gg02; label=method:IP;");
...
# Assign 2.8 to value of gg02 gauge with labels method and IP. It creates the ga
uge if it does not exist.
Module: kamailio
Branch: 5.3
Commit: 9a381b67709d051c3451c7822443540a4e8ef45f
URL: https://github.com/kamailio/kamailio/commit/9a381b67709d051c3451c7822443540…
Author: Ovidiu Sas <osas(a)voipembedded.com>
Committer: Ovidiu Sas <osas(a)voipembedded.com>
Date: 2021-05-18T15:04:42-04:00
xhttp_prom: fix small copy/paste typo in documentation
(cherry picked from commit 2967a41f5537df81745a71226ba9e793ccbdd453)
---
Modified: src/modules/xhttp_prom/doc/xhttp_prom_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/9a381b67709d051c3451c7822443540…
Patch: https://github.com/kamailio/kamailio/commit/9a381b67709d051c3451c7822443540…
---
diff --git a/src/modules/xhttp_prom/doc/xhttp_prom_admin.xml b/src/modules/xhttp_prom/doc/xhttp_prom_admin.xml
index c098f2d382..278d51aac2 100644
--- a/src/modules/xhttp_prom/doc/xhttp_prom_admin.xml
+++ b/src/modules/xhttp_prom/doc/xhttp_prom_admin.xml
@@ -448,7 +448,7 @@ prom_gauge_set("gg01", "-12.5");
...
# Definition of gg02 gauge with two labels method and IP
-modparam("xhttp_prom", "prom_gauge", "name=cnt02; label=method:IP;");
+modparam("xhttp_prom", "prom_gauge", "name=gg02; label=method:IP;");
...
# Assign 2.8 to value of gg02 gauge with labels method and IP. It creates the gauge if it does not exist.
prom_gauge_set("gg02", "2.8", "push", "192.168.0.1");
Module: kamailio
Branch: 5.4
Commit: f2dc57a75ddad30205dadc59c435e423e3ee669f
URL: https://github.com/kamailio/kamailio/commit/f2dc57a75ddad30205dadc59c435e42…
Author: Ovidiu Sas <osas(a)voipembedded.com>
Committer: Ovidiu Sas <osas(a)voipembedded.com>
Date: 2021-05-18T15:03:14-04:00
xhttp_prom: fix small copy/paste typo in documentation
(cherry picked from commit 2967a41f5537df81745a71226ba9e793ccbdd453)
---
Modified: src/modules/xhttp_prom/doc/xhttp_prom_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/f2dc57a75ddad30205dadc59c435e42…
Patch: https://github.com/kamailio/kamailio/commit/f2dc57a75ddad30205dadc59c435e42…
---
diff --git a/src/modules/xhttp_prom/doc/xhttp_prom_admin.xml b/src/modules/xhttp_prom/doc/xhttp_prom_admin.xml
index c098f2d382..278d51aac2 100644
--- a/src/modules/xhttp_prom/doc/xhttp_prom_admin.xml
+++ b/src/modules/xhttp_prom/doc/xhttp_prom_admin.xml
@@ -448,7 +448,7 @@ prom_gauge_set("gg01", "-12.5");
...
# Definition of gg02 gauge with two labels method and IP
-modparam("xhttp_prom", "prom_gauge", "name=cnt02; label=method:IP;");
+modparam("xhttp_prom", "prom_gauge", "name=gg02; label=method:IP;");
...
# Assign 2.8 to value of gg02 gauge with labels method and IP. It creates the gauge if it does not exist.
prom_gauge_set("gg02", "2.8", "push", "192.168.0.1");
Module: kamailio
Branch: 5.5
Commit: ef765d3f4c9fb4bfa5949b2e10f208cd44c5e1db
URL: https://github.com/kamailio/kamailio/commit/ef765d3f4c9fb4bfa5949b2e10f208c…
Author: Ovidiu Sas <osas(a)voipembedded.com>
Committer: Ovidiu Sas <osas(a)voipembedded.com>
Date: 2021-05-18T15:01:44-04:00
xhttp_prom: fix small copy/paste typo in documentation
(cherry picked from commit 2967a41f5537df81745a71226ba9e793ccbdd453)
---
Modified: src/modules/xhttp_prom/doc/xhttp_prom_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/ef765d3f4c9fb4bfa5949b2e10f208c…
Patch: https://github.com/kamailio/kamailio/commit/ef765d3f4c9fb4bfa5949b2e10f208c…
---
diff --git a/src/modules/xhttp_prom/doc/xhttp_prom_admin.xml b/src/modules/xhttp_prom/doc/xhttp_prom_admin.xml
index eb331ab9c6..c35c44a024 100644
--- a/src/modules/xhttp_prom/doc/xhttp_prom_admin.xml
+++ b/src/modules/xhttp_prom/doc/xhttp_prom_admin.xml
@@ -566,7 +566,7 @@ prom_gauge_set("gg01", "-12.5");
...
# Definition of gg02 gauge with two labels method and IP
-modparam("xhttp_prom", "prom_gauge", "name=cnt02; label=method:IP;");
+modparam("xhttp_prom", "prom_gauge", "name=gg02; label=method:IP;");
...
# Assign 2.8 to value of gg02 gauge with labels method and IP. It creates the gauge if it does not exist.
prom_gauge_set("gg02", "2.8", "push", "192.168.0.1");
Module: kamailio
Branch: master
Commit: 2967a41f5537df81745a71226ba9e793ccbdd453
URL: https://github.com/kamailio/kamailio/commit/2967a41f5537df81745a71226ba9e79…
Author: Ovidiu Sas <osas(a)voipembedded.com>
Committer: Ovidiu Sas <osas(a)voipembedded.com>
Date: 2021-05-18T15:00:23-04:00
xhttp_prom: fix small copy/paste typo in documentation
---
Modified: src/modules/xhttp_prom/doc/xhttp_prom_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/2967a41f5537df81745a71226ba9e79…
Patch: https://github.com/kamailio/kamailio/commit/2967a41f5537df81745a71226ba9e79…
---
diff --git a/src/modules/xhttp_prom/doc/xhttp_prom_admin.xml b/src/modules/xhttp_prom/doc/xhttp_prom_admin.xml
index eb331ab9c6..c35c44a024 100644
--- a/src/modules/xhttp_prom/doc/xhttp_prom_admin.xml
+++ b/src/modules/xhttp_prom/doc/xhttp_prom_admin.xml
@@ -566,7 +566,7 @@ prom_gauge_set("gg01", "-12.5");
...
# Definition of gg02 gauge with two labels method and IP
-modparam("xhttp_prom", "prom_gauge", "name=cnt02; label=method:IP;");
+modparam("xhttp_prom", "prom_gauge", "name=gg02; label=method:IP;");
...
# Assign 2.8 to value of gg02 gauge with labels method and IP. It creates the gauge if it does not exist.
prom_gauge_set("gg02", "2.8", "push", "192.168.0.1");
Table version needs to be updated updated in uac-create.sql, to match uac_reg.h version 5. Version 5 reflects the addition of the contact_addr column.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/2735
-- Commit Summary --
* Update uac-create.sql to reflect UACREG_TABLE_VERSION in uac_reg.h
-- File Changes --
M utils/kamctl/db_sqlite/uac-create.sql (2)
M utils/kamctl/mysql/uac-create.sql (2)
M utils/kamctl/oracle/uac-create.sql (2)
M utils/kamctl/postgres/uac-create.sql (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2735.patchhttps://github.com/kamailio/kamailio/pull/2735.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/2735
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [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
- [ ] Small bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds new functionality)
- [x] Breaking change (fix or feature that would change existing functionality)
#### Checklist:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [ ] PR should be backported to stable branches
- [ ] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/2721
-- Commit Summary --
* tm: t_suspend.c - do not t_continue with reply if suspended
-- File Changes --
M src/modules/tm/t_suspend.c (6)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2721.patchhttps://github.com/kamailio/kamailio/pull/2721.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/2721
Module: kamailio
Branch: master
Commit: 061a66f88a91bc3e05224ef95043ebe4fec16dd1
URL: https://github.com/kamailio/kamailio/commit/061a66f88a91bc3e05224ef95043ebe…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2021-05-15T08:58:27+02:00
cfg_rpc: removed history and svn id
---
Modified: src/modules/cfg_rpc/Makefile
Modified: src/modules/cfg_rpc/cfg_rpc.c
---
Diff: https://github.com/kamailio/kamailio/commit/061a66f88a91bc3e05224ef95043ebe…
Patch: https://github.com/kamailio/kamailio/commit/061a66f88a91bc3e05224ef95043ebe…
---
diff --git a/src/modules/cfg_rpc/Makefile b/src/modules/cfg_rpc/Makefile
index a7a980d873..d4e58b4c92 100644
--- a/src/modules/cfg_rpc/Makefile
+++ b/src/modules/cfg_rpc/Makefile
@@ -1,5 +1,5 @@
# cfg_rpc module makefile
-#
+#
# WARNING: do not run this directly, it should be run by the main Makefile
include ../../Makefile.defs
diff --git a/src/modules/cfg_rpc/cfg_rpc.c b/src/modules/cfg_rpc/cfg_rpc.c
index e31429f713..30c72e6083 100644
--- a/src/modules/cfg_rpc/cfg_rpc.c
+++ b/src/modules/cfg_rpc/cfg_rpc.c
@@ -1,6 +1,4 @@
/*
- * $Id$
- *
* Copyright (C) 2007 iptelorg GmbH
*
* This file is part of ser, a free SIP server.
@@ -24,9 +22,6 @@
* along with this program; if not, write to the Free Software
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
*
- * History
- * -------
- * 2007-12-03 Initial version (Miklos)
*/
#include "../../core/sr_module.h"
### Description
In the case of using FQDN in
`listen = ... advertise ... `
FQDN is populated in the `Record-Route` header. In this case `is_first_hop()` function is not returning true even if `FQDN == myself` in replies.
#### Reproduction
Add FQND in advertise and check `is_first_hop()` result in a case of reply
#### Log Messages
```
siputils [sipops.c:185]: is_first_hop(): uri host is not an ip address
```
### Possible Solutions
Not use `is_first_hop()` in the case of reply/FQDN in advertising.
### Additional Information
version: kamailio 5.4.4 (x86_64/linux) e16352
flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MMAP, PKG_MALLOC, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR, USE_DST_BLACKLIST, HAVE_RESOLV_RES
ADAPTIVE_WAIT_LOOPS 1024, MAX_RECV_BUFFER_SIZE 262144, MAX_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB
poll method support: poll, epoll_lt, epoll_et, sigio_rt, select.
id: e16352
compiled on 09:50:10 May 7 2021 with gcc 4.8.5
* **Operating System**:
```
# uname -a
Linux fe-0513-2-dev 3.10.0-1160.21.1.el7.x86_64 #1 SMP Tue Mar 16 18:28:22 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
```
https://github.com/kamailio/kamailio/blob/02240711239149e2f5c4890a70ab158d1…
--
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/2732
Module: kamailio
Branch: 5.5
Commit: 9fe76d00f32aa3adfb0bfb2c7cbe325681089037
URL: https://github.com/kamailio/kamailio/commit/9fe76d00f32aa3adfb0bfb2c7cbe325…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2021-05-12T16:07:52+02:00
secsipid: docs - set GO111MODULE=off in installation example
(cherry picked from commit ecc8a34dae44883b125ae7a3da59d4a332e0b48c)
---
Modified: src/modules/secsipid/doc/secsipid_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/9fe76d00f32aa3adfb0bfb2c7cbe325…
Patch: https://github.com/kamailio/kamailio/commit/9fe76d00f32aa3adfb0bfb2c7cbe325…
---
diff --git a/src/modules/secsipid/doc/secsipid_admin.xml b/src/modules/secsipid/doc/secsipid_admin.xml
index d9d622af17..1e22e371fb 100644
--- a/src/modules/secsipid/doc/secsipid_admin.xml
+++ b/src/modules/secsipid/doc/secsipid_admin.xml
@@ -318,7 +318,7 @@ request_route {
<section id="secsipid.s.installation">
<title>Installation</title>
<para>
- The module needs "secsipdi_proc.so" module thatdepends on "libsecsipid",
+ The module needs "secsipdi_proc.so" module that depends on "libsecsipid",
which is a component of "sipsecidx" project from
https://github.com/asipto/secsipidx/. The library is
implemented in Go language, with generated C API and library. Until the
@@ -334,6 +334,7 @@ request_route {
<title>Libsecsipid Usage</title>
<programlisting format="linespecific">
...
+export GO111MODULE=off
go get https://github.com/asipto/secsipidx
cd $GOPATH/src/github.com/asipto/secsipidx/csecsipid/
make liba
@@ -351,6 +352,10 @@ make install
...
</programlisting>
</example>
+ <para>
+ For more details about compilation and installation of libsecsipid, see:
+ <ulink url="https://github.com/asipto/secsipidx">https://github.com/asipto/secsipidx</ulink>.
+ </para>
</section>
</chapter>
Module: kamailio
Branch: master
Commit: c441c9226635fb825096080029be7ba2d3ba8132
URL: https://github.com/kamailio/kamailio/commit/c441c9226635fb825096080029be7ba…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2021-05-12T13:14:12+02:00
pv: added $rpl(key) variable
- return attributes of the sip reply that is processed at that moment
- the key can be:
- duri - SIP URI corresponding to the destination address where the
reply is going to be sent based on 2nd via
- dhost - host part of duri
- dport - port part of duri
- dproto - proto of duri
- dprotoid - proto id of duri
- cntvia - number of via header bodies
---
Modified: src/modules/pv/pv.c
Modified: src/modules/pv/pv_core.c
Modified: src/modules/pv/pv_core.h
---
Diff: https://github.com/kamailio/kamailio/commit/c441c9226635fb825096080029be7ba…
Patch: https://github.com/kamailio/kamailio/commit/c441c9226635fb825096080029be7ba…
Module: kamailio
Branch: master
Commit: 600c2cf5c7349fc924b53199280baba8bdd329e7
URL: https://github.com/kamailio/kamailio/commit/600c2cf5c7349fc924b53199280baba…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2021-05-12T08:46:14+02:00
modules: readme files regenerated - secsipid ... [skip ci]
---
Modified: src/modules/secsipid/README
---
Diff: https://github.com/kamailio/kamailio/commit/600c2cf5c7349fc924b53199280baba…
Patch: https://github.com/kamailio/kamailio/commit/600c2cf5c7349fc924b53199280baba…
---
diff --git a/src/modules/secsipid/README b/src/modules/secsipid/README
index c96b7830c6..b42915943b 100644
--- a/src/modules/secsipid/README
+++ b/src/modules/secsipid/README
@@ -299,7 +299,7 @@ request_route {
5. Installation
- The module needs "secsipdi_proc.so" module thatdepends on
+ The module needs "secsipdi_proc.so" module that depends on
"libsecsipid", which is a component of "sipsecidx" project from
https://github.com/asipto/secsipidx/. The library is implemented in Go
language, with generated C API and library. Until the libsecsipid is
@@ -313,6 +313,7 @@ request_route {
Example 1.11. Libsecsipid Usage
...
+export GO111MODULE=off
go get https://github.com/asipto/secsipidx
cd $GOPATH/src/github.com/asipto/secsipidx/csecsipid/
make liba
@@ -328,3 +329,6 @@ make install
# make modules modules=src/modules/secsipid_proc
# make modules modules=src/modules/secsipid
...
+
+ For more details about compilation and installation of libsecsipid,
+ see: https://github.com/asipto/secsipidx.
Module: kamailio
Branch: master
Commit: ecc8a34dae44883b125ae7a3da59d4a332e0b48c
URL: https://github.com/kamailio/kamailio/commit/ecc8a34dae44883b125ae7a3da59d4a…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2021-05-12T08:32:37+02:00
secsipid: docs - set GO111MODULE=off in installation example
---
Modified: src/modules/secsipid/doc/secsipid_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/ecc8a34dae44883b125ae7a3da59d4a…
Patch: https://github.com/kamailio/kamailio/commit/ecc8a34dae44883b125ae7a3da59d4a…
---
diff --git a/src/modules/secsipid/doc/secsipid_admin.xml b/src/modules/secsipid/doc/secsipid_admin.xml
index d9d622af17..1e22e371fb 100644
--- a/src/modules/secsipid/doc/secsipid_admin.xml
+++ b/src/modules/secsipid/doc/secsipid_admin.xml
@@ -318,7 +318,7 @@ request_route {
<section id="secsipid.s.installation">
<title>Installation</title>
<para>
- The module needs "secsipdi_proc.so" module thatdepends on "libsecsipid",
+ The module needs "secsipdi_proc.so" module that depends on "libsecsipid",
which is a component of "sipsecidx" project from
https://github.com/asipto/secsipidx/. The library is
implemented in Go language, with generated C API and library. Until the
@@ -334,6 +334,7 @@ request_route {
<title>Libsecsipid Usage</title>
<programlisting format="linespecific">
...
+export GO111MODULE=off
go get https://github.com/asipto/secsipidx
cd $GOPATH/src/github.com/asipto/secsipidx/csecsipid/
make liba
@@ -351,6 +352,10 @@ make install
...
</programlisting>
</example>
+ <para>
+ For more details about compilation and installation of libsecsipid, see:
+ <ulink url="https://github.com/asipto/secsipidx">https://github.com/asipto/secsipidx</ulink>.
+ </para>
</section>
</chapter>
Hello,
We use Kamailio with KAZOO. We have set notifier_processes as follows:
modparam("presence", "notifier_processes", 1)
A value of 1 or greater seems to be required to avoid race conditions when multiple notify's need to be sent.
We've noticed noticed what seems to be a memory leak in Kamailio process. After several days, Kamailio will crash after consuming all system memory.
I have not found any other reports of this happening to others.
Does this seem familiar to anyone?
--
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/2733
Module: kamailio
Branch: master
Commit: ca886ffa64db8cc209dcc7098ffb69902464da72
URL: https://github.com/kamailio/kamailio/commit/ca886ffa64db8cc209dcc7098ffb699…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2021-05-11T17:31:12+02:00
modules: readme files regenerated - siputils ... [skip ci]
---
Modified: src/modules/siputils/README
---
Diff: https://github.com/kamailio/kamailio/commit/ca886ffa64db8cc209dcc7098ffb699…
Patch: https://github.com/kamailio/kamailio/commit/ca886ffa64db8cc209dcc7098ffb699…
---
diff --git a/src/modules/siputils/README b/src/modules/siputils/README
index a1aeae393c..453179d5a1 100644
--- a/src/modules/siputils/README
+++ b/src/modules/siputils/README
@@ -449,8 +449,9 @@ modparam("auth", "rpid_avp", "$avp(myrpid)")
Inserting the call-id in the internal list, which is checked when
further replies arrive. Any 183 reply that is received during the
- timeout value will be converted to a 180 message. Please note that you
- need to set a positive timeout value in order to use this function.
+ timeout value will be converted to a 180 message with removed SDP body.
+ Please note that you need to set a positive ring_timeout value in order
+ to use this function.
The function returns TRUE on success, and FALSE during processing
failures.
Module: kamailio
Branch: master
Commit: 212840c367e4b2945bb105e1fd087543f7fdf316
URL: https://github.com/kamailio/kamailio/commit/212840c367e4b2945bb105e1fd08754…
Author: Henning Westerholt <hw(a)skalatan.de>
Committer: Henning Westerholt <hw(a)skalatan.de>
Date: 2021-05-11T15:24:01Z
siptuils: small doc extension for ring filter functionality
---
Modified: src/modules/siputils/doc/siputils_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/212840c367e4b2945bb105e1fd08754…
Patch: https://github.com/kamailio/kamailio/commit/212840c367e4b2945bb105e1fd08754…
---
diff --git a/src/modules/siputils/doc/siputils_admin.xml b/src/modules/siputils/doc/siputils_admin.xml
index 5764c152d4..8e7140385d 100644
--- a/src/modules/siputils/doc/siputils_admin.xml
+++ b/src/modules/siputils/doc/siputils_admin.xml
@@ -281,8 +281,9 @@ modparam("auth", "rpid_avp", "$avp(myrpid)")
<para>
Inserting the call-id in the internal list, which is checked when
further replies arrive. Any 183 reply that is received during the
- timeout value will be converted to a 180 message. Please note that you
- need to set a positive timeout value in order to use this function.
+ timeout value will be converted to a 180 message with removed SDP body.
+ Please note that you need to set a positive ring_timeout value in order
+ to use this function.
</para>
<para>
The function returns TRUE on success, and FALSE during processing failures.
Module: kamailio
Branch: master
Commit: 9c2fea2df70e3fe609324a9d868efe9998bcda10
URL: https://github.com/kamailio/kamailio/commit/9c2fea2df70e3fe609324a9d868efe9…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2021-05-11T15:46:16+02:00
modules: readme files regenerated - siputils ... [skip ci]
---
Modified: src/modules/siputils/README
---
Diff: https://github.com/kamailio/kamailio/commit/9c2fea2df70e3fe609324a9d868efe9…
Patch: https://github.com/kamailio/kamailio/commit/9c2fea2df70e3fe609324a9d868efe9…
---
diff --git a/src/modules/siputils/README b/src/modules/siputils/README
index 87d3ecf9e4..a1aeae393c 100644
--- a/src/modules/siputils/README
+++ b/src/modules/siputils/README
@@ -96,7 +96,7 @@ Gabriel Vasile
4.30. is_reply()
4.31. is_gruu([uri])
4.32. is_supported(option)
- 4.33. is_first_hop()
+ 4.33. is_first_hop([mode])
4.34. sip_p_charging_vector(flags)
4.35. contact_param_encode(pname, saddr)
4.36. contact_param_decode(pname)
@@ -217,7 +217,7 @@ Chapter 1. Admin Guide
4.30. is_reply()
4.31. is_gruu([uri])
4.32. is_supported(option)
- 4.33. is_first_hop()
+ 4.33. is_first_hop([mode])
4.34. sip_p_charging_vector(flags)
4.35. contact_param_encode(pname, saddr)
4.36. contact_param_decode(pname)
@@ -438,7 +438,7 @@ modparam("auth", "rpid_avp", "$avp(myrpid)")
4.30. is_reply()
4.31. is_gruu([uri])
4.32. is_supported(option)
- 4.33. is_first_hop()
+ 4.33. is_first_hop([mode])
4.34. sip_p_charging_vector(flags)
4.35. contact_param_encode(pname, saddr)
4.36. contact_param_decode(pname)
@@ -994,24 +994,37 @@ if(is_gruu()) { ... }
if (is_supported("outbound")) { ... }
...
-4.33. is_first_hop()
+4.33. is_first_hop([mode])
The function returns true if the proxy is first hop after the original
- sender. For incoming SIP requests, it means there is only one Via
- header. For incoming SIP replies, it means that top Record-Route URI is
- 'myself' and source address is not matching it (to avoid detecting in
- case of local loops). The detection is done only when Record-Route has
- an IP address in its URI (for a domain, it returns -1/false).
+ sender based on a best effort estimation by checking Via and
+ Record-Route headers.
+
+ For incoming SIP requests, it means there is only one Via header.
+
+ For incoming SIP replies, if mode==0, it means that top Record-Route
+ URI is 'myself' and source address is not matching local IP (to avoid
+ detecting in case of local loops). Therefore for mode==0 the detection
+ is done only when Record-Route has an IP address in its URI (for a
+ domain, it returns -1/false). If mode==1, then the check of local IP is
+ no longer done, only if top Record-Route is myself, returning true also
+ if there is a domain, assuming that is expected no looping can happen
+ based on config rules.
Note that it does not detect spirals, which can have the condition for
replies true also in the case of additional SIP reply reception.
+ Parameter mode is optional and can be an integer or a varaible holding
+ an integer. If not provided, the behaviour is like mode==0.
+
This function can be used from ANY_ROUTE.
Example 1.42. is_first_hop() usage
...
if(is_first_hop()) { ... }
...
+if(is_first_hop_mode("1")) { ... }
+...
4.34. sip_p_charging_vector(flags)
Module: kamailio
Branch: master
Commit: 425ba921288c8d5a6ae3076287d0cf39c735a8fd
URL: https://github.com/kamailio/kamailio/commit/425ba921288c8d5a6ae3076287d0cf3…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2021-05-11T15:38:14+02:00
siputils: docs - updates to is_first_hop() for mode parameter
---
Modified: src/modules/siputils/doc/siputils_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/425ba921288c8d5a6ae3076287d0cf3…
Patch: https://github.com/kamailio/kamailio/commit/425ba921288c8d5a6ae3076287d0cf3…
---
diff --git a/src/modules/siputils/doc/siputils_admin.xml b/src/modules/siputils/doc/siputils_admin.xml
index 4667e42f63..5764c152d4 100644
--- a/src/modules/siputils/doc/siputils_admin.xml
+++ b/src/modules/siputils/doc/siputils_admin.xml
@@ -1169,22 +1169,35 @@ if (is_supported("outbound")) { ... }
</section>
<section id="siputils.f.is_first_hop">
<title>
- <function moreinfo="none">is_first_hop()</function>
+ <function moreinfo="none">is_first_hop([mode])</function>
</title>
<para>
The function returns true if the proxy is first hop after the
- original sender. For incoming SIP requests, it means there is only
- one Via header. For incoming SIP replies, it means that top
- Record-Route URI is 'myself' and source address is not matching it
- (to avoid detecting in case of local loops). The detection is done
- only when Record-Route has an IP address in its URI (for a domain,
- it returns -1/false).
+ original sender based on a best effort estimation by checking Via
+ and Record-Route headers.
+ </para>
+ <para>
+ For incoming SIP requests, it means there is only one Via header.
+ </para>
+ <para>
+ For incoming SIP replies, if mode==0, it means that top Record-Route
+ URI is 'myself' and source address is not matching local IP (to avoid
+ detecting in case of local loops). Therefore for mode==0 the detection
+ is done only when Record-Route has an IP address in its URI (for a domain,
+ it returns -1/false). If mode==1, then the check of local IP is no
+ longer done, only if top Record-Route is myself, returning true also if
+ there is a domain, assuming that is expected no looping can happen
+ based on config rules.
</para>
<para>
Note that it does not detect spirals, which can have the condition
for replies true also in the case of additional SIP reply reception.
</para>
<para>
+ Parameter mode is optional and can be an integer or a varaible holding
+ an integer. If not provided, the behaviour is like mode==0.
+ </para>
+ <para>
This function can be used from ANY_ROUTE.
</para>
<example>
@@ -1193,6 +1206,8 @@ if (is_supported("outbound")) { ... }
...
if(is_first_hop()) { ... }
...
+if(is_first_hop_mode("1")) { ... }
+...
</programlisting>
</example>
</section>
### Description
According to design logic HELD request need to send a carrier LIS server. Now lost module sends this request to the preconfigured server via `http_client/httpcon` param.
I prefer to use dynamic LIS server discovery according to [rfc7216#section-4](https://tools.ietf.org/html/rfc7216#section-4) and [rfc5986#section-4](https://tools.ietf.org/html/rfc5986#section-4).
So dynamic LIS discovery works as:
1. Kamailio send reverse `.in-addr.arpa.` or `.ip6.arpa.` DNS request and caller host DNS name;
2. Kamailio send `LIS:HELD` NAPTR request for a resolved caller hostname and LIS server.
3. Kamailio send HELD request for resolved LIS address.
So do get working dynamic LIS discovery required to implement two DNS requests.
If any DNS request will fail, then `lost_held_query` function returns an error code.
To define required dynamic LIS discovery I suggest use an empty string ("") or NULL value ($null) as the first ("con") function param.
This feature request for discussion with lost module author (Wolfgang Kampichler @wkampich) and other interest devs.
The ticket may be closed at any time.
--
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/2574
<!--
Kamailio Project uses GitHub Issues only for bugs in the code or feature requests. Please use this template only for feature requests.
If you have questions about using Kamailio or related to its configuration file, ask on sr-users mailing list:
* http://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users
If you have questions about developing extensions to Kamailio or its existing C code, ask on sr-dev mailing list:
* http://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
Please try to fill this template as much as possible for any issue. It helps the developers to troubleshoot the issue.
If you submit a feature request (or enhancement) add the description of what you would like to be added.
If there is no content to be filled in a section, the entire section can be removed.
You can delete the comments from the template sections when filling.
You can delete next line and everything above before submitting (it is a comment).
-->
### Description
Currently uac_reg_lookup don't have provision to pass the mode parameter but its implement in core fuction:
https://github.com/kamailio/kamailio/blob/951c9ac9adaccc1c35e152593cb3c2e7f…
This should be imported to kemi as well as cfg function.
<!--
Explain what you did, what you expected to happen, and what actually happened.
-->
### Expected behavior
lookup in uacreg should be supported using username based also, currently uuid based support is only there.
#### Actual observed behavior
#### Debugging Data
```
(paste your debugging data here)
```
#### Log Messages
<!--
Check the syslog file and if there are relevant log messages printed by Kamailio, add them next, or attach to issue, or provide a link to download them (e.g., to a pastebin site).
-->
```
(paste your log messages here)
```
#### SIP Traffic
<!--
If the issue is exposed by processing specific SIP messages, grab them with ngrep or save in a pcap file, then add them next, or attach to issue, or provide a link to download them (e.g., to a pastebin site).
-->
```
(paste your sip traffic here)
```
### Possible Solutions
<!--
If you found a solution or workaround for the issue, describe it. Ideally, provide a pull request with a improvement.
-->
### Additional Information
* **Kamailio Version** - output of `kamailio -v`
5.4.5
```
(paste your output here)
```
* **Operating System**:
<!--
Details about the operating system, the type: Linux (e.g.,: Debian 8.4, Ubuntu 16.04, CentOS 7.1, ...), MacOS, xBSD, Solaris, ...;
Kernel details (output of `uname -a`)
-->
```
(paste your output here)
```
--
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/2714
Module: kamailio
Branch: master
Commit: 624da2b7deb2539064be70a7566783fd36a15886
URL: https://github.com/kamailio/kamailio/commit/624da2b7deb2539064be70a7566783f…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2021-05-07T08:09:11+02:00
uac: docs for uac_reg_lookup_uri()
---
Modified: src/modules/uac/doc/uac_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/624da2b7deb2539064be70a7566783f…
Patch: https://github.com/kamailio/kamailio/commit/624da2b7deb2539064be70a7566783f…
---
diff --git a/src/modules/uac/doc/uac_admin.xml b/src/modules/uac/doc/uac_admin.xml
index 352d038ea2..52362af1e3 100644
--- a/src/modules/uac/doc/uac_admin.xml
+++ b/src/modules/uac/doc/uac_admin.xml
@@ -1005,6 +1005,31 @@ if(uac_reg_lookup("$rU", "$ru"))
{
lookup("location");
}
+...
+ </programlisting>
+ </example>
+ </section>
+ <section id="uac.f.uac_reg_lookup_uri">
+ <title>
+ <function moreinfo="none">uac_reg_lookup_uri(uri, dst)</function>
+ </title>
+ <para>
+ This function sets the PV dst to SIP URI that correspond to the URI user
+ in uac registrations table. The uri can contain pseudo-variables and
+ dst must be the name of a pseudo-variable.
+ </para>
+ <para>
+ This function can be used from ANY_ROUTE.
+ </para>
+ <example>
+ <title><function>uac_reg_lookup_uri</function> usage</title>
+ <programlisting format="linespecific">
+...
+
+if(uac_reg_lookup("$var(uri)", "$ru"))
+{
+ lookup("location");
+}
...
</programlisting>
</example>
```
LD (gcc) [M app_lua_sr.so] app_lua_sr.so
CC (gcc) [M lwsc.so] lwsc_mod.o
lwsc_mod.c: In function 'lwsc_get_endpoint':
lwsc_mod.c:533:13: error: 'struct lws_context_creation_info' has no member named 'ws_ping_pong_interval'
533 | ep->crtinfo.ws_ping_pong_interval = 5; /*secs*/
| ^
make[2]: *** [../../Makefile.rules:100: lwsc_mod.o] Error 1
make[1]: *** [Makefile:511: modules] Error 1
make[1]: Leaving directory '/root/rpmbuild/BUILD/kamailio-5.5.0/src'
make: *** [Makefile:34: every-module] Error 2
```
--
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/2727
With several days before releasing v5.5.0, let's see if there is any tuning that should be done to default kamailio.cfg.
Note that the proposals and discussions must focus around current config, if any loaded module should have different values for parameters, if the routing blocks should be adjusted to cope better with common use cases.
This is not for asking refactoring or adding major changes/features. In such cases, the option is to make PR to add to `misc/examples/` (new files or to existing files).
>From the discussions done for 5.4.0, the next were considered for the future, however I haven't seen much activity on the community forums related to them during past months, so I am not sure they worth at this moment, but let list them for a starting point:
* keepalive to be done with usrloc module to get round trip timer, etc
* dlgs module loaded to see the basic stats about active calls
--
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/2719
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [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:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [ ] PR should be backported to stable branches
- [x] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
It looks like the db query is made before checking the configuration parameters for the following features:
- internal keep-alive mechanism
- contact-expired callback execution
For example, although keepalive mode is turned off, the location table is periodically queried.
This is also true when there is no usrloc:contact-expired route.
In this commit, checks are added before these two db queries.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/2726
-- Commit Summary --
* Merge pull request #1 from kamailio/master
* Merge pull request #2 from kamailio/master
* Merge pull request #3 from kamailio/master
* usrloc: unnecessary periodic db queries cancelled
-- File Changes --
M src/modules/usrloc/dlist.c (4)
M src/modules/usrloc/udomain.c (6)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2726.patchhttps://github.com/kamailio/kamailio/pull/2726.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/2726
Allow configuring the hash table size. The default(4) is hardly usable
on anything but a test system.
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [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
- [ ] Small bug fix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)
#### Checklist:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [ ] PR should be backported to stable branches
- [x] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/2725
-- Commit Summary --
* uac: Add reg_hash_size parameter
-- File Changes --
M src/modules/uac/doc/uac_admin.xml (20)
M src/modules/uac/uac.c (1)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2725.patchhttps://github.com/kamailio/kamailio/pull/2725.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/2725
Kamailio v5.5.0 is out – it comes with 6 new modules and a considerable
set of improvements touching again more than 80 existing modules.
You can read detailed release notes at:
* https://www.kamailio.org/w/kamailio-v5-5-0-release-notes/
Many thanks to all developers and community members that made possible
this release.
v5.5.0 brings more flexibility and optimizations across many components,
enhancements to distributed message queue (dmq module), load balancer,
http client, STIR/SHAKEN support, IMS/VoLTE, rtpengine and tls as well
as to make available more functions to KEMI interface ... just to list
only a few here. Enjoy Kamailio v5.5.0!
Thank you for flying Kamailio!
Stay safe and healthy!
Daniel
--
Daniel-Constantin Mierla -- www.asipto.comwww.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio Advanced Training - Online
May 17-20, 2021 (Europe Timezone) - June 7-10, 2021 (America Timezone)
* https://www.asipto.com/sw/kamailio-advanced-training-online/
Hello
We have seen that in cases when callee sends a callid starting by "!!:",
which is also the default callid_prefix value, we do see that when reading
the $ci in the onreply routes it's changed for example like this
from Call-ID
!!:Y5HfD54-DIKfL56nz5TIYIioYVznLb1ID5TUYf1oz8zBYCl0LCllD9loDbD*
to Call-ID
\xA6\xF3\xF1\xFE\xFD:\xFEUq\xCA\xFD\xAB:\xF2\xA5\xA6Wۤ\x83\xABʫe\xFE\xF2\x87\xA7\e[;ã\xA5\xAC\xC1ɬ\xF3\xFD|\xDB\xFE\xAF
in that call we are actually not doing topology hiding, so we are dropping
the message on the topoh:msg-outgoing event route
This seems to cause to send that weird callid in the messages.
I guess that change should not be done. Could you please take a look?
thanks and best regards
david escartin
sonoc
--
<http://www.sonoc.io/>
David Escartín
NOC engineer
*www.sonoc.io* <http://www.sonoc.io/>
[image: LinkedIn] <https://www.linkedin.com/company/sonoc>
[image: Twitter] <https://twitter.com/sonoc_>
[image: Facebook] <https://www.facebook.com/SONOCio/>
This e-mail is for the exclusive use of its recipients and may contain
business secrets or other confidential or privileged information. Any
unauthorised use, copying, publication or distribution of this e-mail is
strictly prohibited. If you are not the intended recipient, please inform
us immediately by replying to this e-mail and delete it, including any
attachments or copies on your system.
In accordance with the GDPR (EU) 2016/679 and the LOPDGDD 3/2018, we inform
you that this e-mail address and/or any other personal data you have
provided us with will be treated by SONOC with absolute confidentiality and
with the only purpose of providing you with the requested services, due to
your condition as a client, supplier or because you have requested
information from us at any time. These data will only be kept for as long
as required to comply with legal obligations. You can exercise your rights
at any time by sending an e-mail to: *dataprotection(a)sonoc.io*
<dataprotection(a)sonoc.io>*.* <dataprotection(a)sonoc.io>
Hello,
after almost 4 weeks since development was frozen, it is time to plan
releasing the next major stable version, respectively 5.5.0, therefore I
propose to do it on Wednesday, May 5, 2021.
The branch 5.5 was already created about one week ago, several doc
resources were published (core/variables/transformations/rpcs/stats
cookbooks, alphabetic indexes, ...).
If anyone is testing an upgrade from branch 5.4 to 5.5 and encounters
changes that should be done in Kamailio config, add notes about at:
* https://www.kamailio.org/wiki/install/upgrade/5.4.x-to-5.5.0
Cheers,
Daniel
--
Daniel-Constantin Mierla -- www.asipto.comwww.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio Advanced Training - Online
May 17-20, 2021 (Europe Timezone) - June 7-10, 2021 (America Timezone)
* https://www.asipto.com/sw/kamailio-advanced-training-online/
Module: kamailio
Branch: 5.3
Commit: febeef975013eacc53e6b11766b7058f408282fc
URL: https://github.com/kamailio/kamailio/commit/febeef975013eacc53e6b11766b7058…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2021-05-04T01:01:32+02:00
modules: readme files regenerated - modules ... [skip ci]
---
Modified: src/modules/siputils/README
---
Diff: https://github.com/kamailio/kamailio/commit/febeef975013eacc53e6b11766b7058…
Patch: https://github.com/kamailio/kamailio/commit/febeef975013eacc53e6b11766b7058…
---
diff --git a/src/modules/siputils/README b/src/modules/siputils/README
index 74344a83c1..fcb315391b 100644
--- a/src/modules/siputils/README
+++ b/src/modules/siputils/README
@@ -705,7 +705,7 @@ if (is_alphanumex("$rU", "+.-_")) {
This function will encode uri-s inside Contact header in the following
manner sip:username:password@ip:port;transport=protocol goes
- sip:encoding_prefix*username*ip*port*protocol@hostpart.
+ sip:encoding_prefix*username*password*ip*port*protocol@hostpart.
* is the default separator and can be changed by setting the
contact_flds_separator module parameter.
@@ -731,9 +731,9 @@ if (src_ip == 10.0.0.0/8) encode_contact("natted_client","1.2.3.4");
4.18. decode_contact()
This function will decode the request URI. If the RURI is in the format
- sip:encoding_prefix*username*ip*port*protocol@hostpart it will be
- decoded to sip:username:password@ip:port;transport=protocol It uses the
- default set parameter for contact encoding separator.
+ sip:encoding_prefix*username*password*ip*port*protocol@hostpart it will
+ be decoded to sip:username:password@ip:port;transport=protocol. It uses
+ the default set parameter for contact encoding separator.
The function returns negative on error, 1 on success.
Module: kamailio
Branch: 5.4
Commit: e0c60cf12d465323bcf06899683b2b59dba7295c
URL: https://github.com/kamailio/kamailio/commit/e0c60cf12d465323bcf06899683b2b5…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2021-05-04T01:01:22+02:00
modules: readme files regenerated - modules ... [skip ci]
---
Modified: src/modules/siputils/README
---
Diff: https://github.com/kamailio/kamailio/commit/e0c60cf12d465323bcf06899683b2b5…
Patch: https://github.com/kamailio/kamailio/commit/e0c60cf12d465323bcf06899683b2b5…
---
diff --git a/src/modules/siputils/README b/src/modules/siputils/README
index 6a1ae76b9f..87d3ecf9e4 100644
--- a/src/modules/siputils/README
+++ b/src/modules/siputils/README
@@ -761,7 +761,7 @@ if (is_alphanumex("$rU", "+.-_")) {
This function will encode uri-s inside Contact header in the following
manner sip:username:password@ip:port;transport=protocol goes
- sip:encoding_prefix*username*ip*port*protocol@hostpart.
+ sip:encoding_prefix*username*password*ip*port*protocol@hostpart.
* is the default separator and can be changed by setting the
contact_flds_separator module parameter.
@@ -787,9 +787,9 @@ if (src_ip == 10.0.0.0/8) encode_contact("natted_client","1.2.3.4");
4.20. decode_contact()
This function will decode the request URI. If the RURI is in the format
- sip:encoding_prefix*username*ip*port*protocol@hostpart it will be
- decoded to sip:username:password@ip:port;transport=protocol It uses the
- default set parameter for contact encoding separator.
+ sip:encoding_prefix*username*password*ip*port*protocol@hostpart it will
+ be decoded to sip:username:password@ip:port;transport=protocol. It uses
+ the default set parameter for contact encoding separator.
The function returns negative on error, 1 on success.
Module: kamailio
Branch: 5.3
Commit: 09b510adbef121f94739d1fcf46bd92baf80ea77
URL: https://github.com/kamailio/kamailio/commit/09b510adbef121f94739d1fcf46bd92…
Author: Ovidiu Sas <osas(a)voipembedded.com>
Committer: Ovidiu Sas <osas(a)voipembedded.com>
Date: 2021-05-03T18:47:43-04:00
siputils: update docs for encode/decode_contact functions
(cherry picked from commit bf73b0d72177828e3c35f0fe318f7ef654c89b6b)
---
Modified: src/modules/siputils/doc/siputils_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/09b510adbef121f94739d1fcf46bd92…
Patch: https://github.com/kamailio/kamailio/commit/09b510adbef121f94739d1fcf46bd92…
---
diff --git a/src/modules/siputils/doc/siputils_admin.xml b/src/modules/siputils/doc/siputils_admin.xml
index d7ef042486..139c99903a 100644
--- a/src/modules/siputils/doc/siputils_admin.xml
+++ b/src/modules/siputils/doc/siputils_admin.xml
@@ -740,7 +740,7 @@ if (is_alphanumex("$rU", "+.-_")) {
This function will encode uri-s inside Contact header in the following
manner
sip:username:password@ip:port;transport=protocol goes
- sip:encoding_prefix*username*ip*port*protocol@hostpart.
+ sip:encoding_prefix*username*password*ip*port*protocol@hostpart.
</para>
<para>
* is the default separator and can be changed by setting the contact_flds_separator
@@ -784,9 +784,9 @@ if (src_ip == 10.0.0.0/8) encode_contact("natted_client","1.2.3.4");
</title>
<para>
This function will decode the request URI. If the RURI is in the format
- sip:encoding_prefix*username*ip*port*protocol@hostpart it will be decoded to
- sip:username:password@ip:port;transport=protocol It uses the default
- set parameter for contact encoding separator.
+ sip:encoding_prefix*username*password*ip*port*protocol@hostpart it will
+ be decoded to sip:username:password@ip:port;transport=protocol. It uses
+ the default set parameter for contact encoding separator.
</para>
<para>
The function returns negative on error, 1 on success.
Module: kamailio
Branch: 5.4
Commit: ea1239e57190da34b3043de3c8266a004554146e
URL: https://github.com/kamailio/kamailio/commit/ea1239e57190da34b3043de3c8266a0…
Author: Ovidiu Sas <osas(a)voipembedded.com>
Committer: Ovidiu Sas <osas(a)voipembedded.com>
Date: 2021-05-03T18:44:48-04:00
siputils: update docs for encode/decode_contact functions
(cherry picked from commit bf73b0d72177828e3c35f0fe318f7ef654c89b6b)
---
Modified: src/modules/siputils/doc/siputils_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/ea1239e57190da34b3043de3c8266a0…
Patch: https://github.com/kamailio/kamailio/commit/ea1239e57190da34b3043de3c8266a0…
---
diff --git a/src/modules/siputils/doc/siputils_admin.xml b/src/modules/siputils/doc/siputils_admin.xml
index 880105792a..4667e42f63 100644
--- a/src/modules/siputils/doc/siputils_admin.xml
+++ b/src/modules/siputils/doc/siputils_admin.xml
@@ -796,7 +796,7 @@ if (is_alphanumex("$rU", "+.-_")) {
This function will encode uri-s inside Contact header in the following
manner
sip:username:password@ip:port;transport=protocol goes
- sip:encoding_prefix*username*ip*port*protocol@hostpart.
+ sip:encoding_prefix*username*password*ip*port*protocol@hostpart.
</para>
<para>
* is the default separator and can be changed by setting the contact_flds_separator
@@ -840,9 +840,9 @@ if (src_ip == 10.0.0.0/8) encode_contact("natted_client","1.2.3.4");
</title>
<para>
This function will decode the request URI. If the RURI is in the format
- sip:encoding_prefix*username*ip*port*protocol@hostpart it will be decoded to
- sip:username:password@ip:port;transport=protocol It uses the default
- set parameter for contact encoding separator.
+ sip:encoding_prefix*username*password*ip*port*protocol@hostpart it will
+ be decoded to sip:username:password@ip:port;transport=protocol. It uses
+ the default set parameter for contact encoding separator.
</para>
<para>
The function returns negative on error, 1 on success.
Module: kamailio
Branch: 5.5
Commit: 2b9dc4713d85b1367631871205547aaf25a822a1
URL: https://github.com/kamailio/kamailio/commit/2b9dc4713d85b1367631871205547aa…
Author: Ovidiu Sas <osas(a)voipembedded.com>
Committer: Ovidiu Sas <osas(a)voipembedded.com>
Date: 2021-05-03T18:44:00-04:00
siputils: update docs for encode/decode_contact functions
(cherry picked from commit bf73b0d72177828e3c35f0fe318f7ef654c89b6b)
---
Modified: src/modules/siputils/doc/siputils_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/2b9dc4713d85b1367631871205547aa…
Patch: https://github.com/kamailio/kamailio/commit/2b9dc4713d85b1367631871205547aa…
---
diff --git a/src/modules/siputils/doc/siputils_admin.xml b/src/modules/siputils/doc/siputils_admin.xml
index 880105792a..4667e42f63 100644
--- a/src/modules/siputils/doc/siputils_admin.xml
+++ b/src/modules/siputils/doc/siputils_admin.xml
@@ -796,7 +796,7 @@ if (is_alphanumex("$rU", "+.-_")) {
This function will encode uri-s inside Contact header in the following
manner
sip:username:password@ip:port;transport=protocol goes
- sip:encoding_prefix*username*ip*port*protocol@hostpart.
+ sip:encoding_prefix*username*password*ip*port*protocol@hostpart.
</para>
<para>
* is the default separator and can be changed by setting the contact_flds_separator
@@ -840,9 +840,9 @@ if (src_ip == 10.0.0.0/8) encode_contact("natted_client","1.2.3.4");
</title>
<para>
This function will decode the request URI. If the RURI is in the format
- sip:encoding_prefix*username*ip*port*protocol@hostpart it will be decoded to
- sip:username:password@ip:port;transport=protocol It uses the default
- set parameter for contact encoding separator.
+ sip:encoding_prefix*username*password*ip*port*protocol@hostpart it will
+ be decoded to sip:username:password@ip:port;transport=protocol. It uses
+ the default set parameter for contact encoding separator.
</para>
<para>
The function returns negative on error, 1 on success.
Module: kamailio
Branch: master
Commit: ff4eea76a523851772da574437f99a9348578db4
URL: https://github.com/kamailio/kamailio/commit/ff4eea76a523851772da574437f99a9…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2021-05-04T00:31:09+02:00
modules: readme files regenerated - siputils ... [skip ci]
---
Modified: src/modules/siputils/README
---
Diff: https://github.com/kamailio/kamailio/commit/ff4eea76a523851772da574437f99a9…
Patch: https://github.com/kamailio/kamailio/commit/ff4eea76a523851772da574437f99a9…
---
diff --git a/src/modules/siputils/README b/src/modules/siputils/README
index 6a1ae76b9f..87d3ecf9e4 100644
--- a/src/modules/siputils/README
+++ b/src/modules/siputils/README
@@ -761,7 +761,7 @@ if (is_alphanumex("$rU", "+.-_")) {
This function will encode uri-s inside Contact header in the following
manner sip:username:password@ip:port;transport=protocol goes
- sip:encoding_prefix*username*ip*port*protocol@hostpart.
+ sip:encoding_prefix*username*password*ip*port*protocol@hostpart.
* is the default separator and can be changed by setting the
contact_flds_separator module parameter.
@@ -787,9 +787,9 @@ if (src_ip == 10.0.0.0/8) encode_contact("natted_client","1.2.3.4");
4.20. decode_contact()
This function will decode the request URI. If the RURI is in the format
- sip:encoding_prefix*username*ip*port*protocol@hostpart it will be
- decoded to sip:username:password@ip:port;transport=protocol It uses the
- default set parameter for contact encoding separator.
+ sip:encoding_prefix*username*password*ip*port*protocol@hostpart it will
+ be decoded to sip:username:password@ip:port;transport=protocol. It uses
+ the default set parameter for contact encoding separator.
The function returns negative on error, 1 on success.
Module: kamailio
Branch: master
Commit: bf73b0d72177828e3c35f0fe318f7ef654c89b6b
URL: https://github.com/kamailio/kamailio/commit/bf73b0d72177828e3c35f0fe318f7ef…
Author: Ovidiu Sas <osas(a)voipembedded.com>
Committer: Ovidiu Sas <osas(a)voipembedded.com>
Date: 2021-05-03T18:20:05-04:00
siputils: update docs for encode/decode_contact functions
---
Modified: src/modules/siputils/doc/siputils_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/bf73b0d72177828e3c35f0fe318f7ef…
Patch: https://github.com/kamailio/kamailio/commit/bf73b0d72177828e3c35f0fe318f7ef…
---
diff --git a/src/modules/siputils/doc/siputils_admin.xml b/src/modules/siputils/doc/siputils_admin.xml
index 880105792a..4667e42f63 100644
--- a/src/modules/siputils/doc/siputils_admin.xml
+++ b/src/modules/siputils/doc/siputils_admin.xml
@@ -796,7 +796,7 @@ if (is_alphanumex("$rU", "+.-_")) {
This function will encode uri-s inside Contact header in the following
manner
sip:username:password@ip:port;transport=protocol goes
- sip:encoding_prefix*username*ip*port*protocol@hostpart.
+ sip:encoding_prefix*username*password*ip*port*protocol@hostpart.
</para>
<para>
* is the default separator and can be changed by setting the contact_flds_separator
@@ -840,9 +840,9 @@ if (src_ip == 10.0.0.0/8) encode_contact("natted_client","1.2.3.4");
</title>
<para>
This function will decode the request URI. If the RURI is in the format
- sip:encoding_prefix*username*ip*port*protocol@hostpart it will be decoded to
- sip:username:password@ip:port;transport=protocol It uses the default
- set parameter for contact encoding separator.
+ sip:encoding_prefix*username*password*ip*port*protocol@hostpart it will
+ be decoded to sip:username:password@ip:port;transport=protocol. It uses
+ the default set parameter for contact encoding separator.
</para>
<para>
The function returns negative on error, 1 on success.
### Description
According [RFC5985](https://tools.ietf.org/html/rfc5985)
> The POST method is the only method REQUIRED for HELD. If a LIS
chooses to support GET or HEAD, it SHOULD consider the kind of
application doing the GET.
So not all LIS servers support the GET method and only the POST method always present.
Also [rfc6753#section-3.2](https://tools.ietf.org/html/rfc6753#section-3.2)
> An HTTP GET request to a HELD URI produces a HELD response as if the
following HELD request had been sent using HTTP POST:
<locationRequest xmlns="urn:ietf:params:xml:ns:geopriv:held">
<locationType exact="false">
geodetic civic
</locationType>
</locationRequest>
### Expected behavior
Dereference Geolocation URLs using POST method with content according [rfc6753#section-3.2](https://tools.ietf.org/html/rfc6753#section-3.2)
--
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/2641
**Description**
facing crashes the server, raised by qm_debug_check_frag().
**Troubleshooting**
The error message:
Jan 23 12:27:26 fep-1 /usr/local/fep-kamailio/sbin/kamailio[2893]: ERROR: <core> [core/tcp_main.c:4451]: handle_tcpconn_ev(): io_watch_del(3) failed: for 0x7fdbdc31f360, fd 31166
Jan 23 12:27:26 fep-1 /usr/local/fep-kamailio/sbin/kamailio[2893]: ERROR: <core> [core/tcp_main.c:4451]: handle_tcpconn_ev(): io_watch_del(3) failed: for 0x7fdbcbf232a8, fd 31326
Jan 23 12:27:26 fep-1 /usr/local/fep-kamailio/sbin/kamailio[2893]: ERROR: <core> [core/tcp_main.c:4451]: handle_tcpconn_ev(): io_watch_del(3) failed: for 0x7fdbd990b9e8, fd 31327
Jan 23 12:27:26 fep-1 /usr/local/fep-kamailio/sbin/kamailio[2893]: ERROR: <core> [core/tcp_main.c:4451]: handle_tcpconn_ev(): io_watch_del(3) failed: for 0x7fdbd89cd1d8, fd 31360
Jan 23 12:27:26 fep-1 /usr/local/fep-kamailio/sbin/kamailio[2893]: ERROR: <core> [core/tcp_main.c:4451]: handle_tcpconn_ev(): io_watch_del(3) failed: for 0x7fdbcf0bab28, fd 31434
Jan 23 12:27:26 fep-1 /usr/local/fep-kamailio/sbin/kamailio[2893]: ERROR: <core> [core/tcp_main.c:4451]: handle_tcpconn_ev(): io_watch_del(3) failed: for 0x7fdbc6e031b0, fd 31471
Jan 23 12:27:26 fep-1 /usr/local/fep-kamailio/sbin/kamailio[2893]: ERROR: <core> [core/tcp_main.c:4451]: handle_tcpconn_ev(): io_watch_del(3) failed: for 0x7fdbdb7fd598, fd 31472
Jan 23 12:27:26 fep-1 /usr/local/fep-kamailio/sbin/kamailio[2893]: ERROR: <core> [core/tcp_main.c:4451]: handle_tcpconn_ev(): io_watch_del(3) failed: for 0x7fdbdeb334c8, fd 31494
Jan 23 12:27:26 fep-1 /usr/local/fep-kamailio/sbin/kamailio[2893]: ERROR: <core> [core/tcp_main.c:4451]: handle_tcpconn_ev(): io_watch_del(3) failed: for 0x7fdbddee6de8, fd 31524
Jan 23 12:27:26 fep-1 /usr/local/fep-kamailio/sbin/kamailio[2893]: ERROR: <core> [core/tcp_main.c:4451]: handle_tcpconn_ev(): io_watch_del(3) failed: for 0x7fdbdb91ef70, fd 31576
Jan 23 12:27:26 fep-1 /usr/local/fep-kamailio/sbin/kamailio[2893]: ERROR: <core> [core/tcp_main.c:4451]: handle_tcpconn_ev(): io_watch_del(3) failed: for 0x7fdbc76a1da8, fd 31690
Jan 23 12:27:31 fep-1 /usr/local/fep-kamailio/sbin/kamailio[2893]: CRITICAL: <core> [core/io_wait.h:596]: io_watch_del(): invalid fd 31327, not in [0, 2054)
Jan 23 12:27:31 fep-1 /usr/local/fep-kamailio/sbin/kamailio[2893]: CRITICAL: <core> [core/io_wait.h:596]: io_watch_del(): invalid fd 31360, not in [0, 2054)
Jan 23 12:27:31 fep-1 /usr/local/fep-kamailio/sbin/kamailio[2893]: CRITICAL: <core> [core/io_wait.h:596]: io_watch_del(): invalid fd 31434, not in [0, 2054)
Jan 23 12:28:09 fep-1 /usr/local/fep-kamailio/sbin/kamailio[2809]: ALERT: <core> [main.c:777]: handle_sigs(): child process 2858 exited by a signal 6
Jan 23 12:28:09 fep-1 /usr/local/fep-kamailio/sbin/kamailio[2809]: ALERT: <core> [main.c:780]: handle_sigs(): core was generated
Jan 23 12:28:09 fep-1 /usr/local/fep-kamailio/sbin/kamailio[2809]: CRITICAL: <core> [core/mem/q_malloc.c:138]: qm_debug_check_frag(): BUG: qm: fragm. 0x7fdbd7e8a0e8 (address 0x7fdbd7e8a120) end overwritten (5fd4cd2e, abcdefed)! Memory allocator was called from core: core/usr_avp.c:626. Fragment marked by core: core/usr_avp.c:175. Exec from core/mem/q_malloc.c:511.
The output from GDB:
GNU gdb (GDB) Red Hat Enterprise Linux 7.6.1-110.el7
Copyright (C) 2013 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law. Type "show copying"
and "show warranty" for details.
This GDB was configured as "x86_64-redhat-linux-gnu".
For bug reporting instructions, please see:
<http://www.gnu.org/software/gdb/bugs/>...
Reading symbols from /usr/local/fep-kamailio/sbin/kamailio...done.
[New LWP 2809]
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib64/libthread_db.so.1".
Core was generated by `/usr/local/fep-kamailio/sbin/kamailio -f /usr/local/fep-kamailio/etc/kamailio/k'.
Program terminated with signal 6, Aborted.
#0 0x00007fde7acbf277 in raise () from /lib64/libc.so.6
Missing separate debuginfos, use: debuginfo-install cyrus-sasl-lib-2.1.26-21.el7.x86_64 glibc-2.17-222.el7.x86_64 keyutils-libs-1.5.8-3.el7.x86_64 krb5-libs-1.15.1-19.el7.x86_64 libcom_err-1.42.9-12.el7_5.x86_64 libcurl-7.29.0-54.el7.x86_64 libgcc-4.8.5-36.el7_6.1.x86_64 libidn-1.28-4.el7.x86_64 libselinux-2.5-12.el7.x86_64 libssh2-1.4.3-10.el7_2.1.x86_64 libstdc++-4.8.5-36.el7_6.1.x86_64 libunistring-0.9.3-9.el7.x86_64 nspr-4.21.0-1.el7.x86_64 nss-3.44.0-4.el7.x86_64 nss-softokn-freebl-3.44.0-5.el7.x86_64 nss-util-3.44.0-3.el7.x86_64 openldap-2.4.44-5.el7.x86_64 openssl-libs-1.0.2k-12.el7.x86_64 pcre-8.32-17.el7.x86_64 zlib-1.2.7-17.el7.x86_64
(gdb) bt
#0 0x00007fde7acbf277 in raise () from /lib64/libc.so.6
#1 0x00007fde7acc0968 in abort () from /lib64/libc.so.6
#2 0x00000000006d7c95 in qm_debug_check_frag (qm=0x7fdbc25ae000, f=0x7fdbd7e8a0e8, file=0x817155 "core: core/usr_avp.c", line=626, efile=0x83f7c5 "core/mem/q_malloc.c", eline=511) at core/mem/q_malloc.c:140
#3 0x00000000006db99e in qm_free (qmp=0x7fdbc25ae000, p=0x7fdbd7e8a120, file=0x817155 "core: core/usr_avp.c", func=0x8189a0 <__FUNCTION__.8526> "destroy_avp_list_unsafe", line=626, mname=0x817150 "core") at core/mem/q_malloc.c:511
#4 0x00000000005e7ede in destroy_avp_list_unsafe (list=0x7fdbd7e8a950) at core/usr_avp.c:626
#5 0x00007fde38d4bd8f in free_cell_helper (dead_cell=0x7fdbd7e8a750, silent=1, fname=0x7fde38e4267f "h_table.c", fline=466) at h_table.c:255
#6 0x00007fde38d4c9e1 in free_hash_table () at h_table.c:466
#7 0x00007fde38df9cd0 in tm_shutdown () at t_funcs.c:88
#8 0x0000000000581960 in destroy_modules () at core/sr_module.c:746
#9 0x000000000041cda7 in cleanup (show_status=1) at main.c:563
#10 0x000000000041e682 in shutdown_children (sig=15, show_status=1) at main.c:706
#11 0x0000000000421715 in handle_sigs () at main.c:806
#12 0x000000000042b9a7 in main_loop () at main.c:1817
#13 0x0000000000433b96 in main (argc=9, argv=0x7ffea05cd2c8) at main.c:2856
(gdb) bt full
#0 0x00007fde7acbf277 in raise () from /lib64/libc.so.6
No symbol table info available.
#1 0x00007fde7acc0968 in abort () from /lib64/libc.so.6
No symbol table info available.
#2 0x00000000006d7c95 in qm_debug_check_frag (qm=0x7fdbc25ae000, f=0x7fdbd7e8a0e8, file=0x817155 "core: core/usr_avp.c", line=626, efile=0x83f7c5 "core/mem/q_malloc.c", eline=511) at core/mem/q_malloc.c:140
p = 0x7ffea05cc3b0
__FUNCTION__ = "qm_debug_check_frag"
#3 0x00000000006db99e in qm_free (qmp=0x7fdbc25ae000, p=0x7fdbd7e8a120, file=0x817155 "core: core/usr_avp.c", func=0x8189a0 <__FUNCTION__.8526> "destroy_avp_list_unsafe", line=626, mname=0x817150 "core") at core/mem/q_malloc.c:511
qm = 0x7fdbc25ae000
f = 0x7fdbd7e8a0e8
size = 408
next = 0x7fdbd7e8a5c8
prev = 0x7fdbd7e8a1d8
__FUNCTION__ = "qm_free"
#4 0x00000000005e7ede in destroy_avp_list_unsafe (list=0x7fdbd7e8a950) at core/usr_avp.c:626
avp = 0x7fdbd7e8a080
foo = 0x7fdbd7e8a120
__FUNCTION__ = "destroy_avp_list_unsafe"
#5 0x00007fde38d4bd8f in free_cell_helper (dead_cell=0x7fdbd7e8a750, silent=1, fname=0x7fde38e4267f "h_table.c", fline=466) at h_table.c:255
b = 0x0
i = 1
rpl = 0x0
tt = 0x0
foo = 0x7fde34a5df80 <__FUNCTION__.7259>
cbs = 0x0
cbs_tmp = 0x7fde388fe570
__FUNCTION__ = "free_cell_helper"
#6 0x00007fde38d4c9e1 in free_hash_table () at h_table.c:466
p_cell = 0x7fdbd7e8a750
tmp_cell = 0x7fdbc27e3060
i = 40845
__FUNCTION__ = "free_hash_table"
#7 0x00007fde38df9cd0 in tm_shutdown () at t_funcs.c:88
__FUNCTION__ = "tm_shutdown"
#8 0x0000000000581960 in destroy_modules () at core/sr_module.c:746
t = 0x7fde3c51e4c0
foo = 0x7fde3c51e020
__FUNCTION__ = "destroy_modules"
#9 0x000000000041cda7 in cleanup (show_status=1) at main.c:563
memlog = 0
__FUNCTION__ = "cleanup"
#10 0x000000000041e682 in shutdown_children (sig=15, show_status=1) at main.c:706
__FUNCTION__ = "shutdown_children"
#11 0x0000000000421715 in handle_sigs () at main.c:806
chld = 0
chld_status = 134
any_chld_stopped = 1
memlog = -1027733624
__FUNCTION__ = "handle_sigs"
#12 0x000000000042b9a7 in main_loop () at main.c:1817
i = 10
pid = 2893
si = 0x0
si_desc = "udp receiver child=9 sock=10.50.7.18:5060\000:0:0:1]:5060\000:5060)\000\000\000\004\000\000\000\000\000\000\000\000\340Z\302\333\177\000\000\000\000\000\000\000\000\000\000 \006\276\302\333\177\000\000`\315\\\240\376\177\000\000\220\006\276\302\333\177\000\000r\fb3\336\177\000\000P\246\222<\336\177\000"
nrprocs = 10
woneinit = 1
__FUNCTION__ = "main_loop"
#13 0x0000000000433b96 in main (argc=9, argv=0x7ffea05cd2c8) at main.c:2856
**Additional Information**
Kamailio Version - output of Kamailio -v
version: kamailio 5.4.2 (x86_64/linux) f8885c
flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MMAP, PKG_MALLOC, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR, USE_DST_BLACKLIST, HAVE_RESOLV_RES
ADAPTIVE_WAIT_LOOPS 1024, MAX_RECV_BUFFER_SIZE 262144, MAX_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB
poll method support: poll, epoll_lt, epoll_et, sigio_rt, select.
id: f8885c
compiled on 04:30:53 Dec 16 2020 with gcc 4.8.5
**Operating System:**
NAME="CentOS Linux"
VERSION="7 (Core)"
ID="centos"
**Note:**
It looks similar to the following issue
https://github.com/kamailio/kamailio/issues/2503
--
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/2607
### Description
Hi, is there anyone working on adding support for [RFC 8898](https://tools.ietf.org/html/rfc8898) and do you know of any clients that already implement it or are working towards supporting it?
How could I go about starting such effort?
### RFC 8898
Third-Party Token-Based Authentication and Authorization for Session
Initiation Protocol (SIP)
Abstract
This document defines the "Bearer" authentication scheme for the
Session Initiation Protocol (SIP) and a mechanism by which user
authentication and SIP registration authorization is delegated to a
third party, using the OAuth 2.0 framework and OpenID Connect Core
1.0. This document updates RFC 3261 to provide guidance on how a SIP
User Agent Client (UAC) responds to a SIP 401/407 response that
contains multiple WWW-Authenticate/Proxy-Authenticate header fields.
...
--
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/2669
### Description
DNS core resolver fails in returning a valid IP when there are too many SRV results in the DNS reply.
It acts like if no records were found, so request is not relayed and a 478 reply is generated instead (in the example of a DNS name in $ru or $du).
### Troubleshooting
#### Reproduction
It is easy to reproduce with DNS failover + NAPTR enabled (cf parameters used far below)
and with such DNS records:
```
# dig +short NAPTR ko.sip.provider.com
50 30 "S" "SIP+D2U" "" _sip._udp.ko.sip.provider.com.
# dig +short SRV _sip._udp.ko.sip.provider.com.
10 10 5060 endpoint-01.k0.sip.provider.com.
10 10 5060 endpoint-02.k0.sip.provider.com.
10 10 5060 endpoint-03.k0.sip.provider.com.
10 10 5060 endpoint-04.k0.sip.provider.com.
10 10 5060 endpoint-05.k0.sip.provider.com.
10 10 5060 endpoint-06.k0.sip.provider.com.
10 10 5060 endpoint-07.k0.sip.provider.com.
10 10 5060 endpoint-08.k0.sip.provider.com.
10 10 5060 endpoint-09.k0.sip.provider.com.
# Each SRV result above has a corresponding
# 'A' record so that command below gives a correct IP:
# dig +short A endpoint-01.k0.sip.provider.com.
```
To reproduce, relay a request towards it, like:
`$du="sip:ko.sip.provider.com"`
#### Debugging data
One interesting thing is that Kamailio behaves exactly the same as the `sip-dig` tool.
But `sip-dig` seems to be limited on the DNS reply size it can handle (cf my comment below about the RFC).
Does Kamailio have this same kind of limitation regarding DNS resolution?
#### Log Messages
##### Failure example: with 9 SRV records
```
DEBUG: <core> [core/dns_cache.c:527]: _dns_hash_find(): (ko.sip.provider.com(26), 35), h=275
DEBUG: <core> [core/mem/q_malloc.c:482]: qm_free(): qm_free(0x7ff6a20f0000, 0x7ff6a27777d8), called from core: core/dns_cache.c: dns_destroy_entry(151)
DEBUG: <core> [core/mem/q_malloc.c:526]: qm_free(): freeing frag. 0x7ff6a27777a0 alloc'ed from core: core/dns_cache.c: dns_cache_mk_rd_entry(1110)
DEBUG: <core> [core/mem/q_malloc.c:374]: qm_malloc(): qm_malloc(0x7ff7234f4010, 58) called from core: core/resolve.c: get_record(862)
DEBUG: <core> [core/mem/q_malloc.c:419]: qm_malloc(): qm_malloc(0x7ff7234f4010, 64) returns address 0x7ff72363d8f8 frag. 0x7ff72363d8c0 (size=64) on 1 -th hit
DEBUG: <core> [core/mem/q_malloc.c:374]: qm_malloc(): qm_malloc(0x7ff7234f4010, 92) called from core: core/resolve.c: dns_naptr_parser(405)
DEBUG: <core> [core/mem/q_malloc.c:419]: qm_malloc(): qm_malloc(0x7ff7234f4010, 96) returns address 0x7ff72363d9a0 frag. 0x7ff72363d968 (size=96) on 1 -th hit
DEBUG: <core> [core/resolve.c:984]: get_record(): skipping 0 NS (p=0x558fb300dba7, end=0x558fb300dba7)
DEBUG: <core> [core/resolve.c:997]: get_record(): parsing 0 ARs (p=0x558fb300dba7, end=0x558fb300dba7)
DEBUG: <core> [core/mem/q_malloc.c:374]: qm_malloc(): qm_malloc(0x7ff6a20f0000, 216) called from core: core/dns_cache.c: dns_cache_mk_rd_entry(1110)
DEBUG: <core> [core/mem/q_malloc.c:419]: qm_malloc(): qm_malloc(0x7ff6a20f0000, 216) returns address 0x7ff6a27748a8 frag. 0x7ff6a2774870 (size=232) on 1 -th hit
DEBUG: <core> [core/mem/q_malloc.c:482]: qm_free(): qm_free(0x7ff7234f4010, 0x7ff72363d9a0), called from core: core/resolve.c: free_rdata_list(678)
DEBUG: <core> [core/mem/q_malloc.c:526]: qm_free(): freeing frag. 0x7ff72363d968 alloc'ed from core: core/resolve.c: dns_naptr_parser(405)
DEBUG: <core> [core/mem/q_malloc.c:482]: qm_free(): qm_free(0x7ff7234f4010, 0x7ff72363d8f8), called from core: core/resolve.c: free_rdata_list(679)
DEBUG: <core> [core/mem/q_malloc.c:526]: qm_free(): freeing frag. 0x7ff72363d8c0 alloc'ed from core: core/resolve.c: get_record(862)
DEBUG: <core> [core/dns_cache.c:1633]: dns_get_related(): (0x7ff6a27748a8 (ko.sip.provider.com, 35), 35, *(nil)) (0)
DEBUG: <core> [core/dns_cache.c:739]: dns_cache_add_unsafe(): adding ko.sip.provider.com(26) 35 (flags=0) at 275
DEBUG: <core> [core/dns_cache.c:2614]: dns_naptr_sip_iterate(): found a valid sip NAPTR rr _sip._udp.ko.sip.provider.com, proto 1
DEBUG: <core> [core/resolve.c:1182]: naptr_choose(): o:-1 w:-1 p:0, o:50 w:30 p:1
DEBUG: <core> [core/resolve.c:1197]: naptr_choose(): changed
DEBUG: <core> [core/dns_cache.c:2625]: dns_naptr_sip_iterate(): choosed NAPTR rr _sip._udp.ko.sip.provider.com, proto 1 tried: 0x0
DEBUG: <core> [core/dns_cache.c:527]: _dns_hash_find(): (_sip._udp.ko.sip.provider.com(36), 33), h=989
DEBUG: <core> [core/dns_cache.c:3041]: dns_srv_resolve_ip(): ("_sip._udp.ko.sip.provider.com", 0, 0), ret=-5, ip=
DEBUG: <core> [core/dns_cache.c:527]: _dns_hash_find(): (_sip._udp.ko.sip.provider.com(36), 33), h=989
DEBUG: <core> [core/dns_cache.c:3041]: dns_srv_resolve_ip(): ("_sip._udp.ko.sip.provider.com", 0, 0), ret=-5, ip=
DEBUG: <core> [core/dns_cache.c:527]: _dns_hash_find(): (_sip._tcp.ko.sip.provider.com(36), 33), h=772
DEBUG: <core> [core/dns_cache.c:3041]: dns_srv_resolve_ip(): ("_sip._tcp.ko.sip.provider.com", 0, 0), ret=-5, ip=
DEBUG: <core> [core/dns_cache.c:527]: _dns_hash_find(): (_sips._tcp.ko.sip.provider.com(37), 33), h=786
DEBUG: <core> [core/dns_cache.c:3041]: dns_srv_resolve_ip(): ("_sips._tcp.ko.sip.provider.com", 0, 0), ret=-5, ip=
DEBUG: <core> [core/dns_cache.c:527]: _dns_hash_find(): (ko.sip.provider.com(26), 1), h=275
DEBUG: <core> [core/dns_cache.c:2803]: dns_a_resolve(): (ko.sip.provider.com, 0) returning -7
DEBUG: <core> [core/dns_cache.c:3167]: dns_srv_sip_resolve(): (ko.sip.provider.com, 0, 0), ip, ret=-7
ERROR: tm [ut.h:284]: uri2dst2(): failed to resolve "ko.sip.provider.com" :unresolvable A or AAAA request (-7)
```
##### Comparison with a working example (only 3 SRV records)
```
DEBUG: <core> [core/dns_cache.c:527]: _dns_hash_find(): (ok.sip.provider.com(26), 35), h=275
DEBUG: <core> [core/mem/q_malloc.c:374]: qm_malloc(): qm_malloc(0x7ff7234f4010, 58) called from core: core/resolve.c: get_record(862)
DEBUG: <core> [core/mem/q_malloc.c:419]: qm_malloc(): qm_malloc(0x7ff7234f4010, 64) returns address 0x7ff723613ff8 frag. 0x7ff723613fc0 (size=64) on 1 -th hit
DEBUG: <core> [core/mem/q_malloc.c:374]: qm_malloc(): qm_malloc(0x7ff7234f4010, 92) called from core: core/resolve.c: dns_naptr_parser(405)
DEBUG: <core> [core/mem/q_malloc.c:419]: qm_malloc(): qm_malloc(0x7ff7234f4010, 96) returns address 0x7ff7236140a0 frag. 0x7ff723614068 (size=96) on 1 -th hit
DEBUG: <core> [core/resolve.c:984]: get_record(): skipping 0 NS (p=0x558fb300dba7, end=0x558fb300dba7)
DEBUG: <core> [core/resolve.c:997]: get_record(): parsing 0 ARs (p=0x558fb300dba7, end=0x558fb300dba7)
DEBUG: <core> [core/mem/q_malloc.c:374]: qm_malloc(): qm_malloc(0x7ff6a20f0000, 216) called from core: core/dns_cache.c: dns_cache_mk_rd_entry(1110)
DEBUG: <core> [core/mem/q_malloc.c:419]: qm_malloc(): qm_malloc(0x7ff6a20f0000, 216) returns address 0x7ff6a27755b8 frag. 0x7ff6a2775580 (size=376) on 1 -th hit
DEBUG: <core> [core/mem/q_malloc.c:482]: qm_free(): qm_free(0x7ff7234f4010, 0x7ff7236140a0), called from core: core/resolve.c: free_rdata_list(678)
DEBUG: <core> [core/mem/q_malloc.c:526]: qm_free(): freeing frag. 0x7ff723614068 alloc'ed from core: core/resolve.c: dns_naptr_parser(405)
DEBUG: <core> [core/mem/q_malloc.c:482]: qm_free(): qm_free(0x7ff7234f4010, 0x7ff723613ff8), called from core: core/resolve.c: free_rdata_list(679)
DEBUG: <core> [core/mem/q_malloc.c:526]: qm_free(): freeing frag. 0x7ff723613fc0 alloc'ed from core: core/resolve.c: get_record(862)
DEBUG: <core> [core/dns_cache.c:1633]: dns_get_related(): (0x7ff6a27755b8 (ok.sip.provider.com, 35), 35, *(nil)) (0)
DEBUG: <core> [core/dns_cache.c:739]: dns_cache_add_unsafe(): adding ok.sip.provider.com(26) 35 (flags=0) at 275
DEBUG: <core> [core/dns_cache.c:2614]: dns_naptr_sip_iterate(): found a valid sip NAPTR rr _sip._udp.ok.sip.provider.com, proto 1
DEBUG: <core> [core/resolve.c:1182]: naptr_choose(): o:-1 w:-1 p:0, o:50 w:30 p:1
DEBUG: <core> [core/resolve.c:1197]: naptr_choose(): changed
DEBUG: <core> [core/dns_cache.c:2625]: dns_naptr_sip_iterate(): choosed NAPTR rr _sip._udp.ok.sip.provider.com, proto 1 tried: 0x0
DEBUG: <core> [core/dns_cache.c:527]: _dns_hash_find(): (_sip._udp.ok.sip.provider.com(36), 33), h=989
DEBUG: <core> [core/mem/q_malloc.c:374]: qm_malloc(): qm_malloc(0x7ff7234f4010, 68) called from core: core/resolve.c: get_record(862)
DEBUG: <core> [core/mem/q_malloc.c:419]: qm_malloc(): qm_malloc(0x7ff7234f4010, 72) returns address 0x7ff723613ff8 frag. 0x7ff723613fc0 (size=72) on 1 -th hit
DEBUG: <core> [core/mem/q_malloc.c:374]: qm_malloc(): qm_malloc(0x7ff7234f4010, 46) called from core: core/resolve.c: dns_srv_parser(318)
DEBUG: <core> [core/mem/q_malloc.c:419]: qm_malloc(): qm_malloc(0x7ff7234f4010, 48) returns address 0x7ff7236140a8 frag. 0x7ff723614070 (size=48) on 1 -th hit
DEBUG: <core> [core/resolve.c:984]: get_record(): skipping 0 NS (p=0x558fb300dbb4, end=0x558fb300dbb4)
DEBUG: <core> [core/resolve.c:997]: get_record(): parsing 0 ARs (p=0x558fb300dbb4, end=0x558fb300dbb4)
DEBUG: <core> [core/mem/q_malloc.c:374]: qm_malloc(): qm_malloc(0x7ff6a20f0000, 176) called from core: core/dns_cache.c: dns_cache_mk_rd_entry(1110)
DEBUG: <core> [core/mem/q_malloc.c:419]: qm_malloc(): qm_malloc(0x7ff6a20f0000, 176) returns address 0x7ff6a2775900 frag. 0x7ff6a27758c8 (size=176) on 1 -th hit
DEBUG: <core> [core/mem/q_malloc.c:482]: qm_free(): qm_free(0x7ff7234f4010, 0x7ff7236140a8), called from core: core/resolve.c: free_rdata_list(678)
DEBUG: <core> [core/mem/q_malloc.c:526]: qm_free(): freeing frag. 0x7ff723614070 alloc'ed from core: core/resolve.c: dns_srv_parser(318)
DEBUG: <core> [core/mem/q_malloc.c:482]: qm_free(): qm_free(0x7ff7234f4010, 0x7ff723613ff8), called from core: core/resolve.c: free_rdata_list(679)
DEBUG: <core> [core/mem/q_malloc.c:526]: qm_free(): freeing frag. 0x7ff723613fc0 alloc'ed from core: core/resolve.c: get_record(862)
DEBUG: <core> [core/dns_cache.c:1633]: dns_get_related(): (0x7ff6a2775900 (_sip._udp.ok.sip.provider.com, 33), 33, *(nil)) (0)
DEBUG: <core> [core/dns_cache.c:739]: dns_cache_add_unsafe(): adding _sip._udp.ok.sip.provider.com(36) 33 (flags=0) at 989
DEBUG: <core> [core/dns_cache.c:2222]: dns_srv_get_nxt_rr(): (0x7ff6a2775900, 0, 0, 1457300027): selected 0/1 in grp. 0 (rand_w=0, rr=0x7ff6a2775968 rd=0x7ff6a2775980 p=10 w=10 rsum=10)
DEBUG: <core> [core/dns_cache.c:527]: _dns_hash_find(): (endpoint.ok.sip.provider.com(38), 1), h=530
DEBUG: <core> [core/mem/q_malloc.c:374]: qm_malloc(): qm_malloc(0x7ff7234f4010, 70) called from core: core/resolve.c: get_record(862)
DEBUG: <core> [core/mem/q_malloc.c:419]: qm_malloc(): qm_malloc(0x7ff7234f4010, 72) returns address 0x7ff723613ff8 frag. 0x7ff723613fc0 (size=72) on 1 -th hit
DEBUG: <core> [core/mem/q_malloc.c:374]: qm_malloc(): qm_malloc(0x7ff7234f4010, 4) called from core: core/resolve.c: dns_a_parser(474)
DEBUG: <core> [core/mem/q_malloc.c:419]: qm_malloc(): qm_malloc(0x7ff7234f4010, 8) returns address 0x7ff7236140a8 frag. 0x7ff723614070 (size=8) on 1 -th hit
DEBUG: <core> [core/resolve.c:984]: get_record(): skipping 0 NS (p=0x558fb300db8e, end=0x558fb300db8e)
DEBUG: <core> [core/resolve.c:997]: get_record(): parsing 0 ARs (p=0x558fb300db8e, end=0x558fb300db8e)
DEBUG: <core> [core/mem/q_malloc.c:374]: qm_malloc(): qm_malloc(0x7ff6a20f0000, 136) called from core: core/dns_cache.c: dns_cache_mk_rd_entry(1110)
DEBUG: <core> [core/mem/q_malloc.c:419]: qm_malloc(): qm_malloc(0x7ff6a20f0000, 136) returns address 0x7ff6a2775a18 frag. 0x7ff6a27759e0 (size=136) on 1 -th hit
DEBUG: <core> [core/mem/q_malloc.c:482]: qm_free(): qm_free(0x7ff7234f4010, 0x7ff7236140a8), called from core: core/resolve.c: free_rdata_list(678)
DEBUG: <core> [core/mem/q_malloc.c:526]: qm_free(): freeing frag. 0x7ff723614070 alloc'ed from core: core/resolve.c: dns_a_parser(474)
DEBUG: <core> [core/mem/q_malloc.c:482]: qm_free(): qm_free(0x7ff7234f4010, 0x7ff723613ff8), called from core: core/resolve.c: free_rdata_list(679)
DEBUG: <core> [core/mem/q_malloc.c:526]: qm_free(): freeing frag. 0x7ff723613fc0 alloc'ed from core: core/resolve.c: get_record(862)
DEBUG: <core> [core/dns_cache.c:1633]: dns_get_related(): (0x7ff6a2775a18 (endpoint.ok.sip.provider.com, 1), 1, *(nil)) (0)
DEBUG: <core> [core/dns_cache.c:739]: dns_cache_add_unsafe(): adding endpoint.ok.sip.provider.com(38) 1 (flags=0) at 530
DEBUG: <core> [core/dns_cache.c:2803]: dns_a_resolve(): (endpoint.ok.sip.provider.com, 0) returning 0
DEBUG: <core> [core/dns_cache.c:3041]: dns_srv_resolve_ip(): ("_sip._udp.ok.sip.provider.com", 0, 0), ret=0, ip=[RESOLVED_IP]
DEBUG: <core> [core/dns_cache.c:3241]: dns_naptr_sip_resolve(): (ok.sip.provider.com, 0, 0), srv0, ret=0
```
### Possible Solutions
I had a quick look inside the code and did not find any limitation about a maximum number of records.
There are some max defined in `dns_cache.c` but I did not found a relation between them and my issue.
Could there be a limitation in result size? Here is what I got from my RFCs reading regarding that:
* Extract from **RFC 2782 DNS RR** (mentioned in RFC 3263 as being the RFC to follow for implementing DNS in SIP):
> Currently there's a practical limit of 512 bytes for DNS replies.
> Until all resolvers can handle larger responses, domain administrators are strongly advised to keep their SRV replies below 512 bytes.
There is a RFC about how to deal with truncated messages:
> If a truncated response comes back from an SRV query, the rules described in RFC 2181 (https://tools.ietf.org/html/rfc2181#page-11) shall apply.
### Additional Information
* **Kamailio Version** - kamailio 5.3.8
```
dns_try_naptr=yes
dns_tcp_pref = 1
dns_udp_pref = 1
dns_tls_pref = 1
dns_srv_lb=yes
use_dns_failover=yes
use_dns_cache=yes
dns_cache_max_ttl=30
```
* **Operating System**: Debian 9.13 on Docker
Thanks
--
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/2651
### Description
Kamailio server has two legs that are connected to different networks.
I'm using Kamailio v.5.2.3 and the "enable_double_rr" is implicitly set to "1".
The leg "A" IP address is 10.28.80.1
The leg "B" IP address is 192.168.9.103
The call is initiated from 10.28.80.47
According to the "rr" module documentation, function record_route()
should insert two "Record_Route" header fields when a request is
accepted on one leg is should go out via the second leg. This works as
expected in case of UDP protocol:
INVITE sip:2601@192.168.0.178:49162;transport=udp SIP/2.0
Record-Route: <sip:192.168.9.103;r2=on;lr;did=e2c.a191>
Record-Route: <sip:10.28.80.1;r2=on;lr;did=e2c.a191>
Via: SIP/2.0/UDP 192.168.9.103;branch=z9hG4bKcfa5.d64ecbd87d5315b5993c4ccf16f86537.0
Via: SIP/2.0/UDP 10.28.80.47:5060;rport=5060;branch=z9hG4bK3a9e9a4d
But when the TCP protocol is used AND "listen" parameters are defined:
listen=udp:10.28.80.1:5060
listen=tcp:10.28.80.1:5060
listen=udp:192.168.9.103:5060
listen=tcp:192.168.9.103:5060
then the outbound message looks like this:
INVITE sip:2601@192.168.0.178:48758;transport=tcp SIP/2.0
Record-Route: <sip:10.28.80.1;transport=tcp;lr;did=bcd.72f>
Via: SIP/2.0/TCP 10.28.80.1;branch=z9hG4bKb9ca.26ea08654c9dbc32bb0dc6e3b6f92d45.0;i=3
Via: SIP/2.0/TCP 10.28.80.47:5060;rport=33976;branch=z9hG4bK5291fab1
There are two problems there:
a) only one Record-Route with leg "A" is inserted
b) the added "Via" header field contains the leg "A" IP address instead of expected leg "B" IP address (192.168.9.103). In the LAN trace, I see that in reality the message was sent from leg "B".
IMPORTANT: The problem occurs only when the "listen" parameters are defined in the Kamailio configuration. When the "listens" are not configured then everything works correctly.
### Troubleshooting
#### Reproduction
#### Debugging Data
#### Log Messages
http://updates.xorcom.com/~xorcom/kam-tcp-problem.tar.gz
The archive contains the following files:
* syslog-bad.log and bad.cap - Kamailio log and tcpdump trace of a call when the problem occurs
* syslog-good.log and good.cap - Kamailio log and tcpdump trace of a call when the problem doesn't occur
#### SIP Traffic
### Possible Solutions
### Additional Information
kamailio -v
version: kamailio 5.2.3 (x86_64/linux)
flags: STATS: Off, USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MEM, SHM_MMAP, PKG_MALLOC, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR, USE_DST_BLACKLIST, HAVE_RESOLV_RES
ADAPTIVE_WAIT_LOOPS=1024, MAX_RECV_BUFFER_SIZE 262144 MAX_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB
poll method support: poll, epoll_lt, epoll_et, sigio_rt, select.
id: unknown
compiled with gcc 7.3.0
***Operating System**:
Ubuntu 18.0.4
--
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/2033
Module: kamailio
Branch: 5.5
Commit: 39143255145ba6387cb001be4c70decfd3811aec
URL: https://github.com/kamailio/kamailio/commit/39143255145ba6387cb001be4c70dec…
Author: Wolfgang Kampichler <dev(a)kampichler.info>
Committer: Wolfgang Kampichler <dev(a)kampichler.info>
Date: 2021-05-03T12:33:00+02:00
lost: error response fix in lost_function(), lost_held_dereference() and lost_held_function()
- error variable was set to 0 (clean up) before return. Lost functions now returns 500
in case of LOST/HELD server error, or 400 in case of internal error, otherwise 2xx.
(cherry picked from commit 201c219a5496924e701cc2b17fb0a439ee530813)
---
Modified: src/modules/lost/functions.c
Modified: src/modules/lost/response.c
Modified: src/modules/lost/response.h
---
Diff: https://github.com/kamailio/kamailio/commit/39143255145ba6387cb001be4c70dec…
Patch: https://github.com/kamailio/kamailio/commit/39143255145ba6387cb001be4c70dec…
Module: kamailio
Branch: 5.5
Commit: d69c03625fad8c41eb18e23ee928b9f679140ef6
URL: https://github.com/kamailio/kamailio/commit/d69c03625fad8c41eb18e23ee928b9f…
Author: Wolfgang Kampichler <dev(a)kampichler.info>
Committer: Wolfgang Kampichler <dev(a)kampichler.info>
Date: 2021-05-03T12:34:35+02:00
lost: lost_held_dereference() documentation added and code comment corrected
(cherry picked from commit ae13686c80b03693ee2195d38a8c8eb0ede0d0dc)
---
Modified: src/modules/lost/doc/lost_admin.xml
Modified: src/modules/lost/response.c
---
Diff: https://github.com/kamailio/kamailio/commit/d69c03625fad8c41eb18e23ee928b9f…
Patch: https://github.com/kamailio/kamailio/commit/d69c03625fad8c41eb18e23ee928b9f…
---
diff --git a/src/modules/lost/doc/lost_admin.xml b/src/modules/lost/doc/lost_admin.xml
index 7f4597632b..f8065770d9 100644
--- a/src/modules/lost/doc/lost_admin.xml
+++ b/src/modules/lost/doc/lost_admin.xml
@@ -429,9 +429,23 @@ xlog("L_INFO", "HELD locationRequest: Result code $var(res)\nUrl: $var(url)\n$va
</para></listitem>
</itemizedlist>
<para>
- The return value is 200 on success, 400 if an internal error occured, or 500 if an
- error code is returned in the HELD response.
+ The return value is 200..203 on success, 400 if an internal error occured, or 500 if an
+ error code is returned in the HELD response. Success codes in detail are as follows:
</para>
+ <itemizedlist>
+ <listitem><para>
+ <emphasis>200</emphasis> - received 200 OK, but neither location-info nor locationURI element found
+ </para></listitem>
+ <listitem><para>
+ <emphasis>201</emphasis> - received 200 OK with locationURI (aka Location-by-Reference)
+ </para></listitem>
+ <listitem><para>
+ <emphasis>202</emphasis> - received 200 OK with location-info element (aka Location-by-Value)
+ </para></listitem>
+ <listitem><para>
+ <emphasis>203</emphasis> - received 200 OK with location-info and locationURI element
+ </para></listitem>
+ </itemizedlist>
<para>
This function can be used from REQUEST_ROUTE,
ONREPLY_ROUTE, FAILURE_ROUTE, and BRANCH_ROUTE.
diff --git a/src/modules/lost/response.c b/src/modules/lost/response.c
index caa86732ed..cc6d65ad2a 100644
--- a/src/modules/lost/response.c
+++ b/src/modules/lost/response.c
@@ -1006,8 +1006,8 @@ p_lost_fsr_t lost_parse_findServiceResponse(str ret)
* lost_check_HeldResponse(node)
* does a quick check of HELD dereference response and returns ...
* 0: neither location value nor reference found
- * 1: location value found
- * 2: location reference found
+ * 1: location reference found
+ * 2: location value found
* 3: location value and reference found
* multiple occurences are ignored
*/
Module: kamailio
Branch: master
Commit: 2058676daacc2feded18b872bd53c0514710d4bb
URL: https://github.com/kamailio/kamailio/commit/2058676daacc2feded18b872bd53c05…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2021-05-03T11:16:14+02:00
modules: readme files regenerated - lost ... [skip ci]
---
Modified: src/modules/lost/README
---
Diff: https://github.com/kamailio/kamailio/commit/2058676daacc2feded18b872bd53c05…
Patch: https://github.com/kamailio/kamailio/commit/2058676daacc2feded18b872bd53c05…
---
diff --git a/src/modules/lost/README b/src/modules/lost/README
index 14cf3b36a6..00fa5f2ceb 100644
--- a/src/modules/lost/README
+++ b/src/modules/lost/README
@@ -376,8 +376,15 @@ r(pidf)\n");
* pidf-lo - the PIDF-LO returned in the HELD locationRequest response
* error - any error code returned in the HELD response
- The return value is 200 on success, 400 if an internal error occured,
- or 500 if an error code is returned in the HELD response.
+ The return value is 200..203 on success, 400 if an internal error
+ occured, or 500 if an error code is returned in the HELD response.
+ Success codes in detail are as follows:
+ * 200 - received 200 OK, but neither location-info nor locationURI
+ element found
+ * 201 - received 200 OK with locationURI (aka Location-by-Reference)
+ * 202 - received 200 OK with location-info element (aka
+ Location-by-Value)
+ * 203 - received 200 OK with location-info and locationURI element
This function can be used from REQUEST_ROUTE, ONREPLY_ROUTE,
FAILURE_ROUTE, and BRANCH_ROUTE.
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [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
- [ ] Small bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds new functionality)
- [x ] Breaking change (fix or feature that would change existing functionality)
#### Checklist:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [x ] PR should be backported to stable branches
- [x ] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
Add a new parameter validity to jwt_generate(prvkey, alg, claims) in order to set a validity period for generated jwt token.
I know branch 5.5 is almost released but I think that this functionality is very important for this new module and I hope it can be added to this release.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/2722
-- Commit Summary --
* cplc: add a configuration route before redirected call (key: redirect_route)
* pua_dialoginfo: disable publish notifications for subsequent requests (key: publish_dialog_req_within)
* Merge branch 'master' of https://github.com/kamailio/kamailio
* jwt: token may have a validity period (number of seconds)
-- File Changes --
M src/modules/jwt/doc/jwt.xml (9)
M src/modules/jwt/doc/jwt_admin.xml (5)
M src/modules/jwt/jwt_mod.c (17)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2722.patchhttps://github.com/kamailio/kamailio/pull/2722.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/2722
Module: kamailio
Branch: master
Commit: ae13686c80b03693ee2195d38a8c8eb0ede0d0dc
URL: https://github.com/kamailio/kamailio/commit/ae13686c80b03693ee2195d38a8c8eb…
Author: Wolfgang Kampichler <dev(a)kampichler.info>
Committer: Wolfgang Kampichler <dev(a)kampichler.info>
Date: 2021-05-03T11:00:41+02:00
lost: lost_held_dereference() documentation added and code comment corrected
---
Modified: src/modules/lost/doc/lost_admin.xml
Modified: src/modules/lost/response.c
---
Diff: https://github.com/kamailio/kamailio/commit/ae13686c80b03693ee2195d38a8c8eb…
Patch: https://github.com/kamailio/kamailio/commit/ae13686c80b03693ee2195d38a8c8eb…
---
diff --git a/src/modules/lost/doc/lost_admin.xml b/src/modules/lost/doc/lost_admin.xml
index 7f4597632b..f8065770d9 100644
--- a/src/modules/lost/doc/lost_admin.xml
+++ b/src/modules/lost/doc/lost_admin.xml
@@ -429,9 +429,23 @@ xlog("L_INFO", "HELD locationRequest: Result code $var(res)\nUrl: $var(url)\n$va
</para></listitem>
</itemizedlist>
<para>
- The return value is 200 on success, 400 if an internal error occured, or 500 if an
- error code is returned in the HELD response.
+ The return value is 200..203 on success, 400 if an internal error occured, or 500 if an
+ error code is returned in the HELD response. Success codes in detail are as follows:
</para>
+ <itemizedlist>
+ <listitem><para>
+ <emphasis>200</emphasis> - received 200 OK, but neither location-info nor locationURI element found
+ </para></listitem>
+ <listitem><para>
+ <emphasis>201</emphasis> - received 200 OK with locationURI (aka Location-by-Reference)
+ </para></listitem>
+ <listitem><para>
+ <emphasis>202</emphasis> - received 200 OK with location-info element (aka Location-by-Value)
+ </para></listitem>
+ <listitem><para>
+ <emphasis>203</emphasis> - received 200 OK with location-info and locationURI element
+ </para></listitem>
+ </itemizedlist>
<para>
This function can be used from REQUEST_ROUTE,
ONREPLY_ROUTE, FAILURE_ROUTE, and BRANCH_ROUTE.
diff --git a/src/modules/lost/response.c b/src/modules/lost/response.c
index caa86732ed..cc6d65ad2a 100644
--- a/src/modules/lost/response.c
+++ b/src/modules/lost/response.c
@@ -1006,8 +1006,8 @@ p_lost_fsr_t lost_parse_findServiceResponse(str ret)
* lost_check_HeldResponse(node)
* does a quick check of HELD dereference response and returns ...
* 0: neither location value nor reference found
- * 1: location value found
- * 2: location reference found
+ * 1: location reference found
+ * 2: location value found
* 3: location value and reference found
* multiple occurences are ignored
*/
Module: kamailio
Branch: 5.5
Commit: f76512226f8db95b7ecd8c881475e2f53f2aa900
URL: https://github.com/kamailio/kamailio/commit/f76512226f8db95b7ecd8c881475e2f…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2021-05-03T09:45:02+02:00
jwt: more details about claims parameter format
(cherry picked from commit 7d24a7afe02bf1f7c8ddddfcfc9f1c287296c996)
---
Modified: src/modules/jwt/doc/jwt_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/f76512226f8db95b7ecd8c881475e2f…
Patch: https://github.com/kamailio/kamailio/commit/f76512226f8db95b7ecd8c881475e2f…
---
diff --git a/src/modules/jwt/doc/jwt_admin.xml b/src/modules/jwt/doc/jwt_admin.xml
index dba928b523..36d3d0d0f7 100644
--- a/src/modules/jwt/doc/jwt_admin.xml
+++ b/src/modules/jwt/doc/jwt_admin.xml
@@ -109,6 +109,10 @@ modparam("jwt", "key_mode", 1)
<para>
claims - the list of claims to be added to JWT, in the format
"name1=value1;name2=value2;..." (same as the SIP parameters format).
+ The string values can be enclosed in single or double quotes. If a
+ value is not eclosed in between quotes, it is added as numeric
+ value if it is successfully converted to a long value, otherwise is
+ added as string value.
</para>
</listitem>
</itemizedlist>
@@ -120,7 +124,7 @@ modparam("jwt", "key_mode", 1)
<programlisting format="linespecific">
...
jwt_generate("/path/to/prvkey.pem", "RS256",
- "caller=$fU;callee=$tU;callid=$ci");
+ "caller='$fU';callee='$tU';callid='$ci';index=100");
...
</programlisting>
</example>
@@ -151,7 +155,8 @@ modparam("jwt", "key_mode", 1)
<listitem>
<para>
claims - the list of claims to be checked they are in the JWT, in the format
- "name1=value1;name2=value2;..." (same as the SIP parameters format).
+ "name1=value1;name2=value2;..." (same as the SIP parameters format,
+ see also the description of claims parameter for jwt_generate()).
</para>
</listitem>
<listitem>
@@ -168,7 +173,7 @@ modparam("jwt", "key_mode", 1)
<programlisting format="linespecific">
...
if(!jwt_verify("/path/to/pubkey.pem", "RS256",
- "caller=$fU;callee=$tU;callid=$ci",
+ "caller='$fU';callee='$tU';callid='$ci';index=100",
"$var(jwt)") {
xwarn("failed to verify jwt\n");
}
@@ -206,7 +211,7 @@ modparam("jwt", "key_mode", 1)
<programlisting format="linespecific">
...
jwt_generate("/path/to/prvkey.pem", "RS256",
- "caller=$fU;callee=$tU;callid=$ci");
+ "caller='$fU';callee='$tU';callid='$ci';index=100");
xinfo("jwt is: $jwt(val)");
...
</programlisting>
Hello,
the "new in Kamailio 5.5.x" document has been updated. Again, a really impressive set of changes that lead to this release:
Over 1350 commits have been done in the last development period from over 70 individual contributors.
You find the document at the usual place in our wiki: https://www.kamailio.org/wiki/features/new-in-5.5.x
Many thanks to everybody who contributed to Kamailio.
Help in updating the documentation (e.g. core cookbook, pseudo-variable and transformation section) in the wiki is really welcome, as some content is probably still missing there.
Best regards,
Henning
--
Henning Westerholt - https://skalatan.de/blog/
Kamailio services - https://gilawa.com<https://gilawa.com/>
Module: kamailio
Branch: master
Commit: 26e3adfd0af949cf6e611798ad02506d927a8c8f
URL: https://github.com/kamailio/kamailio/commit/26e3adfd0af949cf6e611798ad02506…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2021-05-03T08:01:12+02:00
modules: readme files regenerated - jwt ... [skip ci]
---
Modified: src/modules/jwt/README
---
Diff: https://github.com/kamailio/kamailio/commit/26e3adfd0af949cf6e611798ad02506…
Patch: https://github.com/kamailio/kamailio/commit/26e3adfd0af949cf6e611798ad02506…
---
diff --git a/src/modules/jwt/README b/src/modules/jwt/README
index 7a7400a4b1..459f10a822 100644
--- a/src/modules/jwt/README
+++ b/src/modules/jwt/README
@@ -122,14 +122,17 @@ modparam("jwt", "key_mode", 1)
libjwt (e.g., RS256, HS256, ES256, ...)
* claims - the list of claims to be added to JWT, in the format
"name1=value1;name2=value2;..." (same as the SIP parameters
- format).
+ format). The string values can be enclosed in single or double
+ quotes. If a value is not eclosed in between quotes, it is added as
+ numeric value if it is successfully converted to a long value,
+ otherwise is added as string value.
This function can be used from ANY_ROUTE.
Example 1.2. jwt_generate usage
...
jwt_generate("/path/to/prvkey.pem", "RS256",
- "caller=$fU;callee=$tU;callid=$ci");
+ "caller='$fU';callee='$tU';callid='$ci';index=100");
...
4.2. jwt_verify(pubkey, alg, claims, jwtval)
@@ -142,7 +145,8 @@ modparam("jwt", "key_mode", 1)
libjwt (e.g., RS256, HS256, ES256, ...)
* claims - the list of claims to be checked they are in the JWT, in
the format "name1=value1;name2=value2;..." (same as the SIP
- parameters format).
+ parameters format, see also the description of claims parameter for
+ jwt_generate()).
* jwtval - the value of the JWT to verify
This function can be used from ANY_ROUTE.
@@ -150,7 +154,7 @@ modparam("jwt", "key_mode", 1)
Example 1.3. jwt_verify usage
...
if(!jwt_verify("/path/to/pubkey.pem", "RS256",
- "caller=$fU;callee=$tU;callid=$ci",
+ "caller='$fU';callee='$tU';callid='$ci';index=100",
"$var(jwt)") {
xwarn("failed to verify jwt\n");
}
@@ -171,6 +175,6 @@ modparam("jwt", "key_mode", 1)
Example 1.4. $jwt(name) usage
...
jwt_generate("/path/to/prvkey.pem", "RS256",
- "caller=$fU;callee=$tU;callid=$ci");
+ "caller='$fU';callee='$tU';callid='$ci';index=100");
xinfo("jwt is: $jwt(val)");
...
Module: kamailio
Branch: master
Commit: 7d24a7afe02bf1f7c8ddddfcfc9f1c287296c996
URL: https://github.com/kamailio/kamailio/commit/7d24a7afe02bf1f7c8ddddfcfc9f1c2…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2021-05-03T08:00:32+02:00
jwt: more details about claims parameter format
---
Modified: src/modules/jwt/doc/jwt_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/7d24a7afe02bf1f7c8ddddfcfc9f1c2…
Patch: https://github.com/kamailio/kamailio/commit/7d24a7afe02bf1f7c8ddddfcfc9f1c2…
---
diff --git a/src/modules/jwt/doc/jwt_admin.xml b/src/modules/jwt/doc/jwt_admin.xml
index dba928b523..36d3d0d0f7 100644
--- a/src/modules/jwt/doc/jwt_admin.xml
+++ b/src/modules/jwt/doc/jwt_admin.xml
@@ -109,6 +109,10 @@ modparam("jwt", "key_mode", 1)
<para>
claims - the list of claims to be added to JWT, in the format
"name1=value1;name2=value2;..." (same as the SIP parameters format).
+ The string values can be enclosed in single or double quotes. If a
+ value is not eclosed in between quotes, it is added as numeric
+ value if it is successfully converted to a long value, otherwise is
+ added as string value.
</para>
</listitem>
</itemizedlist>
@@ -120,7 +124,7 @@ modparam("jwt", "key_mode", 1)
<programlisting format="linespecific">
...
jwt_generate("/path/to/prvkey.pem", "RS256",
- "caller=$fU;callee=$tU;callid=$ci");
+ "caller='$fU';callee='$tU';callid='$ci';index=100");
...
</programlisting>
</example>
@@ -151,7 +155,8 @@ modparam("jwt", "key_mode", 1)
<listitem>
<para>
claims - the list of claims to be checked they are in the JWT, in the format
- "name1=value1;name2=value2;..." (same as the SIP parameters format).
+ "name1=value1;name2=value2;..." (same as the SIP parameters format,
+ see also the description of claims parameter for jwt_generate()).
</para>
</listitem>
<listitem>
@@ -168,7 +173,7 @@ modparam("jwt", "key_mode", 1)
<programlisting format="linespecific">
...
if(!jwt_verify("/path/to/pubkey.pem", "RS256",
- "caller=$fU;callee=$tU;callid=$ci",
+ "caller='$fU';callee='$tU';callid='$ci';index=100",
"$var(jwt)") {
xwarn("failed to verify jwt\n");
}
@@ -206,7 +211,7 @@ modparam("jwt", "key_mode", 1)
<programlisting format="linespecific">
...
jwt_generate("/path/to/prvkey.pem", "RS256",
- "caller=$fU;callee=$tU;callid=$ci");
+ "caller='$fU';callee='$tU';callid='$ci';index=100");
xinfo("jwt is: $jwt(val)");
...
</programlisting>
Daniel,
I studied a bit topos_redis source and noticed that it lacks x_context
td and tt keys.
Is there something else missing in order to make spirals work also when
redis backend is in use?
-- Juha
Hello,
the formal notification that the development for the next major version
5.5.0 is now frozen. The focus has to be on testing the master branch.
Also, the master branch should not get commits with new features till
the branch 5.5 is created, expected to happen in 2-4 weeks, a matter of
how testing goes on. Meanwhile, the commits with new features in the C
code can be pushed to personal branches, new pull requests can still be
done, but they will be merged after branching 5.5.
Can still be done commits with documentation improvements, enhancements
to related tools (e.g., kamctl, kamcmd), merging exiting pull requests
at this
moment, exporting missing KEMI functions and completing the functionality
of the new modules added in 5.5.
Once the branch 5.5 is created, new features can be pushed again to
master branch as usual. From that moment, the v5.5.0 should be out very
soon, time used for further testing but also preparing the release of
packages.
If someone is not sure if a commit brings a new feature, just make a
pull request and it can be discussed there on github portal or via
sr-dev mailing list.
A summary of what is new in upcoming 5.5 is going to be built at:
* https://www.kamailio.org/wiki/features/new-in-5.5.x
Upgrade guidelines will be collected at:
* https://www.kamailio.org/wiki/install/upgrade/5.4.x-to-5.5.0
Everyone is more than welcome to contribute to the above wiki pages,
especially to the upgrade guidelines, to help everyone else during the
migration process from v5.4.x to 5.4.x.
Cheers,
Daniel
--
Daniel-Constantin Mierla -- www.asipto.comwww.twitter.com/miconda -- www.linkedin.com/in/miconda
### Description
I am connecting two kamailio (5.4.5) using TLS with client cert verification. When one side is OpenSSL 1.1.1 (CentOS 8) + P-256 cert and the other side is OpenSSL 1.0.2 (CentOS 7) + P-521 cert *and* `cipher_list` is set(e.g., `DEFAULT`), the ClientHello fails because the only supported curve sent is P-256 (server has P-521).
### Troubleshooting
1. If `cipher_list` is *not* set at all in `[client:defaut]`l, then the ClientHello sends 4 supported EC groups and the handshake succeeds.
2 . If `cipher_list` is set in `[client:defaut]` (e.g. `DEFAULT` or `ANY`) then the ClientHello sends only 1 supported EC group P-256 (I guess because the client is using a P-256 cert), then the handshake will fail.
#### Reproduction
1. Connect two kamailos one on CentOS 8 sending SIP to one on CentOS 7, enable TLS, with client cert verification. Put a P-256 cert on CentOS 8 and a P-521 cert on CentOS 7.
#### Debugging Data
1. `cipher_list` is not set: TLS ClientHello Supported Groups is 4 groups: secp256r1 secp521r1 secp384r1 secp256k1
2. `cipher_list` is set to `DEFAULT`: TLS ClientHello Supported Groups is 1 group: secp256r1.
#### Log Messages
```
ERROR: tls [tls_util.h:42]: tls_err_ret(): TLS accept:error:1408A0C1:SSL routines:ssl3_get_client_hello:no shared cipher
```
#### SIP Traffic
NA - couldn't complete TLS handshake so no SIP was transferred.
### Possible Solutions
1. Don't set cipher_list at all; the moment you set cipher_list to anything(`ANY`, `DEFAULT` etc), the ClientHello will narrow the supported groups to P-256 and the server will reject.
I have a thought: does `setup_ecdh()` in `tls_domain.c` behave differently on OpenSSL 1.1.1? Maybe it is forcing the handshake to send only the curve of the client.
### Additional Information
1. kamailio version is 5.4.5
2. Must be OpenSSL 1.1.1 to OpenSSL 1.0.2. BTW in the reverse direction P-521 CentOS 7 to P-256 CentOS 8 doesn't encounter this issue.
--
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/2716
Address GH #2716. Also see https://bugs.python.org/issue29697.
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [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)
- [ ] 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:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [X] PR should be backported to stable branches
- [X ] Tested changes locally
- [ ] Related to issue #2716
#### Description
For OpenSSL 1.1.x initialization of EC SSL contexts has changed — we should be using the < 1.0.2 technique on OpenSSL 1.1+. This addresses a corner case where TLS server with P-256 cert would not handshake with a TLS client presenting a P-521 cert.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/2717
-- Commit Summary --
* [tls] Don't use OpenSSL<1.0.2 fallback on 1.1+
-- File Changes --
M src/modules/tls/tls_domain.c (12)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2717.patchhttps://github.com/kamailio/kamailio/pull/2717.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/2717
**missing braces around initializer**
*mohq_funcs.c:2163*
```
LD (gcc) [M matrix.so] matrix.so
CC (gcc) [M mohqueue.so] mohq_locks.o
CC (gcc) [M mohqueue.so] mohq_funcs.o
mohq_funcs.c: In function 'send_rtp_answer':
mohq_funcs.c:2163:23: warning: missing braces around initializer [-Wmissing-braces]
2163 | fparam_t pzflag [1] = {"", FPARAM_STRING, {pflagbuf}, 0};
| ^
| {
mohq_funcs.c:2163:23: warning: missing braces around initializer [-Wmissing-braces]
2163 | fparam_t pzflag [1] = {"", FPARAM_STRING, {pflagbuf}, 0};
| ^
| { }
CC (gcc) [M mohqueue.so] mohq_db.o
CC (gcc) [M mohqueue.so] mohqueue_mod.o
make[3]: 'libsrdb1.so.1.0' is up to date.
LD (gcc) [M mohqueue.so] mohqueue.so
CC (gcc) [M msilo.so] msfuncs.o
CC (gcc) [M msilo.so] msilo.o
```
**"_POSIX_C_SOURCE" redefined**
```
CC (gcc) [M app_python.so] python_iface.o
CC (gcc) [M app_python.so] apy_kemi_export.o
In file included from /usr/include/python2.7/pyconfig.h:6,
from /usr/include/python2.7/Python.h:8,
from apy_kemi_export.c:30:
/usr/include/python2.7/pyconfig-64.h:1232: warning: "_POSIX_C_SOURCE" redefined
1232 | #define _POSIX_C_SOURCE 200112L
|
In file included from /usr/include/bits/libc-header-start.h:33,
from /usr/include/stdio.h:27,
from apy_kemi_export.c:26:
/usr/include/features.h:265: note: this is the location of the previous definition
265 | # define _POSIX_C_SOURCE 200809L
|
CC (gcc) [M app_python.so] mod_Core.o
CC (gcc) [M app_python.so] apy_kemi.o
In file included from /usr/include/python2.7/pyconfig.h:6,
from /usr/include/python2.7/Python.h:8,
from apy_kemi.c:25:
/usr/include/python2.7/pyconfig-64.h:1232: warning: "_POSIX_C_SOURCE" redefined
1232 | #define _POSIX_C_SOURCE 200112L
|
In file included from /usr/include/bits/libc-header-start.h:33,
from /usr/include/stdio.h:27,
from apy_kemi.c:21:
/usr/include/features.h:265: note: this is the location of the previous definition
265 | # define _POSIX_C_SOURCE 200809L
|
CC (gcc) [M app_python.so] python_msgobj.o
CC (gcc) [M app_python.so] mod_Router.o
CC (gcc) [M app_python.so] app_python_mod.
```
--
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/2710