Hello,
I've just removed modules_k/sl module. modules_s/sl was moved to
modules/sl and all the other modules depending on any of sl modules were
updated to use it.
The are some differences, especially looking from K sl point of view:
- statistics - S sl module had a more detailed set of stats per reply
codes (they were available only via RPC). Therefore they were exported
to K stats API instead of importing the K implementation. Now you get
counters for common replies like 100, 400, 401, 403, 500, ... The other
replies which do not mach an exact code are counted in NXX vars (e.g.,
2xx, 3xx, 4xx)
- there is a single API exported by the module, making available several
SL send reply functions, get To-tag and register callbacks -- see
modules/sl/sl.h. The callbacks framework is completely new, using new
api and more compact parameters -- they were used only in siptrace and
nat_traversal modules, so the impact was rather small
I did a commit per module for the updates in modules depending on former
sl modules in order to be easy to trace and spot issues, therefore each
developer can see very easy the changes in his/her modules.
Tests were conducted with default config and some typical cases,
practically should not be a real big impact in modules -- sl reply
functions from former APIs use distinct names but same parameters. The
implementation was adapted to fit S SL module internals.
Testing and feedback is very much appreciate.
Cheers,
Daniel
--
Daniel-Constantin Mierla
http://www.asipto.com/
THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.
The following task has a new comment added:
FS#80 - Major regression in db_postgres with FS#77 patch
User who did this - Bayan (bayan)
----------
Thanks for looking into it, that is nasty :-\.
----------
More information can be found at the following URL:
http://sip-router.org/tracker/index.php?do=details&task_id=80#comment98
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.
A user has added themself to the list of users assigned to this task.
FS#81 - patch to add method masking to xlog function
User who did this - Juha Heinanen (jh)
http://sip-router.org/tracker/index.php?do=details&task_id=81
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.
A new Flyspray task has been opened. Details are below.
User who did this - Juha Heinanen (jh)
Attached to Project - sip-router
Summary - patch to add method masking to xlog function
Task Type - Feature Request
Category - Modules kamailio
Status - Assigned
Assigned To - Daniel-Constantin Mierla
Operating System - All
Severity - Medium
Priority - Normal
Reported Version - Development
Due in Version - Undecided
Due Date - Undecided
Details - attached find a patch to modules_k/xlog module that adds possibility, via method_mask module parameter, to filter out log messages based on method of request or reply. if method_mask parameter is set, it can be inspected and changed on the fly via rpc commands.
this patch was rejected by owner of xlog module, because it adds an overhead of two machine instructions to processing of xlog script function. i'm submitting the patch to the tracker anyhow in case some other people see it useful.
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=81
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#80 - Major regression in db_postgres with FS#77 patch
User who did this - Henning Westerholt (henningw)
----------
Ok, now with some delay because of some other (also nasty) bug..
I managed to reproduce your issue, indeed the postgres behaves wrong here:
openser=> select * from presentity;
id | username | domain | event | etag | expires | received_time | body | sender
----+----------+--------+-------+------+---------+---------------+------+--------
2 | 11 | 2 | 3 | 4 | 5 | 6 | | 7
-> Jul 23 15:05:52 ca ../kamailio[3526]: my AVP body is '
(syslog stops printing because of the \0)
mysql> select * from presentity;
+----+----------+--------+-------+------+---------+---------------+------+--------+
| id | username | domain | event | etag | expires | received_time | body | sender |
+----+----------+--------+-------+------+---------+---------------+------+--------+
| 1 | 11 | 2 | 3 | 4 | 5 | 6 | | 7 |
+----+----------+--------+-------+------+---------+---------------+------+--------+
-> Jul 23 15:06:25 ca ../kamailio[3719]: my AVP body is '<null>'
Here its correct. So there is something else missing in the db_postgres module, i'll investigate further.
----------
More information can be found at the following URL:
http://sip-router.org/tracker/index.php?do=details&task_id=80#comment97
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.
Revision: 6023
http://openser.svn.sourceforge.net/openser/?rev=6023&view=rev
Author: mariuszbihlei
Date: 2010-07-23 12:29:22 +0000 (Fri, 23 Jul 2010)
Log Message:
-----------
Fixed a bug in pv transformation (tobody transformation), that caused a segmentation fault.
The bug was caused by a incorrect error handling of the parse_to() method (defined in parser/parse_to.c).
The function returns a char* (pointer to the last character parsed), in case of error it updates an error status.
On incorrect to params, a segmentation fault happened after some time.(backported from sip-router commit
e7096bf24310b71623520a3b963ddc50c111827c)
Modified Paths:
--------------
branches/1.5/modules/pv/pv_trans.c
This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.