Module: kamailio
Branch: master
Commit: e976054f71692b536f59a81347e8254886f08a69
URL: https://github.com/kamailio/kamailio/commit/e976054f71692b536f59a81347e8254…
Author: JailBird <jailbird(a)fdf.net>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2021-12-03T13:02:06+01:00
sqlite: module group creation
- Add sqlite as an option to Makefile.groups
- Add sqlite to INSTALL documentation
---
Modified: INSTALL
Modified: src/Makefile.groups
---
Diff: https://…
[View More]github.com/kamailio/kamailio/commit/e976054f71692b536f59a81347e8254…
Patch: https://github.com/kamailio/kamailio/commit/e976054f71692b536f59a81347e8254…
---
diff --git a/INSTALL b/INSTALL
index 63c03062f4..7982d1d380 100644
--- a/INSTALL
+++ b/INSTALL
@@ -102,6 +102,7 @@ Requirements:
- python and devel headers for the Python module
- jdk and gcj for Java module
- the jansson library for the jansson module used for JSON document handling
+- libsqlite3 if you need SQLite support (the db_sqlite module)
Some modules have other requirements on third-party libraries. This is documented
in the README for the specific module.
@@ -120,6 +121,7 @@ OS Notes:
- expat-1.95.8 (/usr/ports/textproc/expat2) for libexpat
- libxml2-2.6.18 (/usr/ports/textproc/libxml2) for libxml2
- radiusclient-0.4.7 (/usr/ports/net/radiusclient) for libradiusclient-ng
+ - sqlite3-3.35.5 (/usr/ports/databases/sqlite3) for libsqlite3
NOTE: you'll need to add radiusclient_ng=4 to the gmake command line if you
use the 0.4.* version.
@@ -135,6 +137,7 @@ OS Notes:
http://download.berlios.de/radiusclient-ng/radiusclient-ng-0.5.1.tar.gz
(you need to download and install it, since there is no "official"
openbsd port for it) for libradiusclient-ng
+ - sqlite3-3.35.5 (/usr/ports/databases/sqlite3) for libsqlite3
Compile example (all the modules and SIP server core in a tar.gz):
gmake bin include_modules="mysql jabber cpl-c auth_radius group_radius uri_radius pa"
@@ -145,6 +148,7 @@ OS Notes:
- expat-1.95.8nb2 (/usr/pkgsrc/textproc/expat) for libexpat
- libxml2-2.6.19 (/usr/pkgsrc/textproc/libxml2) for libxml2
- radiusclient-ng-0.5.1 (see OpenBSD)
+ - sqlite3-3.36.0 (/usr/pkgsrc/databases/sqlite3) for libsqlite
Compile example (all the modules and SIP server in a tar.gz):
gmake bin include_modules="mysql jabber cpl-c auth_radius group_radius uri_radius pa"
@@ -172,6 +176,7 @@ OS Notes:
- libexpat1-dev for libexpat
- libxml2-dev for libxml2
- libradiusclient-ng-dev for libradiusclient
+ - libsqlite3-dev for libsqlite
- other libraries are needed for some other modules,
see README of the module you want to use
Kamailio have APT deb repositories that allow you to
@@ -258,8 +263,8 @@ Including groups of modules:
- db - Modules in this group use databases and need a database driver to run.
Included are drivers for the text mode db (dbtext) and for dumping
- large amount of data to files (db_flatstore). See also the mysql or
- postgres groups.
+ large amount of data to files (db_flatstore). See also the mysql,
+ postgres, or sqlite groups.
- standard_dep - Modules in this group are considered a standard part of Kamailio
(due to widespread usage) but they have dependencies that must be satisfied
@@ -290,6 +295,7 @@ are grouped based on Debian packaging rules. For example:
mysql - Include all the db modules dependent and the MySQL db driver
postgres - Include all the db modules and the PostgreSQL db driver
+ sqlite - Include all the db modules and the SQLite db driver
radius - Include all modules on radiusclient
presence - Include all the presence modules
diff --git a/src/Makefile.groups b/src/Makefile.groups
index 62a585cafb..c388489a66 100644
--- a/src/Makefile.groups
+++ b/src/Makefile.groups
@@ -319,6 +319,10 @@ module_group_mysql=$(module_group_mysql_driver) $(module_group_db)
module_group_postgres_driver=$(mod_list_postgres)
module_group_postgres=$(module_group_postgres_driver) $(module_group_db)
+# For sqlite
+module_group_sqlite_driver=$(mod_list_sqlite)
+module_group_sqlite=$(module_group_sqlite_driver) $(module_group_db)
+
# For radius
module_group_radius=$(mod_list_radius)
[View Less]
### Description
If two kamailio servers share the same dialog db table, on restart, they both load all dialogues info stored in db into memory and
dlg.stats_alive returns the same amount of alive dialogues on both serves.
### Expected behavior
I would expect that dlg.stats_alive would return just the dialogues info that belongs to the server ( socket_info is not empty )
### Possible Solutions
Introduce an optional parameter to the RPC command to filter just the dialogues that belongs to …
[View More]the server
--
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/2920
[View Less]
We use Kamailio 5.5.0 and topos/topos_redis as well as the uac module to register with sipconnect 1.1 based trunks. When we want to originate a call via such a trunk, we encounter the following problem with ACK routing.
200 OK sent by Trunk
`U 217.10.68.150:5060 -> 172.31.16.103:5060 #7583
SIP/2.0 200 OK.
Via: SIP/2.0/UDP 3.67.244.157:5060;branch=z9hG4bK496f.744a3c48b24be1ba5fdda68cf5fe3878.1.
Record-Route: <sip:172.20.40.8;lr>.
Record-Route: <sip:217.10.68.150;lr>.
Call-ID: …
[View More]7cd83e052202a97f4e28728276e0c758@35.157.121.101:5060.
From: "01705441258" <sip:2908206t0@sipconnect.sipgate.de>;tag=as010d26b4.
To: <sip:+493046996025@voip.sandbox.aaron.ai>;tag=221f5413-4c48-4f53-97e8-d96ea6413f26.
CSeq: 102 INVITE.
Allow: OPTIONS, SUBSCRIBE, NOTIFY, PUBLISH, INVITE, ACK, BYE, CANCEL, UPDATE, MESSAGE, REFER.
Contact: <sip:217.10.77.61:5060>.
Supported: replaces.
Content-Type: application/sdp.
Content-Length: 242.
..
v=0.
o=- 56188352 56188354 IN IP4 212.9.44.180.
s=sGW.
c=IN IP4 212.9.44.180.
t=0 0.
m=audio 27558 RTP/AVP 8 101.
a=maxptime:150.
a=rtpmap:8 PCMA/8000.
a=rtpmap:101 telephone-event/8000.
a=fmtp:101 0-16.
a=sendrecv.
a=rtcp:27559.
a=ptime:20.
`
ACK sent by Kamailio
`ACK sip:217.10.68.150:5060 SIP/2.0.
Via: SIP/2.0/UDP 3.67.244.157:5060;branch=z9hG4bK496f.af6a45e3907d2bea8ea65cc42f223fb2.0.
Max-Forwards: 69.
From: "01705441258" <sip:01705441258@35.157.121.101>;tag=as010d26b4.
To: <sip:+493046996025@voip.sandbox.aaron.ai:5060>;tag=221f5413-4c48-4f53-97e8-d96ea6413f26.
Call-ID: 7cd83e052202a97f4e28728276e0c758@35.157.121.101:5060.
CSeq: 102 ACK.
User-Agent: Asterisk PBX 13.38.1.
Content-Length: 0.
Route: <sip:217.10.68.150;lr>.
Route: <sip:172.20.40.8;lr>.
Contact: <sip:btpsh-61a6122b-a-1@3.67.244.157>.
..
`
The ACK is not accepted by the trunk, because the R-URI does not contain the address fromt the 200 OK/Contact. 200 OK is then retransmitted by the trunk.
[kamailio-uac-topos-bug-report.zip](https://github.com/kamailio/kamailio/fil…
--
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/2961
[View Less]
- sipt_has_isup_body()
- $sipt(redirection_information.reason)
- $sipt(redirection_information.original_reason)
- $sipt(redirecting_number.nai)
- $sipt(redirecting_number)
- $sipt(original_called_number.nai)
- $sipt(original_called_number)
- $sipt(generic_number.nai)
- $sipt(generic_number)
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
-…
[View More] 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 -->
- [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 -->
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/2841
-- Commit Summary --
* sipt: added one function and 8 pseudo-variables
-- File Changes --
M src/modules/sipt/doc/sipt_admin.xml (95)
M src/modules/sipt/sipt.c (358)
M src/modules/sipt/ss7.h (14)
M src/modules/sipt/ss7_parser.c (208)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2841.patchhttps://github.com/kamailio/kamailio/pull/2841.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/2841
[View Less]
- Execute db_redis_consume_replies on failed update same as in db_redis_insert:
if db_redis_perform_update encounters an error it was leaving unconsumed data
thus causing unexpected reply later on when reading from the same connection.
<!-- 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 …
[View More]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
- [ ] 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/2960
-- Commit Summary --
* db_redis: don't leave unconsumed redis replies in case of error on update
-- File Changes --
M src/modules/db_redis/redis_dbase.c (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2960.patchhttps://github.com/kamailio/kamailio/pull/2960.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/2960
[View Less]
As suggested in previous PR (#2936).
This is much more flexible as it can also handle sub XAVPs, and indexes.
This ended up being more complicated than I anticipated, but I wanted to handle all possible cases.
The tests I performed are detailled after, they should cover every possible use case.
#### Type Of Change
- [x] New feature (non-breaking change which adds new functionality)
#### Checklist:
- [x] Tested changes locally
#### Description
Allows counting XAVPs with `$cnt($xavp(...))`
…
[View More]In addition to the `$cnt($avp(...))` - which is not modified.
/* Usage:
*
* $cnt($xavp(key[*])) : number of XAVPs "key".
* $cnt($xavp(key[n]=>sub[*])) : number of children "sub" in XAVP "key[n]".
* $cnt($xavp(key[*]=>sub[*])) : total number of children "sub" in all XAVPs "key".
*
* $cnt($xavp(key[n])) : 1 or 0 (if this index exists or not).
* $cnt($xavp(key[-n])) : same but with reverse indexing (-1 is the last index).
*
* $cnt($xavp(key[*]=>sub[n])) : number of children "sub[n]" that exist in all XAPVs "key".
*
* $cnt($xavp(key)) is the same as $cnt($xavp(key[*])).
* $cnt($xavp(key=>sub)) is the same as $cnt($xavp(key[*]=>sub[*])).
*
* Note: Usually for a XAVP no index means "index 0", not all.
* But this would be less intuitive in our case for counting.
*/
#### Tests
I tried to be exhaustive. All the test cases below do work as expected.
```
$xavp(case1) = "case1.value1";
$xavp(case1) = "case1.value2";
$xavp(case1) = "case1.value3";
$xavp(case2=>sub1) = "case2.value1 sub1.value1";
$xavp(case2[0]=>sub1) = "case2.value1 sub1.value2";
$xavp(case2[0]=>sub1) = "case2.value1 sub1.value3";
$xavp(case2[0]=>sub2) = "case2.value1 sub2.value1";
$xavp(case2[0]=>sub2) = "case2.value1 sub2.value2";
# => will become case2[1] after adding the next entry
$xavp(case2=>sub1) = "case2.value2 sub1.value1";
$xavp(case2[0]=>sub1) = "case2.value2 sub1.value2";
xlog("L_INFO", "----------- TESTS $$cnt($$xavp...) - root XAVPs with no sub XAVPs\n");
xlog("L_INFO", "xavp(case1[0]) : value = $xavp(case1[0]) : cnt (expect: 1) = $cnt($xavp(case1[0]))\n");
xlog("L_INFO", "xavp(case1[1]) : value = $xavp(case1[1]) : cnt (expect: 1) = $cnt($xavp(case1[1]))\n");
xlog("L_INFO", "xavp(case1[-1]) : value = $xavp(case1[-1]) : cnt (expect: 1) = $cnt($xavp(case1[-1]))\n");
xlog("L_INFO", "xavp(case1[-2]) : value = $xavp(case1[-2]) : cnt (expect: 1) = $cnt($xavp(case1[-2]))\n");
xlog("L_INFO", "xavp(case1[-66]) : value = $xavp(case1[-66]) : cnt (expect: 0) = $cnt($xavp(case1[-66]))\n");
xlog("L_INFO", "xavp(case1[66]) : value = $xavp(case1[66]) : cnt (expect: 0) = $cnt($xavp(case1[66]))\n");
xlog("L_INFO", "xavp(case1[*]) : cnt (expect: 3) = $cnt($xavp(case1[*]))\n");
xlog("L_INFO", "xavp(case1) : cnt (expect: 3) = $cnt($xavp(case1))\n");
xlog("L_INFO", "xavp(case1=>sub) : cnt (expect: 0) = $cnt($xavp(case1=>sub))\n");
xlog("L_INFO", "xavp(case1[0]=>sub) : cnt (expect: 0) = $cnt($xavp(case1[0]=>sub))\n");
xlog("L_INFO", "xavp(case1[0]=>sub[0]) : value = $xavp(case1[0]=>sub[0]) : cnt (expect: 0) = $cnt($xavp(case1[0]=>sub[0]))\n");
xlog("L_INFO", "----------- TESTS $$cnt($$xavp...) - root XAVPs using single index\n");
xlog("L_INFO", "xavp(case2[1]=>sub1[0]) : value = $xavp(case2[1]=>sub1[0]) : cnt (expect: 1) = $cnt($xavp(case2[1]=>sub1[0]))\n");
xlog("L_INFO", "xavp(case2[1]=>sub1[-1]) : value = $xavp(case2[1]=>sub1[-1]) : cnt (expect: 1) = $cnt($xavp(case2[1]=>sub1[-1]))\n");
xlog("L_INFO", "xavp(case2[1]=>sub1[3]) : value = $xavp(case2[1]=>sub1[3]) : cnt (expect: 0) = $cnt($xavp(case2[1]=>sub1[3]))\n");
xlog("L_INFO", "xavp(case2[1]=>sub1[*]) : cnt (expect: 3) = $cnt($xavp(case2[1]=>sub1[*]))\n");
xlog("L_INFO", "xavp(case2[1]=>sub1) : cnt (expect: 3) = $cnt($xavp(case2[1]=>sub1))\n");
xlog("L_INFO", "xavp(case2[1]=>sub2[*]) : cnt (expect: 2) = $cnt($xavp(case2[1]=>sub2[*]))\n");
xlog("L_INFO", "xavp(case2[1]=>sub2) : cnt (expect: 2) = $cnt($xavp(case2[1]=>sub2))\n");
xlog("L_INFO", "xavp(case2[66]=>sub1) : cnt (expect: 0) = $cnt($xavp(case2[66]=>sub1))\n");
xlog("L_INFO", "xavp(case2[66]=>sub1[0]) : cnt (expect: 0) = $cnt($xavp(case2[66]=>sub1[0]))\n");
xlog("L_INFO", "----------- TESTS $$cnt($$xavp...) - root XAVPs using all index\n");
xlog("L_INFO", "xavp(case2=>sub1) : cnt (expect: 5) = $cnt($xavp(case2=>sub1))\n");
xlog("L_INFO", "xavp(case2=>sub1[*]) : cnt (expect: 5) = $cnt($xavp(case2=>sub1[*]))\n");
xlog("L_INFO", "xavp(case2[*]=>sub1) : cnt (expect: 5) = $cnt($xavp(case2[*]=>sub1))\n");
xlog("L_INFO", "xavp(case2[*]=>sub1[*]) : cnt (expect: 5) = $cnt($xavp(case2[*]=>sub1[*]))\n");
xlog("L_INFO", "xavp(case2[*]=>sub1) : cnt (expect: 5) = $cnt($xavp(case2[*]=>sub1))\n");
xlog("L_INFO", "xavp(case2[*]=>sub1[0]) : cnt (expect: 2) = $cnt($xavp(case2[*]=>sub1[0]))\n");
xlog("L_INFO", "xavp(case2[*]=>sub1[2]) : cnt (expect: 1) = $cnt($xavp(case2[*]=>sub1[2]))\n");
xlog("L_INFO", "xavp(case2[*]=>sub1[-1]) : cnt (expect: 2) = $cnt($xavp(case2[*]=>sub1[-1]))\n");
xlog("L_INFO", "xavp(case2[*]=>sub2) : cnt (expect: 2) = $cnt($xavp(case2[*]=>sub2))\n");
xlog("L_INFO", "xavp(case2[*]=>sub2[*]) : cnt (expect: 2) = $cnt($xavp(case2[*]=>sub2[*]))\n");
#xlog("L_INFO", "----------- TESTS $$cnt(...) - parsing errors\n");
# parsing error => server fails to start
#xlog("L_INFO", "PARSE ERROR: $cnt($xavu(test))\n");
#xlog("L_INFO", "PARSE ERROR: $cnt($var(test))\n");
#xlog("L_INFO", "PARSE ERROR: $cnt($unknown(test))\n");
```
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/2940
-- Commit Summary --
* pv: extend $cnt to work with $xavp
-- File Changes --
M src/modules/pv/pv_core.c (156)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2940.patchhttps://github.com/kamailio/kamailio/pull/2940.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/2940
[View Less]
Module: kamailio
Branch: master
Commit: 557952f00bff4e9e70a41cad0be22c2f545f3c22
URL: https://github.com/kamailio/kamailio/commit/557952f00bff4e9e70a41cad0be22c2…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2021-11-30T08:46:13+01:00
modules: readme files regenerated - presence ... [skip ci]
---
Modified: src/modules/presence/README
---
Diff: https://github.com/kamailio/kamailio/commit/…
[View More]557952f00bff4e9e70a41cad0be22c2…
Patch: https://github.com/kamailio/kamailio/commit/557952f00bff4e9e70a41cad0be22c2…
---
diff --git a/src/modules/presence/README b/src/modules/presence/README
index eb59dec135..c06ccdd6ec 100644
--- a/src/modules/presence/README
+++ b/src/modules/presence/README
@@ -899,8 +899,9 @@ modparam("presence", "timer_mode", 0)
3.38. subs_respond_200 (integer)
Specify the response code for accepted SUBSCRIBE requests. If set to 0,
- 202 Accepted will be returned. If set to 1, 200 OK will be returned
- instead, in conformance to RFC6665 which prohibits 202 responses.
+ "202 Accepted" will be returned (default behaviour till version 5.5).
+ If set to 1, "200 OK" will be returned instead, in conformance to
+ RFC6665, which prohibits 202 responses.
Default value is 1.
[View Less]
Module: kamailio
Branch: master
Commit: 1ef340457d0bddd8b15be2195c0a9d5d79f88d12
URL: https://github.com/kamailio/kamailio/commit/1ef340457d0bddd8b15be2195c0a9d5…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2021-11-30T08:33:34+01:00
presence: docs - note the version for default behaviour change on 202 reply
---
Modified: src/modules/presence/doc/presence_admin.xml
---
Diff: https://github.com/kamailio/…
[View More]kamailio/commit/1ef340457d0bddd8b15be2195c0a9d5…
Patch: https://github.com/kamailio/kamailio/commit/1ef340457d0bddd8b15be2195c0a9d5…
---
diff --git a/src/modules/presence/doc/presence_admin.xml b/src/modules/presence/doc/presence_admin.xml
index 764aa26a56..25e43ef0b6 100644
--- a/src/modules/presence/doc/presence_admin.xml
+++ b/src/modules/presence/doc/presence_admin.xml
@@ -983,10 +983,10 @@ modparam("presence", "timer_mode", 0)
<section id="presence.p.subs_respond_200">
<title><varname>subs_respond_200</varname> (integer)</title>
<para>
- Specify the response code for accepted SUBSCRIBE requests.
- If set to 0, 202 Accepted will be returned.
- If set to 1, 200 OK will be returned instead, in conformance to RFC6665
- which prohibits 202 responses.
+ Specify the response code for accepted SUBSCRIBE requests. If set to 0,
+ "202 Accepted" will be returned (default behaviour till version 5.5). If set
+ to 1, "200 OK" will be returned instead, in conformance to RFC6665, which
+ prohibits 202 responses.
</para>
<para>
<emphasis>
[View Less]
<!-- 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, …
[View More]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
- [x] Related to issue #2930
#### Description
See https://github.com/kamailio/kamailio/pull/2930#issuecomment-971748854.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/2949
-- Commit Summary --
* presence: return 200 instead of 202 by default
-- File Changes --
M src/modules/presence/doc/presence_admin.xml (4)
M src/modules/presence/presence.c (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2949.patchhttps://github.com/kamailio/kamailio/pull/2949.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/2949
[View Less]
If ds_probing_mode is set to 3, flag in the dispatcher entry needs to include 8
for probing to happen.
Changing a destination to not include this anymore while the destination do
not respond to OPTIONS will cause the probing flag to be re-added because of the
delayed response. A restart will fix it.
We need to check against the probing flag in the CB function to determine if
we should continue probing this destination. If probing is deativated while
waiting for a reply, we should abort.
You …
[View More]can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/2959
-- Commit Summary --
* dispatcher: Not possible to deactivate probing of destination by reload
-- File Changes --
M src/modules/dispatcher/dispatch.c (5)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2959.patchhttps://github.com/kamailio/kamailio/pull/2959.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/2959
[View Less]
Module: kamailio
Branch: master
Commit: 429fea8069f0753e6b8dbfbfaf36701e30af6dda
URL: https://github.com/kamailio/kamailio/commit/429fea8069f0753e6b8dbfbfaf36701…
Author: Morten Tryfoss <morten(a)tryfoss.no>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2021-11-30T08:24:03+01:00
dispatcher: Not possible to deactivate probing of destination by reload
If ds_probing_mode is set to 3, flag in the dispatcher entry needs to include 8
for probing to happen.
Changing a …
[View More]destination to not include this anymore while the destination do
not respond to OPTIONS will cause the probing flag to be re-added because of the
delayed response. A restart will fix it.
We need to check against the probing flag in the CB function to determine if
we should continue probing this destination. If probing is deativated while
waiting for a reply, we should abort.
---
Modified: src/modules/dispatcher/dispatch.c
---
Diff: https://github.com/kamailio/kamailio/commit/429fea8069f0753e6b8dbfbfaf36701…
Patch: https://github.com/kamailio/kamailio/commit/429fea8069f0753e6b8dbfbfaf36701…
---
diff --git a/src/modules/dispatcher/dispatch.c b/src/modules/dispatcher/dispatch.c
index dd3c93f9a8..661b4a9533 100644
--- a/src/modules/dispatcher/dispatch.c
+++ b/src/modules/dispatcher/dispatch.c
@@ -3529,6 +3529,11 @@ static void ds_options_callback(
}
}
+ /* Check if in the meantime someone disabled probing of the target through RPC, MI or reload */
+ if(ds_probing_mode == DS_PROBE_ONLYFLAGGED && !(ds_get_state(group, &uri) & DS_PROBING_DST)) {
+ return;
+ }
+
/* ps->code contains the result-code of the request.
*
* We accept both a "200 OK" or the configured reply as a valid response */
[View Less]
### Description
Kamailio compiled with ASAN.
During Kamailio start I see messages
```
core/flags.c:46:18: runtime error: shift exponent 4294967295 is too large for 32-bit type 'int'
```
and
```
nat_traversal.c:1293:27: runtime error: left shift of 1 by 31 places cannot be represented in type 'int'
```
tested on current master
--
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/2954