Re: [sr-dev] [kamailio/kamailio] 200 OK & ACK & topoh issues (#1541)

2018-05-22 Thread Daniel-Constantin Mierla
Probably you can setup quickly a testbed to reproduce. For example, you can make a test unit using docker container and out recent testing framework: * https://github.com/kamailio/kamailio-tests Then you can use sipsak or the internal misc/tools/protoshoot to send the packet. -- You are

[sr-dev] git:5.0:0045f52d: tls: Fixed build tls module on Alpine dist

2018-05-22 Thread Henning Westerholt
Module: kamailio Branch: 5.0 Commit: 0045f52da587a2aa1c050d920bc3900273409a23 URL: https://github.com/kamailio/kamailio/commit/0045f52da587a2aa1c050d920bc3900273409a23 Author: Sergey Safarov Committer: Henning Westerholt Date: 2018-05-22T21:32:14+02:00

[sr-dev] git:5.1:649e3766: tls: Fixed build tls module on Alpine dist

2018-05-22 Thread Henning Westerholt
Module: kamailio Branch: 5.1 Commit: 649e3766f4159a366a1befe49f132637239e7e9a URL: https://github.com/kamailio/kamailio/commit/649e3766f4159a366a1befe49f132637239e7e9a Author: Sergey Safarov Committer: Henning Westerholt Date: 2018-05-22T21:31:18+02:00

Re: [sr-dev] [kamailio/kamailio] Bugfix issue 1168 - Unregistered Terminating Services (PSI) Configured at HSS (#1538)

2018-05-22 Thread Henning Westerholt
As described in the pull template: " Each component has a single commit (if not, squash them into one commit)" - so yes this is generally good. It makes review easier, we also don't want to include your local development "history" in our master branch. -- You are receiving this because you

Re: [sr-dev] [kamailio/kamailio] tls: Fixed build tls module on Alpine dist (#1537)

2018-05-22 Thread Henning Westerholt
Thank you! -- 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/1537#issuecomment-391103409___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] tls: Fixed build tls module on Alpine dist (#1537)

2018-05-22 Thread Henning Westerholt
Merged #1537. -- 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/1537#event-1640113588___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] 200 OK & ACK & topoh issues (#1541)

2018-05-22 Thread Alexandru Pirvulescu
Thank you. This is one of the problems we have. We are running in GKE with a container built using .deb packages, this being the reason we've "tested" with a small C app. Rebuilding the image from sources instead of .deb will take a while and we don't have that provider anymore. Is there a

Re: [sr-dev] [kamailio/kamailio] 200 OK & ACK & topoh issues (#1541)

2018-05-22 Thread Daniel-Constantin Mierla
Can you try with the patch from commit 5a0086fb5d54cf0411180a8426ea49ad440f147a and then grab again the debug messages printed for the bad handling ACK? I want to see if the r-uri is the same as in the ngrep trace, not altered somehow before. -- You are receiving this because you are

[sr-dev] git:master:5a0086fb: topoh: more verbose debug messages when decoding r-uri

2018-05-22 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 5a0086fb5d54cf0411180a8426ea49ad440f147a URL: https://github.com/kamailio/kamailio/commit/5a0086fb5d54cf0411180a8426ea49ad440f147a Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

[sr-dev] [kamailio/kamailio] 200 OK & ACK & topoh issues (#1541)

2018-05-22 Thread Alexandru Pirvulescu
### Description A certain inbound provider does not return ACK R-URI as an exact copy of contact. A parameter is appended to it, like below: 200 OK Contact: `Contact: ` ACK R-URI: `ACK

Re: [sr-dev] [kamailio/kamailio] Bugfix issue 1168 - Unregistered Terminating Services (PSI) Configured at HSS (#1538)

2018-05-22 Thread Christoph VALENTIN
Should I also send this request as one commit, as it was requested for PR 1539 -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] p_usrloc: new parameter "mdb_availability_control" has been created. (#1529)

2018-05-22 Thread hdikme
hdikme commented on this pull request. > @@ -96,7 +96,10 @@ void check_dbs(unsigned int ticks, void *param){ ul_db_handle_list_t * tmp2, * new_element; int found; int i; - + + if(mdb_availability_control) { correct, i m removing the one inside the

Re: [sr-dev] [kamailio/kamailio] Bugfix trash expires (#1539)

2018-05-22 Thread Christoph VALENTIN
Should be reduced to one commit, now. Please try again -- 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/1539#issuecomment-391029667___

Re: [sr-dev] [kamailio/kamailio] Bugfix trash expires (#1539)

2018-05-22 Thread Victor Seva
@christoph-v can you please send this change as one commit, please. ``git rebase -i origin/master`` and ``git push --force`` are your friends -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[sr-dev] [kamailio/kamailio] mod kazoo cannot be loaded on latest alpine:edge (#1540)

2018-05-22 Thread sergey-safarov
When loaded mod kazoo i get error ``` / # kamailio -DD -E 0(178) WARNING: [core/cfg.y:3426]: warn_at(): warning in config file /etc/kamailio/default.cfg, line 67, column 12: raw socket support not compiled in 0(178) ERROR: [core/sr_module.c:582]: load_module(): could not open module : Error

Re: [sr-dev] [kamailio/kamailio] p_usrloc: new parameter "mdb_availability_control" has been created. (#1529)

2018-05-22 Thread Lucian Balaceanu
lbalaceanu commented on this pull request. > @@ -149,6 +152,25 @@ void check_dbs(unsigned int ticks, void *param){ lock_release(list_lock); } +void check_master_db(int dbm_write_default) { + if(mdb_availability_control) { + if(mdb.write.dbh){ Is it necessary to

[sr-dev] [kamailio/kamailio] Bugfix issue 1168 - Unregistered Terminating Services (PSI) Configured at HSS (#1538)

2018-05-22 Thread Christoph VALENTIN
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

[sr-dev] git:master:420fa5e5: core: cfg.lex - removed redundant token defining max_branches

2018-05-22 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 420fa5e581af5668cdecd0c97b555f4d673105f7 URL: https://github.com/kamailio/kamailio/commit/420fa5e581af5668cdecd0c97b555f4d673105f7 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: