[sr-dev] git:master:c85ec2ee: core: fix timer typo

2019-05-05 Thread Seven Du
Module: kamailio Branch: master Commit: c85ec2eed71c5d1bc3957437f27ffe9b6f18f4df URL: https://github.com/kamailio/kamailio/commit/c85ec2eed71c5d1bc3957437f27ffe9b6f18f4df Author: Seven Du Committer: Seven Du Date: 2019-05-04T07:30:23+08:00 core: fix timer typo --- Modified: src/core/timer.c

[sr-dev] [kamailio/kamailio] add rport to uac register msg (#2639)

2021-02-16 Thread Seven Du
### Description Sometimes need to add rport on outbound uac reg, but seems it's not possible in the current uac module. Would be good to an an extra param to do it, or is it possible to trigger a KEMI event route so a script can add whatever it need before sending the register msg?

Re: [sr-dev] [kamailio/kamailio] add rport to uac register msg (#2639)

2021-02-16 Thread Seven Du
with docker image image: kamailio/kamailio-ci:5.4.4-alpine added the following config I get err local_rport=yes ``` xcc-kamailio| 0(9) CRITICAL: [core/cfg.y:3592]: yyerror_at(): parse error in config file /etc/kamailio/kamailio.cfg, line 128, column 12: syntax error xcc-kamailio

Re: [sr-dev] [kamailio/kamailio] add rport to uac register msg (#2639)

2021-02-16 Thread Seven Du
on the master ``` commit fd8d0f66c80f0ab296bcaff08713a2eb578dd886 (HEAD -> master, origin/master, origin/HEAD) Author: Daniel-Constantin Mierla Date: Mon Feb 15 12:11:33 2021 +0100 Makefile.groups: added secsipid_proc to secsipid group ``` -- You are receiving this because you are

Re: [sr-dev] [kamailio/kamailio] add rport to uac register msg (#2639)

2021-02-16 Thread Seven Du
built master and local_rport is in but seems has no effect. the lua event callback get called, but as soon as it hit `KSR.add_local_rport();` it crashes. comment out that line doesn't crash. some bt: ``` Core was generated by `/usr/local/sbin/kamailio -S -DD -dd -E -e -m 32 -M 2 -A

Re: [sr-dev] [kamailio/kamailio] add rport to uac register msg (#2639)

2021-02-16 Thread Seven Du
updated and now hit another problem. again, comment out the `add_rport` function removes the err. ``` 11(37206) INFO: [core/kemi.c:105]: sr_kemi_core_info(): = tm module triggered event: tm:local-request 11(37206) INFO: [core/kemi.c:105]: sr_kemi_core_info(): = add local rport =

Re: [sr-dev] [kamailio/kamailio] add rport to uac register msg (#2639)

2021-02-17 Thread Seven Du
I added some debug log, the `create_via_hf` func is not been called when `FL_ADD_LOCAL_RPORT` is not set, so in the following line it's going to the `else` branch unless force send socket is set. https://github.com/kamailio/kamailio/blob/master/src/modules/tm/uac.c#L315 -- You are receiving

Re: [sr-dev] [kamailio/kamailio] add rport to uac register msg (#2639)

2021-02-17 Thread Seven Du
thanks a lot, it's working. gloal `local_rport=yes` has no effect. and the event callback works for me. -- 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] add rport to uac register msg (#2639)

2021-02-17 Thread Seven Du
Closed #2639. -- 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/2639#event-4339932691___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] add rport to uac register msg (#2639)

2021-02-17 Thread Seven Du
cool. it's working now. -- 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/2639#issuecomment-781009054___ Kamailio (SER) - Development

[sr-dev] Re: [kamailio/kamailio] core: fix a seg while t_continue with nats module (PR #3396)

2023-03-22 Thread Seven Du
https://github.com/kamailio/kamailio/commit/ffbb6ad2837326e7309be342ce62527c16f6db16 fixed this one. should I add some warning log or close this? -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3396#issuecomment-1480568852 You are receiving this

[sr-dev] Re: [kamailio/kamailio] nats: add a reply param to nats_publish and expose as nats_publish_request in KEMI (PR #3397)

2023-03-22 Thread Seven Du
@seven1240 commented on this pull request. > @@ -60,6 +60,8 @@ static param_export_t params[] = { static cmd_export_t cmds[] = { {"nats_publish", (cmd_function)w_nats_publish_f, 2, fixup_publish_get_value, fixup_publish_get_value_free, ANY_ROUTE}, I removed and

[sr-dev] Re: [kamailio/kamailio] nats: add a reply param to nats_publish and expose as nats_publish_request in KEMI (PR #3397)

2023-03-22 Thread Seven Du
@seven1240 pushed 1 commit. 0ca307be15ea26a45252e68c0f27f44f7c0279b4 nats: add a reply param to nats_publish and expose as nats_publish_request in KEMI -- View it on GitHub:

[sr-dev] [kamailio/kamailio] core: fix a seg while t_continue with nats module (PR #3396)

2023-03-20 Thread Seven Du
!-- 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

[sr-dev] Re: [kamailio/kamailio] nats: add a reply param to nats_publish and expose as nats_publish_request in KEMI (PR #3397)

2023-03-23 Thread Seven Du
> does this change require a 3rd parameter to `nats_publish()` ? No, the 3rd one is optional. ``` route[nats] { nats_publish("cn.xswitch.ctrl.route", 'hello'); nats_publish("cn.xswitch.ctrl.route", 'hello', 'blah'); } ``` -- Reply to this email directly or view it on GitHub:

[sr-dev] Re: [kamailio/kamailio] nats: add a reply param to nats_publish and expose as nats_publish_request in KEMI (PR #3397)

2023-03-24 Thread Seven Du
@seven1240 pushed 1 commit. 76e5317cca8337d5de78b4506a91cd5cca789f7c nats: add a reply param to nats_publish and expose as nats_publish_request in KEMI -- View it on GitHub:

[sr-dev] [kamailio/kamailio] nats: add a reply param to nats_publish and expose as nats_publish_request in KEMI (PR #3397)

2023-03-21 Thread Seven Du
!-- 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

[sr-dev] Re: [kamailio/kamailio] nats: add a reply param to nats_publish and expose as nats_publish_request in KEMI (PR #3397)

2023-03-22 Thread Seven Du
@eschmidbauer can you take a look if this one makes sense? Thanks. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3397#issuecomment-1479521579 You are receiving this because you are subscribed to this thread. Message ID: