sr-dev May 2023

sr-dev@lists.kamailio.org

16 participants
319 discussions

git:master:339891e7: pre-commit: add mixed-line-ending hook [skip ci]

by Victor Seva

Module: kamailio Branch: master Commit: 339891e7da584abcc378aa2104f536841d615e57 URL: https://github.com/kamailio/kamailio/commit/339891e7da584abcc378aa2104f5368… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2023-05-19T01:40:44+02:00 pre-commit: add mixed-line-ending hook [skip ci] --- Modified: .pre-commit-config.yaml --- Diff: https://github.com/kamailio/kamailio/commit/339891e7da584abcc378aa2104f5368… Patch: https://github.com/kamailio/kamailio/commit/339891e7da584abcc378aa2104f5368… --- diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 6bd7b3d08e..7e8b8a8860 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -7,6 +7,7 @@ repos: - id: end-of-file-fixer - id: trailing-whitespace - id: check-merge-conflict + - id: mixed-line-ending - repo: https://github.com/pre-commit/mirrors-clang-format rev: v15.0.7 hooks:

12 months

[kamailio/kamailio] core: clang-format for coherent indentation and coding style (PR #3461)

by Victor Seva

> for i in $(find src/core -name .git -prune -o -regextype posix-egrep -regex '^.*\.(c|h)$') ; do clang-format -i $i; done You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3461 -- Commit Summary -- * core: clang-format for coherent indentation and coding style -- File Changes -- M src/core/action.c (1735) M src/core/action.h (21) M src/core/async_task.c (183) M src/core/async_task.h (6) M src/core/atomic/atomic_alpha.h (371) M src/core/atomic/atomic_arm.h (313) M src/core/atomic/atomic_common.h (15) M src/core/atomic/atomic_mips2.h (380) M src/core/atomic/atomic_native.h (4) M src/core/atomic/atomic_ppc.h (302) M src/core/atomic/atomic_sparc.h (55) M src/core/atomic/atomic_sparc64.h (328) M src/core/atomic/atomic_unknown.h (263) M src/core/atomic/atomic_x86.h (382) M src/core/atomic_ops.c (47) M src/core/atomic_ops.h (2) M src/core/basex.c (551) M src/core/basex.h (701) M src/core/bit_count.h (8) M src/core/bit_scan.c (14) M src/core/bit_scan.h (259) M src/core/bit_test.h (51) M src/core/cfg/cfg.c (197) M src/core/cfg/cfg.h (74) M src/core/cfg/cfg_ctx.c (1116) M src/core/cfg/cfg_ctx.h (112) M src/core/cfg/cfg_script.c (288) M src/core/cfg/cfg_script.h (32) M src/core/cfg/cfg_select.c (202) M src/core/cfg/cfg_select.h (10) M src/core/cfg/cfg_struct.c (674) M src/core/cfg/cfg_struct.h (314) M src/core/cfg_core.c (457) M src/core/cfg_core.h (53) M src/core/cfg_parser.c (982) M src/core/cfg_parser.h (98) M src/core/char_msg_val.h (37) M src/core/clist.h (64) M src/core/compiler_opt.h (11) M src/core/config.h (138) M src/core/core_cmd.c (1192) M src/core/core_stats.h (19) M src/core/counters.c (0) M src/core/counters.h (0) M src/core/crc.c (0) M src/core/crc.h (0) M src/core/crypto/md5.c (0) M src/core/crypto/md5.h (0) M src/core/crypto/md5utils.c (0) M src/core/crypto/md5utils.h (0) M src/core/crypto/rijndael.c (0) M src/core/crypto/rijndael.h (0) M src/core/crypto/sha256.c (0) M src/core/crypto/sha256.h (0) M src/core/crypto/shautils.c (0) M src/core/daemonize.c (0) M src/core/daemonize.h (0) M src/core/data_lump.c (0) M src/core/data_lump.h (0) M src/core/data_lump_rpl.c (0) M src/core/data_lump_rpl.h (0) M src/core/dns_cache.c (0) M src/core/dns_cache.h (0) M src/core/dns_func.c (0) M src/core/dns_func.h (0) M src/core/dns_wrappers.h (0) M src/core/dprint.c (0) M src/core/dprint.h (0) M src/core/dset.c (0) M src/core/dset.h (0) M src/core/dst_blocklist.c (0) M src/core/dst_blocklist.h (0) M src/core/endianness.c (0) M src/core/endianness.h (0) M src/core/error.c (0) M src/core/error.h (0) M src/core/events.c (0) M src/core/events.h (0) M src/core/fastlock.h (0) M src/core/fix_lumps.h (0) M src/core/flags.c (0) M src/core/flags.h (0) M src/core/fmsg.c (0) M src/core/fmsg.h (0) M src/core/forward.c (0) M src/core/forward.h (0) M src/core/futexlock.h (0) M src/core/globals.h (0) M src/core/hash_func.c (0) M src/core/hash_func.h (0) M src/core/hashes.h (0) M src/core/id.c (0) M src/core/id.h (0) M src/core/io_wait.c (0) M src/core/io_wait.h (0) M src/core/ip_addr.c (0) M src/core/ip_addr.h (0) M src/core/kemi.c (0) M src/core/kemi.h (0) M src/core/kemiexec.c (0) M src/core/list.h (0) M src/core/local_timer.c (0) M src/core/local_timer.h (0) M src/core/lock_alloc.h (0) M src/core/lock_ops.c (0) M src/core/lock_ops.h (0) M src/core/locking.c (0) M src/core/locking.h (0) M src/core/lump_struct.h (0) M src/core/lvalue.c (0) M src/core/lvalue.h (0) M src/core/mem/dl_malloc.c (0) M src/core/mem/dl_malloc.h (0) M src/core/mem/f_malloc.c (0) M src/core/mem/f_malloc.h (0) M src/core/mem/ll_malloc.c (0) M src/core/mem/ll_malloc.h (0) M src/core/mem/mem.h (0) M src/core/mem/memapi.h (0) M src/core/mem/memdbg.h (0) M src/core/mem/meminfo.h (0) M src/core/mem/memtest.c (0) M src/core/mem/pkg.c (0) M src/core/mem/pkg.h (0) M src/core/mem/q_malloc.c (0) M src/core/mem/q_malloc.h (0) M src/core/mem/sf_malloc.c (0) M src/core/mem/sf_malloc.h (0) M src/core/mem/shm.c (0) M src/core/mem/shm.h (0) M src/core/mem/shm_mem.h (0) M src/core/mem/src_loc.h (0) M src/core/mem/tlsf_malloc.c (0) M src/core/mem/tlsf_malloc.h (0) M src/core/mem/tlsf_malloc_bits.h (0) M src/core/mod_fix.c (0) M src/core/mod_fix.h (0) M src/core/modparam.c (0) M src/core/modparam.h (0) M src/core/msg_translator.c (0) M src/core/msg_translator.h (0) M src/core/name_alias.h (0) M src/core/nonsip_hooks.c (0) M src/core/nonsip_hooks.h (0) M src/core/onsend.c (0) M src/core/onsend.h (0) M src/core/parser/contact/contact.c (0) M src/core/parser/contact/contact.h (0) M src/core/parser/contact/parse_contact.c (0) M src/core/parser/contact/parse_contact.h (0) M src/core/parser/digest/digest.c (0) M src/core/parser/digest/digest.h (0) M src/core/parser/digest/digest_keys.h (0) M src/core/parser/digest/digest_parser.c (0) M src/core/parser/digest/digest_parser.h (0) M src/core/parser/digest/param_parser.c (0) M src/core/parser/digest/param_parser.h (0) M src/core/parser/hf.c (0) M src/core/parser/hf.h (0) M src/core/parser/keys.h (0) M src/core/parser/msg_parser.c (0) M src/core/parser/msg_parser.h (0) M src/core/parser/parse_addr_spec.c (0) M src/core/parser/parse_addr_spec.h (0) M src/core/parser/parse_allow.c (0) M src/core/parser/parse_allow.h (0) M src/core/parser/parse_body.c (0) M src/core/parser/parse_body.h (0) M src/core/parser/parse_content.c (0) M src/core/parser/parse_content.h (0) M src/core/parser/parse_cseq.c (0) M src/core/parser/parse_cseq.h (0) M src/core/parser/parse_date.c (0) M src/core/parser/parse_date.h (0) M src/core/parser/parse_def.h (0) M src/core/parser/parse_disposition.c (0) M src/core/parser/parse_disposition.h (0) M src/core/parser/parse_diversion.c (0) M src/core/parser/parse_diversion.h (0) M src/core/parser/parse_event.c (0) M src/core/parser/parse_event.h (0) M src/core/parser/parse_expires.c (0) M src/core/parser/parse_expires.h (0) M src/core/parser/parse_fline.c (0) M src/core/parser/parse_fline.h (0) M src/core/parser/parse_from.h (0) M src/core/parser/parse_hname2.c (0) M src/core/parser/parse_hname2.h (0) M src/core/parser/parse_identity.c (0) M src/core/parser/parse_identity.h (0) M src/core/parser/parse_identityinfo.c (0) M src/core/parser/parse_identityinfo.h (0) M src/core/parser/parse_methods.c (0) M src/core/parser/parse_methods.h (0) M src/core/parser/parse_nameaddr.c (0) M src/core/parser/parse_nameaddr.h (0) M src/core/parser/parse_option_tags.c (0) M src/core/parser/parse_option_tags.h (0) M src/core/parser/parse_param.c (0) M src/core/parser/parse_param.h (0) M src/core/parser/parse_ppi_pai.c (0) M src/core/parser/parse_ppi_pai.h (0) M src/core/parser/parse_privacy.c (0) M src/core/parser/parse_privacy.h (0) M src/core/parser/parse_refer_to.c (0) M src/core/parser/parse_refer_to.h (0) M src/core/parser/parse_require.c (0) M src/core/parser/parse_require.h (0) M src/core/parser/parse_retry_after.c (0) M src/core/parser/parse_retry_after.h (0) M src/core/parser/parse_rpid.c (0) M src/core/parser/parse_rpid.h (0) M src/core/parser/parse_rr.c (0) M src/core/parser/parse_rr.h (0) M src/core/parser/parse_sipifmatch.c (0) M src/core/parser/parse_sipifmatch.h (0) M src/core/parser/parse_subscription_state.c (0) M src/core/parser/parse_subscription_state.h (0) M src/core/parser/parse_supported.c (0) M src/core/parser/parse_supported.h (0) M src/core/parser/parse_to.c (0) M src/core/parser/parse_to.h (0) M src/core/parser/parse_uri.c (0) M src/core/parser/parse_uri.h (0) M src/core/parser/parse_via.c (0) M src/core/parser/parse_via.h (0) M src/core/parser/parser_f.h (0) M src/core/parser/sdp/sdp.c (0) M src/core/parser/sdp/sdp.h (0) M src/core/parser/sdp/sdp_cloner.h (0) M src/core/parser/sdp/sdp_helpr_funcs.c (0) M src/core/parser/sdp/sdp_helpr_funcs.h (0) M src/core/pass_fd.c (0) M src/core/pass_fd.h (0) M src/core/poll_types.h (0) M src/core/ppcfg.c (0) M src/core/ppcfg.h (0) M src/core/proxy.c (0) M src/core/proxy.h (0) M src/core/pt.c (0) M src/core/pt.h (0) M src/core/pv_core.c (0) M src/core/pvapi.c (0) M src/core/pvapi.h (0) M src/core/pvar.h (0) M src/core/qvalue.c (0) M src/core/qvalue.h (0) M src/core/rad_dict.h (0) M src/core/rand/cryptorand.c (0) M src/core/rand/fastrand.c (0) M src/core/rand/fortuna/fortuna.c (0) M src/core/rand/fortuna/random.c (0) M src/core/rand/fortuna/random.h (0) M src/core/rand/isaac/rand.c (0) M src/core/rand/isaac/rand.h (0) M src/core/rand/isaac/standard.h (0) M src/core/rand/kam_rand.h (0) M src/core/raw_listener.c (0) M src/core/raw_listener.h (0) M src/core/raw_sock.c (0) M src/core/raw_sock.h (0) M src/core/re.c (0) M src/core/re.h (0) M src/core/receive.c (0) M src/core/receive.h (0) M src/core/resolve.c (0) M src/core/resolve.h (0) M src/core/route.c (0) M src/core/route.h (0) M src/core/route_struct.c (0) M src/core/route_struct.h (0) M src/core/rpc.h (0) M src/core/rpc_lookup.c (0) M src/core/rpc_lookup.h (0) M src/core/rvalue.c (0) M src/core/rvalue.h (0) M src/core/sched_yield.h (0) M src/core/script_cb.c (0) M src/core/script_cb.h (0) M src/core/sctp_core.c (0) M src/core/sctp_core.h (0) M src/core/select.c (0) M src/core/select.h (0) M src/core/select_buf.c (0) M src/core/select_buf.h (0) M src/core/select_core.c (0) M src/core/select_core.h (0) M src/core/ser_time.h (0) M src/core/shm_init.c (0) M src/core/signals.c (0) M src/core/signals.h (0) M src/core/sock_ut.c (0) M src/core/sock_ut.h (0) M src/core/socket_info.c (0) M src/core/socket_info.h (0) M src/core/sr_compat.c (0) M src/core/sr_module.c (0) M src/core/sr_module.h (0) M src/core/srapi.c (0) M src/core/srapi.h (0) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3461.patch https://github.com/kamailio/kamailio/pull/3461.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3461 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3461(a)github.com>

12 months

git:master:e66a63a2: README: add pre-commit badge

by Victor Seva

Module: kamailio Branch: master Commit: e66a63a230e91461af0a82e3bbc54eabbf5195c7 URL: https://github.com/kamailio/kamailio/commit/e66a63a230e91461af0a82e3bbc54ea… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2023-05-19T01:08:17+02:00 README: add pre-commit badge --- Modified: README.md --- Diff: https://github.com/kamailio/kamailio/commit/e66a63a230e91461af0a82e3bbc54ea… Patch: https://github.com/kamailio/kamailio/commit/e66a63a230e91461af0a82e3bbc54ea… --- diff --git a/README.md b/README.md index aff44050c6a..a4616f9d66c 100644 --- a/README.md +++ b/README.md @@ -2,6 +2,7 @@ [![Build Status](https://github.com/kamailio/kamailio/actions/workflows/main.yml/bad… [![Code Triage Badge](https://www.codetriage.com/kamailio/kamailio/badges/users.svg)](http… +[![pre-commit](https://img.shields.io/badge/pre--commit-enabled-brightgreen?logo=pre-commit)](https://github.com/pre-commit/pre-commit) Project Website:

12 months

git:master:e15a57e0: pre-commit: update pre-commit-hooks [skip ci]

by Victor Seva

Module: kamailio Branch: master Commit: e15a57e0e027425827509277203341d55ace49fa URL: https://github.com/kamailio/kamailio/commit/e15a57e0e027425827509277203341d… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2023-05-19T00:51:27+02:00 pre-commit: update pre-commit-hooks [skip ci] --- Modified: .pre-commit-config.yaml --- Diff: https://github.com/kamailio/kamailio/commit/e15a57e0e027425827509277203341d… Patch: https://github.com/kamailio/kamailio/commit/e15a57e0e027425827509277203341d… --- diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index f9af7b878e1..6bd7b3d08e1 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -1,6 +1,6 @@ repos: - repo: https://github.com/pre-commit/pre-commit-hooks - rev: v2.3.0 + rev: v4.4.0 hooks: - id: check-yaml - id: check-xml

12 months

git:master:f781dc9c: pre-commit: tool to keep thing tidy [skip ci]

by Victor Seva

Module: kamailio Branch: master Commit: f781dc9ce914cc619abd9d53f987f1155557b720 URL: https://github.com/kamailio/kamailio/commit/f781dc9ce914cc619abd9d53f987f11… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2023-05-19T00:25:47+02:00 pre-commit: tool to keep thing tidy [skip ci] --- Added: .pre-commit-config.yaml --- Diff: https://github.com/kamailio/kamailio/commit/f781dc9ce914cc619abd9d53f987f11… Patch: https://github.com/kamailio/kamailio/commit/f781dc9ce914cc619abd9d53f987f11… --- diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml new file mode 100644 index 00000000000..f9af7b878e1 --- /dev/null +++ b/.pre-commit-config.yaml @@ -0,0 +1,13 @@ +repos: + - repo: https://github.com/pre-commit/pre-commit-hooks + rev: v2.3.0 + hooks: + - id: check-yaml + - id: check-xml + - id: end-of-file-fixer + - id: trailing-whitespace + - id: check-merge-conflict + - repo: https://github.com/pre-commit/mirrors-clang-format + rev: v15.0.7 + hooks: + - id: clang-format

12 months

git:master:2cac52c6: README: remove travis-ci info [skip ci]

by Victor Seva

Module: kamailio Branch: master Commit: 2cac52c68feeacc1dc90e4ff322b6109c32669e8 URL: https://github.com/kamailio/kamailio/commit/2cac52c68feeacc1dc90e4ff322b610… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2023-05-19T00:25:47+02:00 README: remove travis-ci info [skip ci] no longer used --- Modified: README.md --- Diff: https://github.com/kamailio/kamailio/commit/2cac52c68feeacc1dc90e4ff322b610… Patch: https://github.com/kamailio/kamailio/commit/2cac52c68feeacc1dc90e4ff322b610… --- diff --git a/README.md b/README.md index f0a904a0831..aff44050c6a 100644 --- a/README.md +++ b/README.md @@ -121,9 +121,4 @@ News: * https://www.kamailio.org/w/category/news/ * Twitter @kamailio -## Travis-CI - Testing Build Environment - - * [travis-ci](https://travis-ci.org/kamailio/kamailio/builds/) - * [docker build](/test/travis/README.md) - **Thank you for flying Kamailio!**

12 months

Re: clang-format changes

by Juha Heinanen

I guess I missed the clang-format discussion or did it happen? Fr example, I would have voted for SpaceBeforeParens: ControlStatements -- Juha

12 months

[kamailio/kamailio] presence: Add publ_cache sync RPC command. (PR #3460)

by Jake Greene

#### 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 Needed a way to sync the publ_cache with changes made to the presentity table when using publ_cache modes 1 or 2. This seems pretty simple, and works for me. Beneficial when adding numerous "message-summary" event entries from an external Voicemail application to the 'presentity' table. Usage: `presence.htable_db_restore` You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3460 -- Commit Summary -- * presence: Add publ_cache sync RPC command. -- File Changes -- M src/modules/presence/doc/presence_admin.xml (21) M src/modules/presence/presence.c (14) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3460.patch https://github.com/kamailio/kamailio/pull/3460.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3460 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3460(a)github.com>

12 months

clang-format changes

by Victor Seva

Hi, I'm thinking we should start enforcing clang-format in order to have a coherent indentation and coding style in the project. I've added a check in the pull-request Github Action to check the clang-format in the commits but for sure now it will almost always fail due to our code is not always formatted properly. Is is OK if I start committing changes module by module of just changes related to clang-format formatting? I'm aware this will our life harder to backport fixes to supported branches but I think it will help us in the long run. Cheers -- ----------------------------------------------------------------- | ,''`. Victor Seva | | : :' : linuxmaniac(a)torreviejawireless.org | | `. `' PGP Key ID: 0x51A09B18CF5A5068 | | `- Debian Developer | -----------------------------------------------------------------

12 months

git:master:61833193: dispatcher: fix force send socket functionality, was not working in some TCP scenarios

by Henning Westerholt

Module: kamailio Branch: master Commit: 6183319381573e42b882d05ae1748539f7547d8c URL: https://github.com/kamailio/kamailio/commit/6183319381573e42b882d05ae174853… Author: Henning Westerholt <hw(a)gilawa.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2023-05-18T10:52:52Z dispatcher: fix force send socket functionality, was not working in some TCP scenarios --- Modified: src/modules/dispatcher/dispatch.c --- Diff: https://github.com/kamailio/kamailio/commit/6183319381573e42b882d05ae174853… Patch: https://github.com/kamailio/kamailio/commit/6183319381573e42b882d05ae174853… --- diff --git a/src/modules/dispatcher/dispatch.c b/src/modules/dispatcher/dispatch.c index a07f779ab8..f0322a6b57 100644 --- a/src/modules/dispatcher/dispatch.c +++ b/src/modules/dispatcher/dispatch.c @@ -1869,7 +1869,8 @@ static inline int ds_push_dst( break; } if(sock) { - msg->force_send_socket = sock; + LM_DBG("set send socket to %.*s\n", sock->sock_str.len, sock->sock_str.s); + set_force_socket(msg, sock); } return 0; }

12 months

← Newer
1
...
13
14
15
16
17
18
19
...
32
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev May 2023