THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.
The following task is now closed:
FS#407 - Debian init script log_faiulre_msg
User who did this - Daniel-Constantin Mierla (miconda)
Reason for closing: Fixed
Additional comments about closing: Committed by Ovidiu Sas, e22293d865926a200e336293692886f983904275
More information can be found at the following URL:
http://sip-router.org/tracker/index.php?do=details&task_id=407
You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.
Hi
We have found that in kamailio processing if you:
1. set msg->path_vec
2. call msg_apply_changes()
3. t_telay
The route-header of the outgoing message is set twice.
We think this is because msg_apply_changes() adds the router header if the
path is set - but leaves the path set. So the router header is added again
when you t_relay.
To fix this:
--- a/modules/textopsx/textopsx.c
+++ b/modules/textopsx/textopsx.c
@@ -166,7 +166,7 @@ static int msg_apply_changes_f(sip_msg_t *msg, char
*str1, char *str2)
} else {
obuf.s = build_req_buf_from_sip_req(msg,
(unsigned int*)&obuf.len, &dst,
- BUILD_NO_LOCAL_VIA|BUILD_NO_VIA1_UPDATE);
+
BUILD_NO_PATH|BUILD_NO_LOCAL_VIA|BUILD_NO_VIA1_UPDATE);
}
if(obuf.s == NULL)
{
This stops msg_apply_changes() from adding the router header if path is set.
Any objections to comitting this to master? Might other functionality be
effected?
The alternative is for msg_apply_changes() to add the router header but
then unset the path.
Regards
Richard.
--
This email is subject to the disclaimer of Smile Communications at http://www.smilecoms.com/home/email-disclaimer/ <http://www.smilecoms.com/disclaimer>
THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.
A new Flyspray task has been opened. Details are below.
User who did this - Corey Edwards (tensai)
Attached to Project - sip-router
Summary - Debian init script log_faiulre_msg
Task Type - Bug Report
Category - tools/scripts
Status - Unconfirmed
Assigned To -
Operating System - Linux
Severity - Low
Priority - Normal
Reported Version - Development
Due in Version - Undecided
Due Date - Undecided
Details - The Debian init scripts have a typo in the check_kamailio_config subroutine. "log_failure_msg" is misspelled as "log_faiulre_msg".
Attached is a patch against git master to correct this.
One or more files have been attached.
More information can be found at the following URL:
http://sip-router.org/tracker/index.php?do=details&task_id=407
You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.
THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.
The following task has a new comment added:
FS#406 - Crash in websocket module
User who did this - Vitaliy Aleksandrov (Vitaliy)
----------
In the attachment a patch that add reference counting to websocket connections.
Modified version is working right now on my ws gateway without any visible issues.
Can somebody look through it to check if the way i chose and its implementation are right ?
At now it has lots of debug messages and commented code. I'll prepare a clean version after more testing and approving by someone from the kamailio community.
----------
One or more files have been attached.
More information can be found at the following URL:
http://sip-router.org/tracker/index.php?do=details&task_id=406#comment1340
You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.