sr-dev

sr-dev@lists.kamailio.org

22 participants
45186 discussions

git:master:9a15781f: sipdump: use the right flag to parse ipv6
by Daniel-Constantin Mierla 17 Sep '22

17 Sep '22

Module: kamailio Branch: master Commit: 9a15781f4804ef6620fad5a09ff2e9aacae871cb URL: https://github.com/kamailio/kamailio/commit/9a15781f4804ef6620fad5a09ff2e9a… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-09-17T10:22:22+02:00 sipdump: use the right flag to parse ipv6 --- Modified: src/modules/sipdump/sipdump_pcap.c --- Diff: https://github.com/kamailio/kamailio/commit/9a15781f4804ef6620fad5a09ff2e9a… Patch: https://github.com/kamailio/kamailio/commit/9a15781f4804ef6620fad5a09ff2e9a… --- diff --git a/src/modules/sipdump/sipdump_pcap.c b/src/modules/sipdump/sipdump_pcap.c index e808ff23fe..0db2697790 100644 --- a/src/modules/sipdump/sipdump_pcap.c +++ b/src/modules/sipdump/sipdump_pcap.c @@ -198,8 +198,8 @@ void sipdump_write_pcap(FILE *fs, sipdump_data_t *spd) return; } memcpy(&v_pcap_ipv6_header.ip6_src, &ip6addr, sizeof(struct in6_addr)); - if (inet_pton(AF_INET, spd->dst_ip.s, &ip6addr) != 1) { - LM_ERR("failed to parse IPv4 address %s\n", spd->dst_ip.s); + if (inet_pton(AF_INET6, spd->dst_ip.s, &ip6addr) != 1) { + LM_ERR("failed to parse IPv6 address %s\n", spd->dst_ip.s); return; } memcpy(&v_pcap_ipv6_header.ip6_dst, &ip6addr, sizeof(struct in6_addr));

1 0

