[sr-dev] [kamailio/kamailio] remove lib/kcore/faked_msg.h (#880)

2016-12-02 Thread Mikko Lehto
Looks like lib/kcore/faked_msg.h is not needed anymore You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/880 -- Commit Summary -- * modules: faked message api has moved to core * lib/kcore: faked_msg.h not referenced anymore -- File

[sr-dev] git:master:938ac5e7: modules/ims_qos: added patch for flow-description bug when request originates from called party

2016-12-02 Thread Richard Good
Module: kamailio Branch: master Commit: 938ac5e7223001fb9e3b748262283d323a23c539 URL: https://github.com/kamailio/kamailio/commit/938ac5e7223001fb9e3b748262283d323a23c539 Author: Richard Good Committer: Richard Good Date:

Re: [sr-dev] [kamailio/kamailio] sipcapture: expose pv $hep(src_ip) (#879)

2016-12-02 Thread David Villasmil
@adubovikov I hope this is better. If you find I'm not adhering in any way to guidelines, please point them out as it's my first time contributing. Many thanks! David -- 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] sipcapture: expose pv $hep(src_ip) (#879)

2016-12-02 Thread David Villasmil
- modified pv_parse_hep_name() and pv_get_hep() to expose the HEP's packet source ip address on replies. The associated pv $hep(src_ip), i.e.: onreply_route { ... xlog("[onreply_route] Source IP: $hep(src_ip)\n"); ... } You can view, comment on, or merge this pull request

Re: [sr-dev] [kamailio/kamailio] kamailio crashes on CANCEL due to empty reply_lumps (#872)

2016-12-02 Thread apogrebennyk
Hi @miconda, thanks for the good point. There was indeed another call to append_to_reply in failure_route, rather due to an oversight than by intention. After removing it I'm not getting any new crashes. -- You are receiving this because you are subscribed to this thread. Reply to this email

Re: [sr-dev] [kamailio/kamailio] sipcapture: expose pv $hep(src_ip) (#877)

2016-12-02 Thread David Villasmil
Closed #877. -- 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/877#event-879881875___ sr-dev mailing list sr-dev@lists.sip-router.org

Re: [sr-dev] [kamailio/kamailio] Sipcapture: When inserting a reply, this module would insert the SIP … (#876)

2016-12-02 Thread David Villasmil
I've been looking into Homer's source code, and you are correct. It's easier doing it on the resultCtrl.js. In any case, this has been a good exercise for me in terms of working with Kamailio's source code, as I'm just a beginner. Thanks! -- You are receiving this because you are subscribed

Re: [sr-dev] [kamailio/kamailio] Sipcapture: When inserting a reply, this module would insert the SIP … (#876)

2016-12-02 Thread David Villasmil
Closed #876. -- 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/876#event-879833214___ sr-dev mailing list sr-dev@lists.sip-router.org

Re: [sr-dev] [kamailio/kamailio] kamailio crashes on CANCEL due to empty reply_lumps (#872)

2016-12-02 Thread Daniel-Constantin Mierla
Are you adding the reply headers in the failure_route? Or in the request_route? The transaction is not deleted immediately once it is cancelled (or sends out a negative response), but it is kept in memory for wait_time to absorb the retransmissions. So this situation might be in the case one

Re: [sr-dev] [SR-Users] VERY IMPORTANT: deciding when to remove the MI code

2016-12-02 Thread Daniel-Constantin Mierla
On 02/12/2016 11:21, Alex Hermann wrote: > On donderdag 1 december 2016 15:17:18 CET Daniel-Constantin Mierla wrote: >> RPC is the alternative, a more standardized >> concept, with better structured format. > Before removing MI and letting everyone move to RPC, it might be wise to go > over all

Re: [sr-dev] [kamailio/kamailio] Kamailio crashes when it receives a 404 reply to an INVITE originated after a 302. (#875)

2016-12-02 Thread Daniel-Constantin Mierla
The backtrace is not pointing correctly to source code -- the lines in the files referred in the first frames of the backtrace are not containing related code. Can you reproduce the issue and attach the backtrace full from a new core file? Be sure you use latest version of branch 4.4 (nightly

Re: [sr-dev] [kamailio/kamailio] ratelimit module not working properly when FEEDBACK algorithm is used (#873)

2016-12-02 Thread Daniel-Constantin Mierla
Anyone having more comments on this pull request or issue #871? I all fine I will merge. Also, I guess that the pipelimit is affected as well. -- 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] implement pua.publish rpc command in order to get rid of mi interface (#878)

2016-12-02 Thread juha-h
There was also another later thread related to my pua.publish effort: http://lists.sip-router.org/pipermail/sr-dev/2014-December/026293.html -- 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] implement pua.publish rpc command in order to get rid of mi interface (#878)

2016-12-02 Thread juha-h
There is currently no rpc version of mi pua_publish. I once tried to implement it, but was not successful: http://lists.sip-router.org/pipermail/sr-dev/2014-November/026177.html Perhaps someone else could complete the job. -- Juha -- You are receiving this because you are subscribed to

[sr-dev] git:master:b72af394: modules: readme files regenerated - jsonrpc-s ...

2016-12-02 Thread Kamailio Dev
Module: kamailio Branch: master Commit: b72af394936be5b69cb83ff768f11e2dc33bfc19 URL: https://github.com/kamailio/kamailio/commit/b72af394936be5b69cb83ff768f11e2dc33bfc19 Author: Kamailio Dev Committer: Kamailio Dev Date:

Re: [sr-dev] [SR-Users] VERY IMPORTANT: deciding when to remove the MI code

2016-12-02 Thread Daniel-Constantin Mierla
On 02/12/2016 00:24, Juha Heinanen wrote: > Brandon Armstead writes: > >> +1 - I'm all for cleaning up any technical debt and moving on with more >> normalized concept. > I'm also in favor of getting rid on MI interface. I'm myself relying > only on one MI command (pua_publish) for which an RPC

[sr-dev] git:master:e8086c7d: kamctl: implemented moni command for jsonrpc engine

2016-12-02 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: e8086c7db45d5b0136e4c5871cb1a2c5a0e1d5d5 URL: https://github.com/kamailio/kamailio/commit/e8086c7db45d5b0136e4c5871cb1a2c5a0e1d5d5 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

[sr-dev] git:master:10912437: jsonrpc-s: use same name when setting user/group via str/int values

2016-12-02 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 10912437a992048c4b5ab297ace4f29cbf9b9f3d URL: https://github.com/kamailio/kamailio/commit/10912437a992048c4b5ab297ace4f29cbf9b9f3d Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

[sr-dev] git:master:cf5a7487: jsonrpc-s: added documentation for datagram transport

2016-12-02 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: cf5a748719e4770e481601e75adb3e89b33d910b URL: https://github.com/kamailio/kamailio/commit/cf5a748719e4770e481601e75adb3e89b33d910b Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: