[sr-dev] git:master:67419f91: sipcapture: Fixed missing 'break' in pv_parse_hep_name switch

2017-02-13 Thread David Villasmil
Module: kamailio Branch: master Commit: 67419f91a4b6f2ad5d2590434b64087cf812b794 URL: https://github.com/kamailio/kamailio/commit/67419f91a4b6f2ad5d2590434b64087cf812b794 Author: David Villasmil <david.villasmil.w...@gmail.com> Committer: David Villasmil <david.villasmil.w...@gmail.

[sr-dev] [kamailio/kamailio] sipcapture: Added exported pseudo-variable $hep(dst_ip) and updated README (#995)

2017-02-15 Thread David Villasmil
Added a new exported pseudo-variable to expose HEP's message's original source ip address. Added an "Exported pseudo-variabled" to the README file to document $hep(src_ip) and $hep(dst_ip). You can view, comment on, or merge this pull request online at:

[sr-dev] [kamailio/kamailio] sipcapture: Added exported pseudo-variable $hep(dst_ip) and updated README (#994)

2017-02-15 Thread David Villasmil
Added a new exported pseudo-variable to expose HEP's message's original source ip address. Added an "Exported pseudo-variabled" to the README file to document $hep(src_ip) and $hep(dst_ip). You can view, comment on, or merge this pull request online at:

[sr-dev] git:master:6960cb80: sipcapture: updated README to document exported variables

2017-02-15 Thread David Villasmil
Module: kamailio Branch: master Commit: 6960cb80e0d3fe8103306e65258d62a9ab6620c7 URL: https://github.com/kamailio/kamailio/commit/6960cb80e0d3fe8103306e65258d62a9ab6620c7 Author: David Villasmil <david.villasmil.w...@gmail.com> Committer: David Villasmil <david.villasmil.w...@gmail.

[sr-dev] git:master:16cea04b: sipcapture: Fixed missing 'break' after case-switch.

2017-02-15 Thread David Villasmil
Module: kamailio Branch: master Commit: 16cea04b08c648bd32b036e35bdb10f4bfdafdff URL: https://github.com/kamailio/kamailio/commit/16cea04b08c648bd32b036e35bdb10f4bfdafdff Author: David Villasmil <david.villasmil.w...@gmail.com> Committer: David Villasmil <david.villasmil.w...@gmail.

[sr-dev] git:master:4f960122: sipcapture: added $hep(dst_ip) variable

2017-02-15 Thread David Villasmil
Module: kamailio Branch: master Commit: 4f960122142c845862ad140e85827a1c530bc22e URL: https://github.com/kamailio/kamailio/commit/4f960122142c845862ad140e85827a1c530bc22e Author: David Villasmil <david.villasmil.w...@gmail.com> Committer: David Villasmil <david.villasmil.w...@gmail.

Re: [sr-dev] [kamailio/kamailio] sipcapture: Added exported pseudo-variable $hep(dst_ip) and updated README (#994)

2017-02-15 Thread David Villasmil
Closed #994. -- 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/994#event-963637659___ sr-dev mailing list sr-dev@lists.sip-router.org

[sr-dev] [kamailio/kamailio] sipcapture: Fixed missing 'break' in pv_parse_hep_name switch (#985)

2017-02-13 Thread David Villasmil
- Added missing 'break;' in sipcapture.c's pv_parse_hep_name function's switch statement. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/985 -- Commit Summary -- * sipcapture: Fixed missing 'break' in pv_parse_hep_name switch --

Re: [sr-dev] [kamailio/kamailio] sipcapture: Fixed missing 'break' in pv_parse_hep_name switch (#985)

2017-02-13 Thread David Villasmil
Closed #985. -- 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/985#event-959553499___ sr-dev mailing list sr-dev@lists.sip-router.org

[sr-dev] [kamailio/kamailio] sipcapture: Fixed missing 'break' in pv_parse_hep_name switch (#986)

2017-02-13 Thread David Villasmil
- Added missing 'break;' in sipcapture.c's pv_parse_hep_name function's switch statement. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/986 -- Commit Summary -- * sipcapture: Fixed missing 'break' in pv_parse_hep_name switch --

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

2016-12-08 Thread David Villasmil
Yes, sorry about those beautifications... vim (spf13 mod) did it automatically.. i'll be more careful in the future. -- 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] 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] 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

[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] 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: 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: When inserting a reply, this module would insert the SIP … (#876)

2016-12-01 Thread David Villasmil
…Status as the reply method. I've patched sipcapture.c and sipcapture.h to insert the actual SIP Status into a 'status' field on replies, and the Method extracted from the CSEQ into the 'method' field. This means when using this version of the module, you're tables must include a 'status'

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

2016-12-01 Thread David Villasmil
The way it works now, on a reply the SIP Status is inserted into the "method" field, this isn't very nice as, if you're working with the table, you'd need to actually look into the cseq and break it into Sequence and Method to be able to do anything useful. Then you'd need to check if the

[sr-dev] sipcapture sip-status and method

2016-11-30 Thread David Villasmil
ed to accept the status field. How do i do the pull request? I don't seem to be allowed to do commits to a branch i created locally Regards, Regards, David Villasmil email: david.villasmil.w...@gmail.com <https://mail.google.com/mail/?view=cm=1=1=david.vi

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

2016-12-01 Thread David Villasmil
- when using with Homer, on onreply_route it is not possible to get the source ip address sending the reply. This patch exposes 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

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

2016-12-06 Thread David Villasmil
@davidcsi pushed 1 commit. fb6328e sipcapture: Set char buf_ip as static -- You are receiving this because you are subscribed to this thread. View it on GitHub:

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

2016-12-06 Thread David Villasmil
Changed "char buf_ip" as static -- 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/879#issuecomment-265088845___ sr-dev mailing list