sr-dev

sr-dev@lists.kamailio.org

17 participants
45431 discussions

[kamailio/kamailio] Invalid $nghttp2(...) (Issue #4299)
by QWERTY 26 Jun '25

26 Jun '25

2954456878 created an issue (kamailio/kamailio#4299) version: kamailio 6.0.1 (x86_64/linux) loadmodule "nghttp2.so" modparam("nghttp2", "listen_addr", "0.0.0.0") modparam("nghttp2", "listen_port", "8363") event_route[nghttp2:request] { xinfo("request: $nghttp2(method) - url: $nghttp2(url) - data: [$nghttp2(data)]\n"); nghttp2_reply("200", "<html><body>OK</body></html>"); } Startup error： Jun 25 10:42:50 debiancrs /usr/local/kamailio6-devel/sbin/kamailio[41907]: ERROR: nghttp2 [nghttp2_mod.c:246]: pv_parse_nghttp2_name(): invalid variable name [url] Jun 25 10:42:50 debiancrs /usr/local/kamailio6-devel/sbin/kamailio[41907]: ERROR: <core> [core/pvapi.c:954]: pv_parse_spec2(): pvar "nghttp2" has an invalid name param [url] Jun 25 10:42:50 debiancrs /usr/local/kamailio6-devel/sbin/kamailio[41907]: ERROR: <core> [core/pvapi.c:1092]: pv_parse_spec2(): wrong char [)/41] in [$nghttp2(url)] at [12 (5)] Jun 25 10:42:50 debiancrs /usr/local/kamailio6-devel/sbin/kamailio[41907]: ERROR: xlog [xlog.c:550]: xdbg_fixup_helper(): wrong format[request: $nghttp2(method) - url: $nghttp2(url) - data: [$nghttp2(data)]#012] -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4299 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4299(a)github.com>

2 2

git:master:64bc28f1: modules: readme files regenerated - nghttp2 ... [skip ci]
by Kamailio Dev 26 Jun '25

26 Jun '25

Module: kamailio Branch: master Commit: 64bc28f1279809829b714d23314b3e62030a8ccb URL: https://github.com/kamailio/kamailio/commit/64bc28f1279809829b714d23314b3e6… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2025-06-26T09:31:10+02:00 modules: readme files regenerated - nghttp2 ... [skip ci] --- Modified: src/modules/nghttp2/README --- Diff: https://github.com/kamailio/kamailio/commit/64bc28f1279809829b714d23314b3e6… Patch: https://github.com/kamailio/kamailio/commit/64bc28f1279809829b714d23314b3e6… --- diff --git a/src/modules/nghttp2/README b/src/modules/nghttp2/README index f880b2e5a13..fc79a0fbae3 100644 --- a/src/modules/nghttp2/README +++ b/src/modules/nghttp2/README @@ -184,15 +184,15 @@ event_route[nghttp2:request] { The event route is executed when a new HTTP request is received. Inside it, the $nghttp2(...) group of variables is available, giving - access to several attributes of the HTTP request, such as method, URL, - data (body) or headers. + access to several attributes of the HTTP request, such as method, URL + path, data (body) or headers. ... ... loadmodule "nghttp2.so ... event_route[nghttp2:request] { - xinfo("request: $nghttp2(method) - url: $nghttp2(url) - data: [$nghttp2(data -)]\n"); + xinfo("request: $nghttp2(method) - url: $nghttp2(path) - data: [$nghttp2(dat +a)]\n"); nghttp2_reply("200", "OK", "text/html", "<html><body>OK</body></html>"); }

1 0

git:master:b513666f: nghttp2: docs updated to use the proper $nghttp2(path) variable
by Daniel-Constantin Mierla 26 Jun '25

26 Jun '25

Module: kamailio Branch: master Commit: b513666f76d2220d8914fadcc4968845efbf170f URL: https://github.com/kamailio/kamailio/commit/b513666f76d2220d8914fadcc496884… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-26T09:05:10+02:00 nghttp2: docs updated to use the proper $nghttp2(path) variable --- Modified: src/modules/nghttp2/doc/nghttp2_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/b513666f76d2220d8914fadcc496884… Patch: https://github.com/kamailio/kamailio/commit/b513666f76d2220d8914fadcc496884… --- diff --git a/src/modules/nghttp2/doc/nghttp2_admin.xml b/src/modules/nghttp2/doc/nghttp2_admin.xml index 54cde446de6..a09bb5d0b33 100644 --- a/src/modules/nghttp2/doc/nghttp2_admin.xml +++ b/src/modules/nghttp2/doc/nghttp2_admin.xml @@ -182,7 +182,7 @@ event_route[nghttp2:request] { <para> Inside it, the $nghttp2(...) group of variables is available, giving access to several attributes of the HTTP request, such as method, - URL, data (body) or headers. + URL path, data (body) or headers. </para> <programlisting format="linespecific"> ... @@ -190,7 +190,7 @@ event_route[nghttp2:request] { loadmodule "nghttp2.so ... event_route[nghttp2:request] { - xinfo("request: $nghttp2(method) - url: $nghttp2(url) - data: [$nghttp2(data)]\n"); + xinfo("request: $nghttp2(method) - url: $nghttp2(path) - data: [$nghttp2(data)]\n"); nghttp2_reply("200", "OK", "text/html", "<html><body>OK</body></html>"); }

1 0

git:master:092ad634: core: print ip and port in log message
by Daniel-Constantin Mierla 25 Jun '25

25 Jun '25

Module: kamailio Branch: master Commit: 092ad634a2741ad21fd29586a30bd2bcb9ac653b URL: https://github.com/kamailio/kamailio/commit/092ad634a2741ad21fd29586a30bd2b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-25T20:34:23+02:00 core: print ip and port in log message --- Modified: src/core/msg_translator.c --- Diff: https://github.com/kamailio/kamailio/commit/092ad634a2741ad21fd29586a30bd2b… Patch: https://github.com/kamailio/kamailio/commit/092ad634a2741ad21fd29586a30bd2b… --- diff --git a/src/core/msg_translator.c b/src/core/msg_translator.c index 3a46f7106f5..e27b70deec1 100644 --- a/src/core/msg_translator.c +++ b/src/core/msg_translator.c @@ -2918,7 +2918,7 @@ char *via_builder(unsigned int *len, sip_msg_t *msg, #endif /* USE_COMP */ int port; int proto; - struct ip_addr ip; + struct ip_addr ip = {0}; union sockaddr_union *from = NULL; union sockaddr_union local_addr; struct tcp_connection *con = NULL; @@ -3064,8 +3064,8 @@ char *via_builder(unsigned int *len, sip_msg_t *msg, if(con == NULL) { LM_WARN("TCP/TLS connection (id: %d) for WebSocket could not be " - "found\n", - send_info->id); + "found - likely it is gone (dst: [%s]:%d)\n", + send_info->id, (port) ? ip_addr2a(&ip) : "", port); pkg_free(line_buf); return 0; }

1 0

[kamailio/kamailio] Release 6.0.2 - 6.0.2
by github-actions[bot] 25 Jun '25

25 Jun '25

-- View it on GitHub: https://github.com/kamailio/kamailio/releases/tag/6.0.2 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/releases/227792045(a)github.com>

1 0

git:master:3e3cfc6b: github: alpine docker image, removed cleanup
by Sergey Safarov 25 Jun '25

25 Jun '25

Module: kamailio Branch: master Commit: 3e3cfc6b113b3e9a7f1a3ac91897687fbb71f29a URL: https://github.com/kamailio/kamailio/commit/3e3cfc6b113b3e9a7f1a3ac91897687… Author: Sergey Safarov <s.safarov(a)anycast-lb.net> Committer: Sergey Safarov <s.safarov(a)anycast-lb.net> Date: 2025-06-25T18:46:07+03:00 github: alpine docker image, removed cleanup --- Modified: .github/workflows/alpine.yml --- Diff: https://github.com/kamailio/kamailio/commit/3e3cfc6b113b3e9a7f1a3ac91897687… Patch: https://github.com/kamailio/kamailio/commit/3e3cfc6b113b3e9a7f1a3ac91897687… --- diff --git a/.github/workflows/alpine.yml b/.github/workflows/alpine.yml index e5f7885e898..82fc4082199 100644 --- a/.github/workflows/alpine.yml +++ b/.github/workflows/alpine.yml @@ -426,18 +426,3 @@ jobs: context: "context" build-args: | IMAGE=${{ needs.check.outputs.alpine_tag }} - - cleanup: - name: Cleanup untaged images - runs-on: ubuntu-latest - continue-on-error: true - permissions: - packages: write - needs: - - images - steps: - - name: Cleanup untaged - uses: dataaxiom/ghcr-cleanup-action@v1 - with: - owner: ${{ github.repository_owner }} - package: ${{ env.IMAGE_NAME }}

1 0

git:master:db653681: dispatcher: clone dispatcher uri for event route variable
by Daniel-Constantin Mierla 25 Jun '25

25 Jun '25

Module: kamailio Branch: master Commit: db65368108b0de14e40cbbd7c92561e5c8740dac URL: https://github.com/kamailio/kamailio/commit/db65368108b0de14e40cbbd7c92561e… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-25T17:03:58+02:00 dispatcher: clone dispatcher uri for event route variable --- Modified: src/modules/dispatcher/dispatch.c --- Diff: https://github.com/kamailio/kamailio/commit/db65368108b0de14e40cbbd7c92561e… Patch: https://github.com/kamailio/kamailio/commit/db65368108b0de14e40cbbd7c92561e… --- diff --git a/src/modules/dispatcher/dispatch.c b/src/modules/dispatcher/dispatch.c index cb7ccf44685..4a5c2d90cf8 100644 --- a/src/modules/dispatcher/dispatch.c +++ b/src/modules/dispatcher/dispatch.c @@ -143,6 +143,7 @@ static int ds_strictest_match = 0; #define _ds_list (ds_lists[*ds_crt_idx]) #define _ds_list_nr (*ds_list_nr) +void ds_rctx_set_uri(ds_rctx_t *rctx, str *uri); static void ds_run_route( struct sip_msg *msg, str *uri, char *route, ds_rctx_t *rctx); @@ -3045,7 +3046,7 @@ int ds_mark_addr(sip_msg_t *msg, int state, int group, str *uri, int mode) rctx.code = 800; } rctx.setid = group; - rctx.uri = *uri; + ds_rctx_set_uri(&rctx, uri); ret = ds_update_state(msg, group, uri, state, mode, &rctx); @@ -3478,6 +3479,28 @@ int ds_update_state(sip_msg_t *msg, int group, str *address, int state, * */ static ds_rctx_t *_ds_rctx = NULL; +static char _ds_rctx_buri[MAX_URI_SIZE]; + +/** + * + */ +void ds_rctx_set_uri(ds_rctx_t *rctx, str *uri) +{ + _ds_rctx_buri[0] = '\0'; + rctx->uri.s = _ds_rctx_buri; + rctx->uri.len = 0; + if(uri == NULL || uri->s == NULL || uri->len < 0) { + return; + } + if(uri->len >= MAX_URI_SIZE - 1) { + LM_ERR("uri too long: %d\n", uri->len); + return; + } + memcpy(_ds_rctx_buri, uri->s, uri->len); + _ds_rctx_buri[uri->len] = '\0'; + rctx->uri.len = uri->len; + return; +} /** * @@ -4073,7 +4096,7 @@ static void ds_options_callback( } } rctx.setid = group; - rctx.uri = uri; + ds_rctx_set_uri(&rctx, &uri); /* Check if in the meantime someone disabled probing of the target * through RPC or reload */ @@ -4237,7 +4260,7 @@ void ds_ping_set(ds_set_t *node) rctx.reason.s = "Sending keepalive failed"; rctx.reason.len = 24; rctx.setid = node->id; - rctx.uri = node->dlist[j].uri; + ds_rctx_set_uri(&rctx, &node->dlist[j].uri); /* check if meantime someone disabled the target via RPC */ if(!(node->dlist[j].flags & DS_DISABLED_DST) && ds_update_state(NULL, node->id, &node->dlist[j].uri,

1 0

git:master:0e914414: dispatcher: $dsv(name) extended to return setid and uri
by Daniel-Constantin Mierla 25 Jun '25

25 Jun '25

Module: kamailio Branch: master Commit: 0e914414e48637168ab9ab682a9463412cf8f3fe URL: https://github.com/kamailio/kamailio/commit/0e914414e48637168ab9ab682a94634… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-25T13:22:17+02:00 dispatcher: $dsv(name) extended to return setid and uri --- Modified: src/modules/dispatcher/dispatch.c Modified: src/modules/dispatcher/dispatch.h Modified: src/modules/dispatcher/dispatcher.c --- Diff: https://github.com/kamailio/kamailio/commit/0e914414e48637168ab9ab682a94634… Patch: https://github.com/kamailio/kamailio/commit/0e914414e48637168ab9ab682a94634… --- diff --git a/src/modules/dispatcher/dispatch.c b/src/modules/dispatcher/dispatch.c index d484c40233c..cb7ccf44685 100644 --- a/src/modules/dispatcher/dispatch.c +++ b/src/modules/dispatcher/dispatch.c @@ -3044,6 +3044,9 @@ int ds_mark_addr(sip_msg_t *msg, int state, int group, str *uri, int mode) } else { rctx.code = 800; } + rctx.setid = group; + rctx.uri = *uri; + ret = ds_update_state(msg, group, uri, state, mode, &rctx); LM_DBG("state [%d] grp [%d] dst [%.*s]\n", state, group, uri->len, uri->s); @@ -4069,6 +4072,8 @@ static void ds_options_callback( rctx.reason = ps->rpl->first_line.u.reply.reason; } } + rctx.setid = group; + rctx.uri = uri; /* Check if in the meantime someone disabled probing of the target * through RPC or reload */ @@ -4231,6 +4236,8 @@ void ds_ping_set(ds_set_t *node) rctx.code = 500; rctx.reason.s = "Sending keepalive failed"; rctx.reason.len = 24; + rctx.setid = node->id; + rctx.uri = node->dlist[j].uri; /* check if meantime someone disabled the target via RPC */ if(!(node->dlist[j].flags & DS_DISABLED_DST) && ds_update_state(NULL, node->id, &node->dlist[j].uri, diff --git a/src/modules/dispatcher/dispatch.h b/src/modules/dispatcher/dispatch.h index c47ad4fbf5e..ffaab7cb7e8 100644 --- a/src/modules/dispatcher/dispatch.h +++ b/src/modules/dispatcher/dispatch.h @@ -90,6 +90,8 @@ typedef struct ds_rctx int flags; int code; str reason; + str uri; + int setid; } ds_rctx_t; extern str ds_db_url; diff --git a/src/modules/dispatcher/dispatcher.c b/src/modules/dispatcher/dispatcher.c index 00960311ead..35cfd3cc4a5 100644 --- a/src/modules/dispatcher/dispatcher.c +++ b/src/modules/dispatcher/dispatcher.c @@ -1392,6 +1392,13 @@ static int pv_get_dsv(sip_msg_t *msg, pv_param_t *param, pv_value_t *res) return pv_get_null(msg, param, res); case 2: return pv_get_sintval(msg, param, res, rctx->flags); + case 3: + if(rctx->uri.s != NULL && rctx->uri.len > 0) { + return pv_get_strval(msg, param, res, &rctx->uri); + } + return pv_get_null(msg, param, res); + case 4: + return pv_get_sintval(msg, param, res, rctx->setid); default: return pv_get_null(msg, param, res); } @@ -1406,6 +1413,12 @@ static int pv_parse_dsv(pv_spec_p sp, str *in) return -1; switch(in->len) { + case 3: + if(strncmp(in->s, "uri", 3) == 0) + sp->pvp.pvn.u.isname.name.n = 3; + else + goto error; + break; case 4: if(strncmp(in->s, "code", 4) == 0) sp->pvp.pvn.u.isname.name.n = 0; @@ -1415,6 +1428,10 @@ static int pv_parse_dsv(pv_spec_p sp, str *in) case 5: if(strncmp(in->s, "flags", 5) == 0) sp->pvp.pvn.u.isname.name.n = 2; + else if(strncmp(in->s, "setid", 5) == 0) + sp->pvp.pvn.u.isname.name.n = 4; + else if(strncmp(in->s, "group", 5) == 0) + sp->pvp.pvn.u.isname.name.n = 4; else goto error; break;

1 0

Kamailio v6.0.2 Released
by Daniel-Constantin Mierla 25 Jun '25

25 Jun '25

Hello, Kamailio SIP Server v6.0.2 stable release is out. This is a maintenance release of the latest stable branch, 6.0, that includes fixes since the release of v6.0.2. There is no change to database schema or configuration language structure that you have to do on previous installations of v6.0.x. Deployments running previous v6.0.x versions are strongly recommended to be upgraded to v6.0.2. For more details about version 6.0.2 (including links and guidelines to download the tarball or from GIT repository), visit: * https://www.kamailio.org/w/2025/06/kamailio-v6-0-2-released/ RPM, Debian/Ubuntu packages will be available soon as well. Many thanks to all contributing and using Kamailio! Cheers, Daniel -- Daniel-Constantin Mierla (@ asipto.com) twitter.com/miconda -- linkedin.com/in/miconda Kamailio Consultancy, Training and Development Services -- asipto.com

1 0

git:6.0:d4dc5dc1: ChangeLog: content updated for v6.0.2
by Daniel-Constantin Mierla 25 Jun '25

25 Jun '25

Module: kamailio Branch: 6.0 Commit: d4dc5dc14c62c7a016a3840d46f00064f926506b URL: https://github.com/kamailio/kamailio/commit/d4dc5dc14c62c7a016a3840d46f0006… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-25T10:35:30+02:00 ChangeLog: content updated for v6.0.2 --- Modified: ChangeLog --- Diff: https://github.com/kamailio/kamailio/commit/d4dc5dc14c62c7a016a3840d46f0006… Patch: https://github.com/kamailio/kamailio/commit/d4dc5dc14c62c7a016a3840d46f0006…

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