Re: [sr-dev] [kamailio/kamailio] dialog: fix dlg_onroute (#2878)
This doesn't seem like a right approach. Anyhow, commit d5c1b46 should fix initializing the sip message structure after applying changes. If still crashes, open an issue, the call-id should be parsed, not set to a static value. -- 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/2878#issuecomment-942549262___ Kamailio (SER) - Development Mailing List sr-dev@lists.kamailio.org https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
Re: [sr-dev] [kamailio/kamailio] dialog: fix dlg_onroute (#2878)
Closed #2878. -- 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/2878#event-5458776045___ Kamailio (SER) - Development Mailing List sr-dev@lists.kamailio.org https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
Re: [sr-dev] [kamailio/kamailio] core: fixup sip_msg_apply_changes (#2877)
Closed #2877. -- 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/2877#event-5458766955___ Kamailio (SER) - Development Mailing List sr-dev@lists.kamailio.org https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
Re: [sr-dev] [kamailio/kamailio] core: fixup sip_msg_apply_changes (#2877)
Thanks for the PR! However I pushed a different commit (d5c1b46) to have the same behaviours as when the sip message is received. Kamailio does not parse all the headers before config is executed due to internal designed, so modules have to check if needed headers are parsed, but Call-Id is indeed parsed before config execution. If there is still an issue after with commit d5c1b46, then open an issue. If works, then I will backport. -- 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/2877#issuecomment-942547745___ Kamailio (SER) - Development Mailing List sr-dev@lists.kamailio.org https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
[sr-dev] git:master:d5c1b469: core: parse main headers after updating msg buffer
Module: kamailio
Branch: master
Commit: d5c1b4693b6e47b4123b5f9417b5fd88d4dafd1f
URL:
https://github.com/kamailio/kamailio/commit/d5c1b4693b6e47b4123b5f9417b5fd88d4dafd1f
Author: Daniel-Constantin Mierla
Committer: Daniel-Constantin Mierla
Date: 2021-10-13T19:16:06+02:00
core: parse main headers after updating msg buffer
- same group of headers as for the case of receiving the message
- applies for msg_apply_changes() as well
- related to GH #2877
---
Modified: src/core/msg_translator.c
---
Diff:
https://github.com/kamailio/kamailio/commit/d5c1b4693b6e47b4123b5f9417b5fd88d4dafd1f.diff
Patch:
https://github.com/kamailio/kamailio/commit/d5c1b4693b6e47b4123b5f9417b5fd88d4dafd1f.patch
---
diff --git a/src/core/msg_translator.c b/src/core/msg_translator.c
index 9fd5635655..bc1b9736df 100644
--- a/src/core/msg_translator.c
+++ b/src/core/msg_translator.c
@@ -3328,7 +3328,13 @@ int sip_msg_update_buffer(sip_msg_t *msg, str *obuf)
* valid/safe for config */
return 0;
}
-
+ if(parse_headers(msg, HDR_FROM_F|HDR_TO_F|HDR_CALLID_F|HDR_CSEQ_F, 0) <
0) {
+ LM_ERR("parsing main headers of new sip message failed
[[%.*s]]\n",
+ msg->len, msg->buf);
+ /* exit config execution - sip_msg_t structure is no longer
+* valid/safe for config */
+ return 0;
+ }
return 1;
}
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
[sr-dev] [kamailio/kamailio] There is no jwt.so (#2883)
### Description ``` apt install kamailio kamailio-* find /usr/lib/x86_64-linux-gnu/kamailio/modules -name jwt.so and nothing. ``` ### Troubleshooting Reproduction Debugging Data ``` (paste your debugging data here) ``` Log Messages ``` (paste your log messages here) ``` SIP Traffic ``` (paste your sip traffic here) ``` ### Possible Solutions ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` kamailio5.5.2 ``` * **Operating System**: ``` debian10 ``` -- 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/issues/2883___ Kamailio (SER) - Development Mailing List sr-dev@lists.kamailio.org https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