[kamailio/kamailio] pkg/kamailio/obs: added readline-devel build dependency (PR #3233)
by Gustavo Almeida 17 Sep '22

17 Sep '22

#### 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 files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated) #### 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 branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description The purpose of this pull request is to allow, when generating rpm's, the possibility of history/autocomplete to be seen in the interactive mode of kamcmd. The proposed solution is to ensure that the readline-devel library is a requirement when building Kamailio, thus ensuring that kamcmd is compiled with this library. Currently the specfile has as BuildRequires, the pkgconfig library. This library is installed by the readline-devel library (via ncurses-devel dependency), i.e. all existing logic remains unchanged. Furthermore, the readline-devel library is accessible on all distros that use this specfile, namely: Centos, Fedora, OpenSuse, RedHat. Initially in my testing, I thought I needed to change the Makefile of kamcmd to look at mock path’s (find path of readline.h), but this is not necessary (at least in CentOS). The possibility of adopting this solution in later branches would be useful, especially in the case of branch 5.6.1. The tests were performed in a CentOS 7 environment (docker image), following the usual process for generating the rpm's (make rpm). This solution was tested on branch 5.5.4 and on master. In both cases, the history/autocomplete is successfully visible in the interactive mode of kamcmd. ```Shell Script [root@kamailio-build kamailio]# cat /etc/centos-release CentOS Linux release 7.9.2009 (Core) [root@kamailio-build kamailio]# uname -a Linux kamailio-build 5.10.124-linuxkit #1 SMP PREEMPT Thu Jun 30 08:18:26 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux ``` I would appreciate if you could evaluate this solution, since the presence of the autocomplete in kamcmd is very useful. Feel free to adopt changes to the proposed solution. Thanks. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3233 -- Commit Summary -- * pkg/kamailio/obs: added readline-devel build dependency -- File Changes -- M pkg/kamailio/obs/kamailio.spec (4) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3233.patch https://github.com/kamailio/kamailio/pull/3233.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3233 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3233(a)github.com>

2 8

git:5.5:33dde3f6: pkg/kamailio/obs: added readline-devel build dependency (#3233)
by Sergey Safarov 16 Sep '22

16 Sep '22

Module: kamailio Branch: 5.5 Commit: 33dde3f66687b120123b227107a385129d65fc8e URL: https://github.com/kamailio/kamailio/commit/33dde3f66687b120123b227107a3851… Author: Gustavo Almeida <gustavo.abcdef(a)hotmail.com> Committer: Sergey Safarov <s.safarov(a)gmail.com> Date: 2022-09-16T23:13:18+03:00 pkg/kamailio/obs: added readline-devel build dependency (#3233) - Added a readline-devel dependency when generating rpm's. This ensures that when generating rpm's (mock context), kamcmd uses this library in its compilation and autocomplete is available in kamcmd's interactive mode. (cherry picked from commit e49e5a52ded4bef36b5958a748b9556af52d2d33) --- Modified: pkg/kamailio/obs/kamailio.spec --- Diff: https://github.com/kamailio/kamailio/commit/33dde3f66687b120123b227107a3851… Patch: https://github.com/kamailio/kamailio/commit/33dde3f66687b120123b227107a3851… --- diff --git a/pkg/kamailio/obs/kamailio.spec b/pkg/kamailio/obs/kamailio.spec index 4a26128e8c..1833f6086e 100644 --- a/pkg/kamailio/obs/kamailio.spec +++ b/pkg/kamailio/obs/kamailio.spec @@ -282,7 +282,7 @@ Conflicts: kamailio-utils < %ver, kamailio-websocket < %ver Conflicts: kamailio-xhttp-pi < %ver, kamailio-xmlops < %ver Conflicts: kamailio-xmlrpc < %ver, kamailio-xmpp < %ver Conflicts: kamailio-uuid < %ver -BuildRequires: bison, flex, which, make, gcc, gcc-c++, pkgconfig +BuildRequires: bison, flex, which, make, gcc, gcc-c++, pkgconfig, readline-devel %if 0%{?rhel} != 6 Requires: systemd BuildRequires: systemd-devel @@ -2359,6 +2359,8 @@ fi %changelog +* Tue Sep 13 2022 Gustavo Almeida <galmeida(a)broadvoice.com> + - added readline-devel build dependency * Sat Aug 31 2019 Sergey Safarov <s.safarov(a)gmail.com> 5.3.0-dev7 - Packaged kemix, lost and xhttp_prom modules * Sat Mar 30 2019 Sergey Safarov <s.safarov(a)gmail.com> 5.3.0-0

1 0

git:5.6:7cfa1fed: pkg/kamailio/obs: added readline-devel build dependency (#3233)
by Sergey Safarov 16 Sep '22

16 Sep '22

Module: kamailio Branch: 5.6 Commit: 7cfa1feddeeb205ebd3628580881397e9215d786 URL: https://github.com/kamailio/kamailio/commit/7cfa1feddeeb205ebd3628580881397… Author: Gustavo Almeida <gustavo.abcdef(a)hotmail.com> Committer: Sergey Safarov <s.safarov(a)gmail.com> Date: 2022-09-16T23:12:25+03:00 pkg/kamailio/obs: added readline-devel build dependency (#3233) - Added a readline-devel dependency when generating rpm's. This ensures that when generating rpm's (mock context), kamcmd uses this library in its compilation and autocomplete is available in kamcmd's interactive mode. (cherry picked from commit e49e5a52ded4bef36b5958a748b9556af52d2d33) --- Modified: pkg/kamailio/obs/kamailio.spec --- Diff: https://github.com/kamailio/kamailio/commit/7cfa1feddeeb205ebd3628580881397… Patch: https://github.com/kamailio/kamailio/commit/7cfa1feddeeb205ebd3628580881397… --- diff --git a/pkg/kamailio/obs/kamailio.spec b/pkg/kamailio/obs/kamailio.spec index 6a2d17d212..f4d7468cf1 100644 --- a/pkg/kamailio/obs/kamailio.spec +++ b/pkg/kamailio/obs/kamailio.spec @@ -290,7 +290,7 @@ Conflicts: kamailio-utils < %ver, kamailio-websocket < %ver Conflicts: kamailio-xhttp-pi < %ver, kamailio-xmlops < %ver Conflicts: kamailio-xmlrpc < %ver, kamailio-xmpp < %ver Conflicts: kamailio-uuid < %ver -BuildRequires: bison, flex, which, make, gcc, gcc-c++, pkgconfig +BuildRequires: bison, flex, which, make, gcc, gcc-c++, pkgconfig, readline-devel %if 0%{?rhel} != 6 Requires: systemd BuildRequires: systemd-devel @@ -2420,6 +2420,8 @@ fi %changelog +* Tue Sep 13 2022 Gustavo Almeida <galmeida(a)broadvoice.com> + - added readline-devel build dependency * Sat Aug 31 2019 Sergey Safarov <s.safarov(a)gmail.com> 5.3.0-dev7 - Packaged kemix, lost and xhttp_prom modules * Sat Mar 30 2019 Sergey Safarov <s.safarov(a)gmail.com> 5.3.0-0

1 0

git:master:e49e5a52: pkg/kamailio/obs: added readline-devel build dependency (#3233)
by GitHub 16 Sep '22

16 Sep '22

Module: kamailio Branch: master Commit: e49e5a52ded4bef36b5958a748b9556af52d2d33 URL: https://github.com/kamailio/kamailio/commit/e49e5a52ded4bef36b5958a748b9556… Author: Gustavo Almeida <gustavo.abcdef(a)hotmail.com> Committer: GitHub <noreply(a)github.com> Date: 2022-09-16T23:06:53+03:00 pkg/kamailio/obs: added readline-devel build dependency (#3233) * pkg/kamailio/obs: added readline-devel build dependency - Added a readline-devel dependency when generating rpm's. This ensures that when generating rpm's (mock context), kamcmd uses this library in its compilation and autocomplete is available in kamcmd's interactive mode. --- Modified: pkg/kamailio/obs/kamailio.spec --- Diff: https://github.com/kamailio/kamailio/commit/e49e5a52ded4bef36b5958a748b9556… Patch: https://github.com/kamailio/kamailio/commit/e49e5a52ded4bef36b5958a748b9556… --- diff --git a/pkg/kamailio/obs/kamailio.spec b/pkg/kamailio/obs/kamailio.spec index a82187d682..d1425f6874 100644 --- a/pkg/kamailio/obs/kamailio.spec +++ b/pkg/kamailio/obs/kamailio.spec @@ -290,7 +290,7 @@ Conflicts: kamailio-utils < %ver, kamailio-websocket < %ver Conflicts: kamailio-xhttp-pi < %ver, kamailio-xmlops < %ver Conflicts: kamailio-xmlrpc < %ver, kamailio-xmpp < %ver Conflicts: kamailio-uuid < %ver -BuildRequires: bison, flex, which, make, gcc, gcc-c++, pkgconfig +BuildRequires: bison, flex, which, make, gcc, gcc-c++, pkgconfig, readline-devel %if 0%{?rhel} != 6 Requires: systemd BuildRequires: systemd-devel @@ -2420,6 +2420,8 @@ fi %changelog +* Tue Sep 13 2022 Gustavo Almeida <galmeida(a)broadvoice.com> + - added readline-devel build dependency * Sat Aug 31 2019 Sergey Safarov <s.safarov(a)gmail.com> 5.3.0-dev7 - Packaged kemix, lost and xhttp_prom modules * Sat Mar 30 2019 Sergey Safarov <s.safarov(a)gmail.com> 5.3.0-0

1 0

git:master:de6a3501: corex: print address family name for rpc corex.list_sockets
by Daniel-Constantin Mierla 16 Sep '22

16 Sep '22

Module: kamailio Branch: master Commit: de6a35012669ad5892760c368cbd141e77bc91db URL: https://github.com/kamailio/kamailio/commit/de6a35012669ad5892760c368cbd141… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-09-16T16:48:19+02:00 corex: print address family name for rpc corex.list_sockets --- Modified: src/modules/corex/corex_rpc.c --- Diff: https://github.com/kamailio/kamailio/commit/de6a35012669ad5892760c368cbd141… Patch: https://github.com/kamailio/kamailio/commit/de6a35012669ad5892760c368cbd141… --- diff --git a/src/modules/corex/corex_rpc.c b/src/modules/corex/corex_rpc.c index 12fd8044afc..67aabde0477 100644 --- a/src/modules/corex/corex_rpc.c +++ b/src/modules/corex/corex_rpc.c @@ -65,7 +65,8 @@ static void corex_rpc_list_sockets(rpc_t* rpc, void* ctx) return; } - if(rpc->struct_add(th, "ss{", + if(rpc->struct_add(th, "sss{", + "AF", get_af_name(proto), "PROTO", get_valid_proto_name(proto), "NAME", si->name.s, "ADDRLIST", &ih)<0)

1 0

git:master:8bdee74f: core: helper function to get address family name
by Daniel-Constantin Mierla 16 Sep '22

16 Sep '22

Module: kamailio Branch: master Commit: 8bdee74fa6b68012919ddbb403da372187631bd9 URL: https://github.com/kamailio/kamailio/commit/8bdee74fa6b68012919ddbb403da372… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-09-16T16:42:16+02:00 core: helper function to get address family name --- Modified: src/core/ip_addr.c Modified: src/core/ip_addr.h --- Diff: https://github.com/kamailio/kamailio/commit/8bdee74fa6b68012919ddbb403da372… Patch: https://github.com/kamailio/kamailio/commit/8bdee74fa6b68012919ddbb403da372… --- diff --git a/src/core/ip_addr.c b/src/core/ip_addr.c index 77ac233eee..9c91c60e4e 100644 --- a/src/core/ip_addr.c +++ b/src/core/ip_addr.c @@ -717,6 +717,22 @@ char* get_proto_name(unsigned int proto) } } +/** get address family name (asciiz). + * @param af - address family id + * @return string with the adderess family name or "unknown". + */ +char* get_af_name(unsigned int af) +{ + switch(af) { + case AF_INET: + return "IPv4"; + case AF_INET6: + return "IPv6"; + default: + return "unknown"; + } +} + /** * match ip address with net address and bitmask diff --git a/src/core/ip_addr.h b/src/core/ip_addr.h index 4fcdebbe3e..7f910a49ce 100644 --- a/src/core/ip_addr.h +++ b/src/core/ip_addr.h @@ -304,6 +304,8 @@ char* get_proto_name(unsigned int proto); int get_valid_proto_string(unsigned int iproto, int utype, int vtype, str *sproto); +char* get_af_name(unsigned int af); + #ifdef USE_MCAST /* Returns 1 if the given address is a multicast address */ int is_mcast(struct ip_addr* ip);

1 0

git:master:a5d92da6: core: print address family for rpc core.sockets_list
by Daniel-Constantin Mierla 16 Sep '22

16 Sep '22

Module: kamailio Branch: master Commit: a5d92da6370f6db15c1ccb69a0ad7b914163c1db URL: https://github.com/kamailio/kamailio/commit/a5d92da6370f6db15c1ccb69a0ad7b9… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-09-16T16:45:21+02:00 core: print address family for rpc core.sockets_list --- Modified: src/core/core_cmd.c --- Diff: https://github.com/kamailio/kamailio/commit/a5d92da6370f6db15c1ccb69a0ad7b9… Patch: https://github.com/kamailio/kamailio/commit/a5d92da6370f6db15c1ccb69a0ad7b9… --- diff --git a/src/core/core_cmd.c b/src/core/core_cmd.c index c464d3686e..3c9f08e332 100644 --- a/src/core/core_cmd.c +++ b/src/core/core_cmd.c @@ -988,7 +988,8 @@ static void core_sockets_list(rpc_t* rpc, void* c) for(si=list?*list:0; si; si=si->next){ rpc->struct_add(hr, "{", "socket", &ha); if (si->addr_info_lst){ - rpc->struct_add(ha, "ss", + rpc->struct_add(ha, "sss", + "af", get_af_name(si->address.af), "proto", get_proto_name(proto), "address", si->address_str.s); for (ai=si->addr_info_lst; ai; ai=ai->next) @@ -1003,7 +1004,8 @@ static void core_sockets_list(rpc_t* rpc, void* c) printf(" %s: %s", get_proto_name(proto), si->name.s); - rpc->struct_add(ha, "ss", + rpc->struct_add(ha, "sss", + "af", get_af_name(si->address.af), "proto", get_proto_name(proto), "address", si->name.s); if (!(si->flags & SI_IS_IP))

1 0

[kamailio/kamailio] some kemi function exports (PR #3235)
by Stefan Mititelu 16 Sep '22

16 Sep '22

#### 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 files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated) #### 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:  - [ ] PR should be backported to stable branches - [X] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  Exported some functions to kemi. Currently testing. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3235 -- Commit Summary -- * ratelimit: export functions to kemi * siputils: export is_gruu() to kemi * dialplan: export dp_translate to kemi -- File Changes -- M src/modules/dialplan/dialplan.c (90) M src/modules/ratelimit/ratelimit.c (33) M src/modules/siputils/siputils.c (10) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3235.patch https://github.com/kamailio/kamailio/pull/3235.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3235 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3235(a)github.com>

3 6

git:master:4434e83d: websocket: debug message for rpc command when finding no tcp connection
by Daniel-Constantin Mierla 16 Sep '22

16 Sep '22

Module: kamailio Branch: master Commit: 4434e83db1d4b668dabbe915fe36719ede165915 URL: https://github.com/kamailio/kamailio/commit/4434e83db1d4b668dabbe915fe36719… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-09-16T15:00:13+02:00 websocket: debug message for rpc command when finding no tcp connection --- Modified: src/modules/websocket/ws_conn.c --- Diff: https://github.com/kamailio/kamailio/commit/4434e83db1d4b668dabbe915fe36719… Patch: https://github.com/kamailio/kamailio/commit/4434e83db1d4b668dabbe915fe36719… --- diff --git a/src/modules/websocket/ws_conn.c b/src/modules/websocket/ws_conn.c index 0015dfa778..a56eb49238 100644 --- a/src/modules/websocket/ws_conn.c +++ b/src/modules/websocket/ws_conn.c @@ -766,8 +766,10 @@ static int ws_rpc_add_node( tcpconn_put(con); return 1; - } else + } else { + LM_DBG("ws structure [%p] without an active tcp connection\n", wsc); return 0; + } } void ws_rpc_dump(rpc_t *rpc, void *ctx)

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev