[sr-dev] git:master:1ca77084: kazoo: call cfg_update on child init

2017-04-17 Thread lazedo
Module: kamailio Branch: master Commit: 1ca77084e075bd78bc0603e838a268e470480a17 URL: https://github.com/kamailio/kamailio/commit/1ca77084e075bd78bc0603e838a268e470480a17 Author: lazedo <luis.az...@factorlusitano.com> Committer: lazedo <luis.az...@factorlusitano.com> Date: 2017-04-17

[sr-dev] git:master:349def6b: kazoo: add pua transaction lock type

2017-04-13 Thread lazedo
Module: kamailio Branch: master Commit: 349def6b129e63cdc46b88c502562c9e2543d421 URL: https://github.com/kamailio/kamailio/commit/349def6b129e63cdc46b88c502562c9e2543d421 Author: lazedo <luis.az...@factorlusitano.com> Committer: lazedo <luis.az...@factorlusitano.com> Date: 2017-04-14

[sr-dev] git:master:29aec680: kazoo: add consistent worker key

2017-04-13 Thread lazedo
Module: kamailio Branch: master Commit: 29aec6800099a3b1d7a2b2c7e6d4b3865f0bc6e6 URL: https://github.com/kamailio/kamailio/commit/29aec6800099a3b1d7a2b2c7e6d4b3865f0bc6e6 Author: lazedo <luis.az...@factorlusitano.com> Committer: lazedo <luis.az...@factorlusitano.com> Date: 2017-04-14

[sr-dev] git:master:cc7adf79: kazoo: add time msg was received

2017-04-14 Thread lazedo
Module: kamailio Branch: master Commit: cc7adf798c1f533e77dfab94563d7995094aa4c5 URL: https://github.com/kamailio/kamailio/commit/cc7adf798c1f533e77dfab94563d7995094aa4c5 Author: lazedo <luis.az...@factorlusitano.com> Committer: lazedo <luis.az...@factorlusitano.com> Date: 2017-04-14

[sr-dev] git:master:7e693ac3: kazoo: add alternative name for sender

2017-04-17 Thread lazedo
Module: kamailio Branch: master Commit: 7e693ac32ceae7bcc34d6359043d6d9963933d4f URL: https://github.com/kamailio/kamailio/commit/7e693ac32ceae7bcc34d6359043d6d9963933d4f Author: lazedo <luis.az...@factorlusitano.com> Committer: lazedo <luis.az...@factorlusitano.com> Date: 2017-04-17

Re: [sr-dev] [kamailio/kamailio] kazoo: Have been declared rabbitmq-c dependency (#1084)

2017-04-22 Thread lazedo
you can build with 0.5.2 too and code is handling this -- 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/1084#issuecomment-296392029___

[sr-dev] git:master:74563696: pv: add string corehash transform

2017-08-17 Thread lazedo
Module: kamailio Branch: master Commit: 74563696fa064b7cbe169e9af1020ba840c8b8ab URL: https://github.com/kamailio/kamailio/commit/74563696fa064b7cbe169e9af1020ba840c8b8ab Author: lazedo <luis.az...@factorlusitano.com> Committer: lazedo <luis.az...@factorlusitano.com> Date: 2017-08-17

[sr-dev] [kamailio/kamailio] pv: add string corehash transform (#1218)

2017-08-16 Thread lazedo
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: - [X] PR should be backported to stable

Re: [sr-dev] [kamailio/kamailio] dispatcher: runtime status access from script (#1166)

2017-06-27 Thread lazedo
@miconda it was just to get the reflection of what's in the database with runtime information. i check jsonrpcs and it will have to work for now. thanks -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[sr-dev] git:master:551413f0: kazoo: add json.count transformation

2017-06-27 Thread lazedo
Module: kamailio Branch: master Commit: 551413f0952429c3a360b5ccdeaeac48ea15ff8a URL: https://github.com/kamailio/kamailio/commit/551413f0952429c3a360b5ccdeaeac48ea15ff8a Author: lazedo <luis.az...@factorlusitano.com> Committer: lazedo <luis.az...@factorlusitano.com> Date: 2017-06-27

Re: [sr-dev] [kamailio/kamailio] pv: add string corehash transform (#1218)

2017-08-17 Thread lazedo
@miconda thanks. the reason was for large numbers not handled properly in script (uint) afaik. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[sr-dev] git:master:4751b777: kazoo : fix json keys

2017-05-02 Thread lazedo
Module: kamailio Branch: master Commit: 4751b77727730ad67ea003afd07ec91b0f7cd45b URL: https://github.com/kamailio/kamailio/commit/4751b77727730ad67ea003afd07ec91b0f7cd45b Author: lazedo <luis.az...@factorlusitano.com> Committer: lazedo <luis.az...@factorlusitano.com> Date: 2017-05-03

Re: [sr-dev] [kamailio/kamailio] registrar: fix xavp_rcd memory consumption (#1111)

2017-05-15 Thread lazedo
@miconda Hi Daniel, kazoo module does the suspend/continue, but this is only for the initial registration. the 2nd registration doesn't go thru suspend/resume but we're also getting memory leaks here. -- You are receiving this because you are subscribed to this thread. Reply to this email

Re: [sr-dev] [kamailio/kamailio] registrar: fix xavp_rcd memory consumption (#1111)

2017-05-09 Thread lazedo
Hi Daniel, i isolated all the requests and made sure only registration was being processed. turns out that the problem must be elsewhere as the memory is now stable. however, the first registration uses a suspend/continue and in this case the xavp memory is not released.

Re: [sr-dev] [kamailio/kamailio] registrar: fix xavp_rcd memory consumption (#1111)

2017-05-09 Thread lazedo
@miconda sorry, wrong report. the patch was applied, that's why memory is stable. initial registration issues still occurs (xavp not released after t_continue() ) -- 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] registrar: fix xavp_rcd memory consumption (#1111)

2017-05-09 Thread lazedo
@miconda this is the log without the patch applied. [kam.log.gz](https://github.com/kamailio/kamailio/files/986380/kam.log.gz) -- 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] registrar: fix xavp_rcd memory consumption (#1111)

2017-05-16 Thread lazedo
@miconda Hi, yes, all ok with https://github.com/kamailio/kamailio/commit/3fddccd0d7a3f9662699f88d1a9a708e1c1b24bc . the problem was how we obtained the existing `xavp's` and adding new values. this part `xavp = list ? >val.v.xavp : _xavp;` did the trick, we were adding values to xavp list and

Re: [sr-dev] [kamailio/kamailio] registrar: fix xavp_rcd memory consumption (#1111)

2017-05-16 Thread lazedo
@miconda Thanks Daniel. i've asked for more people to confirm the fix and i'm also deploying this to our sandbox environment. will cherry-pick the 1st to 4.4/5.0 and the 2nd to 5.0 if all goes well. so far my environment is stable. -- You are receiving this because you are subscribed to this

Re: [sr-dev] [kamailio/kamailio] registrar: fix xavp_rcd memory consumption (#1111)

2017-05-16 Thread lazedo
@miconda so 1st registrations are ok. i was testing with db_mode = `Write-Through` (sorry about that). with for db_mode = `db_only`, all xavps are cleared after route (with async resume). -- You are receiving this because you are subscribed to this thread. Reply to this email directly or

Re: [sr-dev] [kamailio/kamailio] registrar: fix xavp_rcd memory consumption (#1111)

2017-05-15 Thread lazedo
@miconda thanks for the info. after digging more , i think the issue is not related to `kazoo`. can you try the below script (note that kazoo is not used) ? also maybe fake a transaction to a external auth and set the `$xavp(ulattrs=>extra_info)` on the `continue`block ? with this

Re: [sr-dev] [kamailio/kamailio] registrar: fix xavp_rcd memory consumption (#1111)

2017-05-15 Thread lazedo
@lazedo pushed 1 commit. 649d968 registrar: remove 1st try -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/kamailio/kamailio/pull//files/2e6ea50a6ebd8f8797ed6c20c13b5bbb3b3494b7..649d9689b62b91d584b9a8e2c8b7c56217648058

Re: [sr-dev] [kamailio/kamailio] registrar: fix xavp_rcd memory consumption (#1111)

2017-05-15 Thread lazedo
@miconda i reverted every change and added `xavp_reset_list()` after `t_continue()` in kazoo andalso after `kz_amqp_consumer_fire_event()` and leak continues. is there a way to lookup the memory allocated values to make sure it is coming from where we think it is ? -- You are receiving this

Re: [sr-dev] [kamailio/kamailio] htable race condition on expired keys (#1152)

2017-06-19 Thread lazedo
yes, https://github.com/2600hz/kazoo-configs-kamailio/blob/master/kamailio/nodes-role.cfg -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[sr-dev] git:master:f18da4c9: kazoo: use hardcoded max size of routing key

2017-06-23 Thread lazedo
Module: kamailio Branch: master Commit: f18da4c9d6cc8d1ab061d68cbd8e9d988ebf2be6 URL: https://github.com/kamailio/kamailio/commit/f18da4c9d6cc8d1ab061d68cbd8e9d988ebf2be6 Author: lazedo <luis.az...@factorlusitano.com> Committer: lazedo <luis.az...@factorlusitano.com> Date: 2017-06-23

[sr-dev] [kamailio/kamailio] dispatcher: runtime status access from script (#1166)

2017-06-25 Thread lazedo
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1166 -- Commit Summary -- * dispatcher: runtime status access from script -- File Changes -- M src/modules/dispatcher/dispatch.c (94) M src/modules/dispatcher/dispatch.h (4)

Re: [sr-dev] [kamailio/kamailio] registrar: fix xavp_rcd memory consumption (#1111)

2017-05-19 Thread lazedo
Closed #. -- 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/#event-1089395412___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] registrar: fix xavp_rcd memory consumption (#1111)

2017-05-16 Thread lazedo
i didn't used kazoo for testing, but i will be adding a xavp_reset_list() to event processing (not critical). test with this script with/without the patch. have sip client re-register every 30 and you'll see `shm` memory consumption growing (`xavp_new_value`) ``` ... modparam("usrloc",

[sr-dev] [kamailio/kamailio] htable: fix crash if deleted entry is reused (#1142)

2017-05-29 Thread lazedo
resets 'first' when esize == 0 You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1142 -- Commit Summary -- * htable: fix crash if deleted entry is reused -- File Changes -- M src/modules/htable/ht_api.c (14) -- Patch Links --

[sr-dev] git:master:2d771f56: htable: release pkg memory before returning

2017-06-02 Thread lazedo
Module: kamailio Branch: master Commit: 2d771f563d19fccad0a28e56a5d99de571ef5da6 URL: https://github.com/kamailio/kamailio/commit/2d771f563d19fccad0a28e56a5d99de571ef5da6 Author: lazedo <luis.az...@factorlusitano.com> Committer: lazedo <luis.az...@factorlusitano.com> Date: 2017-06-02

Re: [sr-dev] [kamailio/kamailio] htable: fix crash if deleted entry is reused (#1142)

2017-06-02 Thread lazedo
@miconda thanks for the comment. i think you're right. i re-tested with only the second commit and it seems to be working. -- 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] htable: fix crash if deleted entry is reused (be46742)

2017-06-13 Thread lazedo
@miconda Hi Daniel. not sure why but we're getting some crashes under load without this commit. -- 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] htable race condition on expired keys (#1152)

2017-06-19 Thread lazedo
sorry about that, patch updated ``` diff --git a/src/modules/htable/ht_api.c b/src/modules/htable/ht_api.c index 1bbd079..b5872d0 100644 --- a/src/modules/htable/ht_api.c +++ b/src/modules/htable/ht_api.c @@ -674,6 +674,7 @@ && strncmp(name->s, it->name.s,

[sr-dev] [kamailio/kamailio] registrar: fix xavp_rcd memory consumption (#1111)

2017-05-02 Thread lazedo
xavp_rcd keeps accumulating values leading to shm exaustion You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/ -- Commit Summary -- * registrar: fix xavp_rcd memory consumption -- File Changes -- M

Re: [sr-dev] [kamailio/kamailio] pkg/kamailio/centos/7: mod kazoo enabled default build (#1105)

2017-05-02 Thread lazedo
@miconda i will -- 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/1105#issuecomment-298609765___ Kamailio (SER) - Development Mailing List

[sr-dev] git:master:65254288: kazoo: missing sender alternative for presence

2017-05-02 Thread lazedo
Module: kamailio Branch: master Commit: 6525428887b20bac61825267a30016ebdf2f29a5 URL: https://github.com/kamailio/kamailio/commit/6525428887b20bac61825267a30016ebdf2f29a5 Author: lazedo <luis.az...@factorlusitano.com> Committer: lazedo <luis.az...@factorlusitano.com> Date: 2017-05-02

Re: [sr-dev] [kamailio/kamailio] registrar: fix xavp_rcd memory consumption (#1111)

2017-05-03 Thread lazedo
[kamailio.log.gz](https://github.com/kamailio/kamailio/files/973067/kamailio.log.gz) used gzip -- 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] kazoo: backports & fixes (#1119)

2017-05-04 Thread lazedo
@miconda Daniel, this should be the last set of changes to 5.0 branch (unless bug related issues). kemi event binding will go only into master -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[sr-dev] git: new commits in branch 5.0

2017-05-04 Thread lazedo
5-04T16:55:32+01:00 kazoo: add zone to connection event - URL: https://github.com/kamailio/kamailio/commit/5456da50ac59f8cfa729669048214782db8ae752 Author: lazedo <luis.az...@factorlusitano.com> Date: 2017-05-04T16:55:32+01:00 kazoo: add exchange bindings - URL: h

Re: [sr-dev] [kamailio/kamailio] kazoo: backports & fixes (#1119)

2017-05-04 Thread lazedo
Merged #1119. -- 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/1119#event-1069199830___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] registrar: fix xavp_rcd memory consumption (#1111)

2017-05-02 Thread lazedo
@miconda it seems so, maybe the problem lies in the delete ? i'm using in script to create a json with the `ruid` and memory is always growing. removing just these parts, memory is back to normal. ``` modparam("registrar", "xavp_rcd", "ulrcd") ... after successful save

Re: [sr-dev] [kamailio/kamailio] registrar: fix xavp_rcd memory consumption (#1111)

2017-05-02 Thread lazedo
shared memory tracked by [root@dev-01 kamailio]# kamcmd mod.stats core shm Module: core { *xavp_new_value(94): 4808* } only one client registering every 60 sec On Tue, May 2, 2017 at 3:18 PM, Daniel-Constantin Mierla < notificati...@github.com> wrote: > Which memory is growing? SHM, PKG or

[sr-dev] [kamailio/kamailio] timer: reset xavp using common function (#1246)

2017-09-23 Thread lazedo
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1246 -- Commit Summary -- * timer: reset xavp using common function -- File Changes -- M src/modules/timer/timer.c (2) -- Patch Links --

Re: [sr-dev] [kamailio/kamailio] timer: reset xavp using common function (#1246)

2017-09-24 Thread lazedo
thanks @miconda i cherry-pick to 5.0 branch -- 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/1246#issuecomment-331707634___ Kamailio (SER)

[sr-dev] git:master:b87ac344: kazoo : add consumer key if appropriate

2017-10-10 Thread lazedo
Module: kamailio Branch: master Commit: b87ac3440ae38526cca15dd26625e481e331d51e URL: https://github.com/kamailio/kamailio/commit/b87ac3440ae38526cca15dd26625e481e331d51e Author: lazedo <luis.az...@factorlusitano.com> Committer: lazedo <luis.az...@factorlusitano.com> Date: 2017-10-11

[sr-dev] git:master:33ecbbba: kazoo : increase max url size to 512

2017-10-10 Thread lazedo
Module: kamailio Branch: master Commit: 33ecbbbaf8054220372637ac89ac4e4a7c7748d0 URL: https://github.com/kamailio/kamailio/commit/33ecbbbaf8054220372637ac89ac4e4a7c7748d0 Author: lazedo <luis.az...@factorlusitano.com> Committer: lazedo <luis.az...@factorlusitano.com> Date: 2017-10-11

Re: [sr-dev] [kamailio/kamailio] kazoo: amqp_connection parameter maximum length is too short (#1269)

2017-10-10 Thread lazedo
fixed by https://github.com/kamailio/kamailio/commit/33ecbbbaf8054220372637ac89ac4e4a7c7748d0 -- 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] nat_traversal: handle local generated transactions (#1344)

2017-11-29 Thread lazedo
@miconda thanks for the comments. i think `tmp` will be freed with `free_sip_msg`. will update the `FAKED_REPLY` -- 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] schema: allow null in active_watchers reason (#1345)

2017-11-29 Thread lazedo
@miconda on new dialog subscribes the value is null. we can either add a `default` or allow null -- 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] presence: update contact in update_subs_db (#1341)

2017-11-29 Thread lazedo
Merged #1341. -- 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/1341#event-1363152439___ Kamailio (SER) - Development Mailing List

[sr-dev] git:master:49272f23: presence: update contact in update_subs_db

2017-11-29 Thread lazedo
Module: kamailio Branch: master Commit: 49272f23b22445415721766709bf5841865e8609 URL: https://github.com/kamailio/kamailio/commit/49272f23b22445415721766709bf5841865e8609 Author: lazedo <luis.az...@factorlusitano.com> Committer: lazedo <luis.az...@factorlusitano.com> Date: 2017-11-

[sr-dev] [kamailio/kamailio] nat_traversal: handle local generated transactions (#1344)

2017-11-29 Thread lazedo
when using the below script blocks for handling SUBSCRIBE messages, the nat_keepalive does not work as expected. ``` #!ifdef NAT_TRAVERSAL_ROLE route[PRESENCE_NAT] { force_rport(); if (client_nat_test("3")) { if(is_first_hop()) set_contact_alias();

Re: [sr-dev] [kamailio/kamailio] nat_traversal: handle local generated transactions (#1344)

2017-11-29 Thread lazedo
@miconda ah, didn't know that, fixing -- 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/1344#issuecomment-347817730___ Kamailio (SER) -

Re: [sr-dev] [kamailio/kamailio] nat_traversal: handle local generated transactions (#1344)

2017-11-29 Thread lazedo
@lazedo pushed 1 commit. d4bb995 nat_traversal: handle TMCB_RESPONSE_SENT only for FAKED_REPLY -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/kamailio/kamailio/pull/1344/files/d9d55d2a21cba46228a1a2213884779507ffc9dc

Re: [sr-dev] [kamailio/kamailio] schema: allow null in active_watchers reason (#1345)

2017-11-29 Thread lazedo
Hi Daniel, only with sqlite which uses kamailio db routines to generate the actual sql statement. i can try others or, maybe i'm missing something basic in sqlite. On Wed, Nov 29, 2017 at 2:57 PM, Daniel-Constantin Mierla < notificati...@github.com> wrote: > Have you tested with the updated

[sr-dev] [kamailio/kamailio] core : honor parse_param separator (#1303)

2017-11-07 Thread lazedo
fixes the use case when spaces should not be excluded 'BS ParkingTestCase 1' ``` xlog("L_INFO", "$ci|log exploding test\n"); $var(test) = "Notify=false;Unregister=false;Presence-ID=1...@kamailio.io;Account-Name=BS ParkingTestCase 1"; xavp_params_explode($var(test),

Re: [sr-dev] [kamailio/kamailio] core : honor parse_param separator (#1303)

2017-11-07 Thread lazedo
@miconda thanks for pointing that out. checking again, maybe jumped in here too fast. -- 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] core : honor parse_param separator (#1303)

2017-11-07 Thread lazedo
@miconda enclosing the params with quotes fixes it. sorry for jumping too fast, but when i saw the `separator` and then the exceptions..., didn't think it could break sip grammar. thanks for the explanation. -- You are receiving this because you are subscribed to this thread. Reply to this

Re: [sr-dev] [kamailio/kamailio] core : honor parse_param separator (#1303)

2017-11-07 Thread lazedo
Closed #1303. -- 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/1303#event-1329475099___ Kamailio (SER) - Development Mailing List

[sr-dev] git:master:c0bc3861: presence: extra columns updated for dbmode 2

2017-12-08 Thread lazedo
Module: kamailio Branch: master Commit: c0bc38616cb9a2923e35545fa86a663686bc6746 URL: https://github.com/kamailio/kamailio/commit/c0bc38616cb9a2923e35545fa86a663686bc6746 Author: lazedo <luis.az...@factorlusitano.com> Committer: lazedo <luis.az...@factorlusitano.com> Date: 2017-12-

Re: [sr-dev] [kamailio/kamailio] presence: extra columns updated for dbmode 2 (#1354)

2017-12-08 Thread lazedo
Merged #1354. -- 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/1354#event-1378435613___ Kamailio (SER) - Development Mailing List

[sr-dev] [kamailio/kamailio] presence: update record_route in database (#1357)

2017-12-08 Thread lazedo
missed update from 49272f23b22445415721766709bf5841865e8609 should be backported with 816d2cd44d96245b5a5b53505d6fbc350bda378c You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1357 -- Commit Summary -- * presence: update record_route

Re: [sr-dev] [kamailio/kamailio] schema: allow null in active_watchers reason (#1345)

2017-12-06 Thread lazedo
@miconda hi Daniel, follow the code path of handle_subscribe in presence/subscriber.c that leads to `update_subscription` - new subscription - pres_notifier_processes = 0 - dialog subscription => event->req_auth == 0 this code doesn't change the value its null by default, the code path with

[sr-dev] [kamailio/kamailio] presence: extra columns updated for dbmode 2 (#1354)

2017-12-06 Thread lazedo
we noticed some phones and upstream clusters sending diferent route / contact after initial subscription. this commit adds contact and record_route to updated columns for dbmode = 2 also adds user_agent to initial save and a simple fix for getting the correct message in debug should be

[sr-dev] git:master:e5254795: presence: update record_route in database

2017-12-09 Thread lazedo
Module: kamailio Branch: master Commit: e52547956462ad53032be95ff5f2701785dfdd2a URL: https://github.com/kamailio/kamailio/commit/e52547956462ad53032be95ff5f2701785dfdd2a Author: lazedo <luis.az...@factorlusitano.com> Committer: lazedo <luis.az...@factorlusitano.com> Date: 2017-12-

Re: [sr-dev] [kamailio/kamailio] presence: update record_route in database (#1357)

2017-12-09 Thread lazedo
Merged #1357. -- 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/1357#event-1379986836___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] kazoo: wrong BLF when used application/dialog-info+xml (#1489)

2018-05-10 Thread lazedo
Closed #1489. -- 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/1489#event-1619808776___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] kazoo: wrong BLF when used application/dialog-info+xml (#1489)

2018-05-10 Thread lazedo
@sergey-safarov hi, what you're reporting has nothing to do with kamailio & kazoo module. i'm closing this here. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[sr-dev] git:master:3018595d: kazoo: change string parameters to PARAM_STR

2018-01-23 Thread lazedo
Module: kamailio Branch: master Commit: 3018595d13dde08d15e8063cecb42a01ea271b66 URL: https://github.com/kamailio/kamailio/commit/3018595d13dde08d15e8063cecb42a01ea271b66 Author: lazedo <luis.az...@factorlusitano.com> Committer: lazedo <luis.az...@factorlusitano.com> Date: 2018-01-

Re: [sr-dev] [kamailio/kamailio] pua_json: new module to update presence using JSON data objects (#1424)

2018-02-01 Thread lazedo
lazedo commented on this pull request. > + * Kamailio is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or + * (at your option) any later vers

Re: [sr-dev] [kamailio/kamailio] schema: allow null value for active_watchers user-agent column (#1431)

2018-02-08 Thread lazedo
@miconda here https://github.com/kamailio/kamailio/blob/master/src/modules/presence/subscribe.c#L183 UA is not sending user-agent header in subscribe and its not mandatory in [rfc3265 ](https://www.ietf.org/rfc/rfc3265.txt), the schema should follow the standard -- You are receiving this

Re: [sr-dev] [kamailio/kamailio] schema: allow null value for active_watchers user-agent column (#1431)

2018-02-11 Thread lazedo
@miconda great. should we backport that commit, this PR or that commit & this PR ? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[sr-dev] git:master:7653124c: schema: allow null value for active_watchers user-agent column

2018-02-11 Thread lazedo
Module: kamailio Branch: master Commit: 7653124c2d3615e4f6229f12b699063f65b7afea URL: https://github.com/kamailio/kamailio/commit/7653124c2d3615e4f6229f12b699063f65b7afea Author: lazedo <luis.az...@factorlusitano.com> Committer: lazedo <luis.az...@factorlusitano.com> Date: 2018-02-

Re: [sr-dev] [kamailio/kamailio] schema: allow null value for active_watchers user-agent column (#1431)

2018-02-11 Thread lazedo
@miconda just noticed that you already backported https://github.com/kamailio/kamailio/commit/4654f8b4f341a6c9f8cf4daafc7d618871c95e1b . i would like to backport this PR if that's ok -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it

[sr-dev] [kamailio/kamailio] schema: allow null value for active_watchers user-agent column (#1431)

2018-02-07 Thread lazedo
some UAs not sending User-Agent header on SUBSCRIBE causes insert error needs backport to 5.0/5.1 You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1431 -- Commit Summary -- * schema: allow null value for active_watchers user-agent

Re: [sr-dev] [kamailio/kamailio] acc_json: adding module (#1440)

2018-02-15 Thread lazedo
this is a great addition but i think its restricted in the output. `mqueue` and `syslog` shouldn't be here. the module should emit an event with the json payload as a new pseudo var, and in the script we could the use mqueue, syslog or whatever the integrator/developer wishes to do with json

[sr-dev] [kamailio/kamailio] dispatch: change log level to debug for rpc list empty list (#1448)

2018-02-17 Thread lazedo
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

Re: [sr-dev] [kamailio/kamailio] dispatch: change log level to debug for rpc list empty list (#1448)

2018-02-19 Thread lazedo
@miconda hey, Daniel, can this be backported ? -- 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/1448#issuecomment-366680294___ Kamailio

[sr-dev] [kamailio/kamailio] presence: limit the number of subscriptions handled in timer_dbonly (#1609)

2018-08-01 Thread lazedo
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) - [ ] No commits to README files

Re: [sr-dev] [kamailio/kamailio] presence: limit the number of subscriptions handled in timer_dbonly (#1609)

2018-08-01 Thread lazedo
@miconda yes it was, the callback of the notify deletes the subscription. it actually is required because in the case where we have `fetch_rows=500` and the entire result set is `5000` rows then we would loose `4500` rows because they would be deleted. -- You are receiving this because you

Re: [sr-dev] [kamailio/kamailio] presence: limit the number of subscriptions handled in timer_dbonly (#1609)

2018-08-01 Thread lazedo
this is causing problems in some `production` servers and i would like to backport to stable branches if that's ok -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[sr-dev] git:master:e6a7a3f3: presence: limit the number of subscriptions handled in timer_dbonly

2018-08-01 Thread lazedo
Module: kamailio Branch: master Commit: e6a7a3f37e80683d941321a6e10b636f0bfb4478 URL: https://github.com/kamailio/kamailio/commit/e6a7a3f37e80683d941321a6e10b636f0bfb4478 Author: lazedo Committer: lazedo Date: 2018-08-01T23:36:22+01:00 presence: limit the number of subscriptions handled

Re: [sr-dev] [kamailio/kamailio] presence: limit the number of subscriptions handled in timer_dbonly (#1609)

2018-08-01 Thread lazedo
Merged #1609. -- 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/1609#event-1766405730___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] DB Postgres BLOB conversion for presence (#1255)

2018-04-03 Thread lazedo
@sergey-safarov hot fix for what exactly ? this seems `PostgreSQL` driver specific -- 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] kazoo: wrong BLF when used application/dialog-info+xml (#1489)

2018-04-03 Thread lazedo
@sergey-safarov this doesn't look like a kamailio issue. you may need to capture amqp packets from/to kamailio andalso look into kamailio logs. the items you're reporting as `unexpected` seem like an `expectation from user` rather than `unexpected because current state is ...`. what i mean is,

[sr-dev] git:master:6c40983b: kazoo: free envelope only on normal response

2018-02-28 Thread lazedo
Module: kamailio Branch: master Commit: 6c40983bf04a23bd7394c247980f21c5f2d6d817 URL: https://github.com/kamailio/kamailio/commit/6c40983bf04a23bd7394c247980f21c5f2d6d817 Author: lazedo <luis.az...@factorlusitano.com> Committer: lazedo <luis.az...@factorlusitano.com> Date: 2018-02-

[sr-dev] [kamailio/kamailio] nathelper: fix call to alter_mediaip (#1655)

2018-09-27 Thread lazedo
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:45479865: siputils: add is_uri function

2018-10-04 Thread lazedo
Module: kamailio Branch: master Commit: 454798654c2b2acd252df411d64e55bee0a70f52 URL: https://github.com/kamailio/kamailio/commit/454798654c2b2acd252df411d64e55bee0a70f52 Author: lazedo Committer: lazedo Date: 2018-10-04T12:40:11Z siputils: add is_uri function --- Modified: src/modules

Re: [sr-dev] [kamailio/kamailio] nathelper: initialize local variable (#1661)

2018-10-01 Thread lazedo
Merged #1661 into 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/pull/1661#event-1876997669___ Kamailio (SER) - Development Mailing

Re: [sr-dev] [kamailio/kamailio] nathelper: initialize local variable (#1661)

2018-10-01 Thread lazedo
@miconda thanks. is commit [0d14ce8](https://github.com/kamailio/kamailio/commit/0d14ce8bf45ada579f7ec85793d358a5fa32e4b7#diff-6b4f66e9085d3547acd73e9e61be492f) being backported ? or should i resolve the conflicts for 5.1 ? i can backport it as part of this if you want -- You are receiving

[sr-dev] git:master:3892c78a: Revert "nathelper: fix call to alter_mediaip"

2018-10-01 Thread lazedo
Module: kamailio Branch: master Commit: 3892c78ac59a7d894a5dc8804c20df07b52cc795 URL: https://github.com/kamailio/kamailio/commit/3892c78ac59a7d894a5dc8804c20df07b52cc795 Author: Luis Azedo Committer: lazedo Date: 2018-10-01T17:13:05+02:00 Revert "nathelper: fix call to alter_me

[sr-dev] git:master:c814f9fb: nathelper: initialize local variable

2018-10-01 Thread lazedo
Module: kamailio Branch: master Commit: c814f9fb6aa20ae18253f0d165795516a8869085 URL: https://github.com/kamailio/kamailio/commit/c814f9fb6aa20ae18253f0d165795516a8869085 Author: Luis Azedo Committer: lazedo Date: 2018-10-01T17:13:05+02:00 nathelper: initialize local variable --- Modified

[sr-dev] [kamailio/kamailio] nathelper: initialize local variable (#1661)

2018-10-01 Thread lazedo
Pre-Submission Checklist - [X] Commit message has the format required by CONTRIBUTING guide - [X] 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

[sr-dev] [kamailio/kamailio] siputils: add is_uri function (#1666)

2018-10-04 Thread lazedo
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) - [ ] No commits to README

Re: [sr-dev] [kamailio/kamailio] nathelper: fix call to alter_mediaip (#1655)

2018-09-28 Thread lazedo
Merged #1655 into 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/pull/1655#event-1873886544___ Kamailio (SER) - Development Mailing

[sr-dev] git:master:17c14a96: nathelper: fix call to alter_mediaip

2018-09-28 Thread lazedo
Module: kamailio Branch: master Commit: 17c14a966ff3ba67ce858dfe424d7cdf5100d4fa URL: https://github.com/kamailio/kamailio/commit/17c14a966ff3ba67ce858dfe424d7cdf5100d4fa Author: lazedo Committer: lazedo Date: 2018-09-28T18:00:22+02:00 nathelper: fix call to alter_mediaip --- Modified: src

[sr-dev] [kamailio/kamailio] tmx: add t_drop (#1726)

2018-11-16 Thread lazedo
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] [kamailio/kamailio] tm: bad error in log (#1727)

2018-11-16 Thread lazedo
### Description tm module writes a misleading log error that `triggers` ops alerts. ### Troubleshooting Reproduction using this partial script ``` event_route[core:worker-one-init] { async_route("DEFERRED_CLEANUP", "20"); } route[DEFERRED_CLEANUP] { xlog("L_INFO", "processing

Re: [sr-dev] [kamailio/kamailio] dispatcher : xavp_get fix in ds_update_dst (#1725)

2018-11-16 Thread lazedo
@miconda it works -- 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/1725#issuecomment-439497396___ Kamailio (SER) - Development Mailing List

[sr-dev] [kamailio/kamailio] dispatcher : xavp_get fix in ds_update_dst (#1725)

2018-11-16 Thread lazedo
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

Re: [sr-dev] [kamailio/kamailio] siptrace module not send ACK (#1811)

2019-01-16 Thread lazedo
@sergey-safarov i think the issue is in `freeswitch` you can try these PRs for freeswitch in order [merged] https://freeswitch.org/stash/projects/FS/repos/freeswitch/pull-requests/1660/overview https://freeswitch.org/stash/projects/FS/repos/freeswitch/pull-requests/1523/overview

  1   2   >