sr-dev

sr-dev@lists.kamailio.org

44307 discussions

[ openser-Patches-3035804 ] Multiple return values for force_rtp_proxy

by SourceForge.net

Patches item #3035804, was opened at 2010-07-28 10:03 Message generated for change (Tracker Item Submitted) made by axlh You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743022&aid=3035804&group_… Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: modules Group: ver 1.5.x Status: Open Resolution: None Priority: 5 Private: No Submitted By: Alex Hermann (axlh) Assigned to: Nobody/Anonymous (nobody) Summary: Multiple return values for force_rtp_proxy Initial Comment: Add a separate return value (-2) when an rtp proxy has already mangled the packet. makes it possible to determine in the script if an rtpproxy is in the audio path. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743022&aid=3035804&group_…

14 years, 6 months

[ openser-Patches-3035802 ] Add locking to pua for serializing PUBLISHes

by SourceForge.net

Patches item #3035802, was opened at 2010-07-28 09:58 Message generated for change (Tracker Item Submitted) made by axlh You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743022&aid=3035802&group_… Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: modules Group: ver 1.5.x Status: Open Resolution: None Priority: 5 Private: No Submitted By: Alex Hermann (axlh) Assigned to: Nobody/Anonymous (nobody) Summary: Add locking to pua for serializing PUBLISHes Initial Comment: pua must not send multiple PUBLISH with the same id before receiving a reply as the ETag must be updated on subsequent publishes. Patch by Anca Vamanu. Stolen from opensips r5924. Original commit message: introduced a locking mechanism to prevent sending a new Publish before the reply for a previous one is received (thanks to Piotr Sobolewski <sobolewski at gmail dot com> for help with debugging) ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743022&aid=3035802&group_…

14 years, 6 months

[ openser-Patches-3035794 ] Add option to keep subscriptions on timed out NOTIFY

by SourceForge.net

Patches item #3035794, was opened at 2010-07-28 09:33 Message generated for change (Tracker Item Submitted) made by axlh You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743022&aid=3035794&group_… Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: modules Group: ver 1.5.x Status: Open Resolution: None Priority: 5 Private: No Submitted By: Alex Hermann (axlh) Assigned to: Nobody/Anonymous (nobody) Summary: Add option to keep subscriptions on timed out NOTIFY Initial Comment: This parameter is a flag that should be set if subscriptions should be removed from the active_watchers when a NOTIFY times out. RFC3265 section 3.2.2 defines this behaviour as a SHOULD, so by default it is on. Disabling this will keep subscriptions active on unreliable networks. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743022&aid=3035794&group_…

14 years, 6 months

[ openser-Patches-3035783 ] Add route headers to CANCEL

by SourceForge.net

Patches item #3035783, was opened at 2010-07-28 09:14 Message generated for change (Tracker Item Submitted) made by axlh You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743022&aid=3035783&group_… Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: modules Group: ver 1.5.x Status: Open Resolution: None Priority: 5 Private: No Submitted By: Alex Hermann (axlh) Assigned to: Nobody/Anonymous (nobody) Summary: Add route headers to CANCEL Initial Comment: When an INVITE is sent which had route headers, the CANCEL must have them too. This happens when the registrar sends an INVITE to a contact that registered with a Path header. The Path becomes a Route header. Currently, if that transaction is canceled, the CANCEL doesn't have this Route header. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743022&aid=3035783&group_…

14 years, 6 months

[ openser-Patches-3035780 ] Let pua use the configured outboudn proxy in all cases

by SourceForge.net

Patches item #3035780, was opened at 2010-07-28 09:07 Message generated for change (Tracker Item Submitted) made by axlh You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743022&aid=3035780&group_… Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: modules Group: ver 1.5.x Status: Open Resolution: None Priority: 5 Private: No Submitted By: Alex Hermann (axlh) Assigned to: Nobody/Anonymous (nobody) Summary: Let pua use the configured outboudn proxy in all cases Initial Comment: pua wasn't using the outbound proxy on updates ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743022&aid=3035780&group_…

14 years, 6 months

[ openser-Patches-3035778 ] Fix registrar when fetching binding for unregistered user

by SourceForge.net

Patches item #3035778, was opened at 2010-07-28 09:04 Message generated for change (Tracker Item Submitted) made by axlh You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743022&aid=3035778&group_… Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: modules Group: ver 1.5.x Status: Open Resolution: None Priority: 5 Private: No Submitted By: Alex Hermann (axlh) Assigned to: Nobody/Anonymous (nobody) Summary: Fix registrar when fetching binding for unregistered user Initial Comment: When an UA fetches the bindings for a user and the user isn't registered, properly return an empty contact. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743022&aid=3035778&group_…

14 years, 6 months

[ openser-Patches-3035382 ] Fix transformation tobody.params

by SourceForge.net

Patches item #3035382, was opened at 2010-07-27 18:11 Message generated for change (Comment added) made by axlh You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743022&aid=3035382&group_… Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: core Group: ver 1.5.x Status: Open Resolution: None Priority: 5 Private: No Submitted By: Alex Hermann (axlh) Assigned to: Nobody/Anonymous (nobody) Summary: Fix transformation tobody.params Initial Comment: The transofrmation tobody.params can't handle quoted parameters well. If the last parameter is quoted, the output of the transofrmation is truncated by 2 characters.. Attached patch fixes this. ---------------------------------------------------------------------- >Comment By: Alex Hermann (axlh) Date: 2010-07-28 08:42 Message: Well, not a good start :( I split the patch into 2 parts. Part 1: Fix a off-by-one bug in parsing quoted parameters. The parameter value is shortened by 1 character. Part 2: Fix the tobody.params tranformation when the last paremeter has a quoted value. I found another solution: Just assume the parameter list ends on the end of the header line that gets pushed to the parser. Seems to work just as well, and prevents adjusting the rest of the code (tm). ---------------------------------------------------------------------- Comment By: Daniel-Constantin Mierla (miconda) Date: 2010-07-27 22:25 Message: I reverted the commit on 1.5 branch - it was incomplete as the compilation got broken. Also, there should not be any commit on the trunk branch, the devel version is now on GIT repository hosted at git.sip-router.org. The bug tracker in use is as well by sip-router.org: http://sip-router.org/tracker Back to issue, apart of breaking the compilation, it could be a solution to store the hook for start of parameters and the length, but I didn't understand why you changed the len of quoted value. Moreover, there are other places that needs to be updated when the to_body structure is updated with new attributes storing pointers, like cloning for TM transactions. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743022&aid=3035382&group_…

14 years, 6 months

git:master: core: new param -A to add cfg preprocesor directives

by Daniel-Constantin Mierla

Module: sip-router Branch: master Commit: 95dec431421ce9f809f3a15f4c6407fff2d401a7 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=95dec43… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Jul 27 23:46:48 2010 +0200 core: new param -A to add cfg preprocesor directives - defines can be specified from command line with option -A - these defines are available from begining of parsing the config - examples: - '-A WITH_AUTH' is like having '#!define WITH_AUTH' at top of cfg - '-A LISTENIP=127.0.0.1' is like '#!define LISTENIP=127.0.0.1' --- cfg.lex | 6 ++---- main.c | 28 ++++++++++++++++++++++++++-- ppcfg.h | 3 +++ 3 files changed, 31 insertions(+), 6 deletions(-) diff --git a/cfg.lex b/cfg.lex index 564cb56..804b9e6 100644 --- a/cfg.lex +++ b/cfg.lex @@ -166,8 +166,6 @@ struct sr_yy_fname *next; } *sr_yy_fname_list = 0; - static int pp_define(int len, const char * text); - static int pp_define_set(int len, char * text); static str *pp_define_get(int len, const char * text); static int pp_ifdef_type(int pos); static void pp_ifdef_var(int len, const char * text); @@ -1614,7 +1612,7 @@ static int pp_lookup(int len, const char * text) return -1; } -static int pp_define(int len, const char * text) +int pp_define(int len, const char * text) { if (pp_num_defines == MAX_DEFINES) { LOG(L_CRIT, "ERROR: too many defines -- adjust MAX_DEFINES\n"); @@ -1636,7 +1634,7 @@ static int pp_define(int len, const char * text) return 0; } -static int pp_define_set(int len, char *text) +int pp_define_set(int len, char *text) { if(len<=0) { LOG(L_DBG, "no define value - ignoring\n"); diff --git a/main.c b/main.c index 5b0c476..cc6f137 100644 --- a/main.c +++ b/main.c @@ -183,6 +183,7 @@ #include "basex.h" /* init */ #include "pvapi_init.h" /* init */ #include "pv_core.h" /* register core pvars */ +#include "ppcfg.h" #ifdef DEBUG_DMALLOC #include <dmalloc.h> @@ -246,7 +247,8 @@ Options:\n\ -G file Create a pgid file\n\ -O nr Script optimization level (debugging option)\n\ -a mode Auto aliases mode: enable with yes or on,\n\ - disable with no or off\n" + disable with no or off\n\ + -A define Add config pre-processor define (e.g., -A WITH_AUTH)\n" #ifdef STATS " -s file File to which statistics is dumped (disabled otherwise)\n" #endif @@ -1649,6 +1651,7 @@ int main(int argc, char** argv) struct timeval tval; fd_set fds; int res; + char *p; /*init*/ time(&up_since); @@ -1667,7 +1670,7 @@ int main(int argc, char** argv) "DBG_MSG_QA enabled, ser may exit abruptly\n"); #endif - options= ":f:cm:dVhEb:l:L:n:vrRDTN:W:w:t:u:g:P:G:SQ:O:a:" + options= ":f:cm:dVhEb:l:L:n:vrRDTN:W:w:t:u:g:P:G:SQ:O:a:A:" #ifdef STATS "s:" #endif @@ -1749,6 +1752,26 @@ int main(int argc, char** argv) /* user needed for possible shm. pre-init */ user=optarg; break; + case 'A': + p = strchr(optarg, '='); + if(p) { + *p = '\0'; + } + if(pp_define(strlen(optarg), optarg)<0) { + fprintf(stderr, "error at define param: -A %s\n", + optarg); + goto error; + } + if(p) { + *p = '='; + p++; + if(pp_define_set(strlen(p), p)<0) { + fprintf(stderr, "error at define value: -A %s\n", + optarg); + goto error; + } + } + break; case 'b': case 'l': case 'n': @@ -1864,6 +1887,7 @@ try_again: case 'V': case 'h': case 'O': + case 'A': break; case 'E': log_stderr=1; // use in both getopt switches diff --git a/ppcfg.h b/ppcfg.h index c8ff4fa..c3a01f4 100644 --- a/ppcfg.h +++ b/ppcfg.h @@ -25,6 +25,9 @@ int pp_subst_add(char *data); int pp_subst_run(char **data); +int pp_define(int len, const char * text); +int pp_define_set(int len, char * text); + #endif /*_PPCFG_H_*/ /* vi: set ts=4 sw=4 tw=79:ai:cindent: */

14 years, 6 months

git:master: core: added option '-a' to control auto aliases

by Daniel-Constantin Mierla

Module: sip-router Branch: master Commit: b01dc20a7a45e3b80b00b03889762fdff8feda8b URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=b01dc20… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Jul 27 23:20:24 2010 +0200 core: added option '-a' to control auto aliases - enable auto aliases with '-a yes' or -'-a on' - disable auto aliases with '-a no' or -'-a off' - the option overwirtes config value of global parameter auto_aliases --- main.c | 19 +++++++++++++++++-- 1 files changed, 17 insertions(+), 2 deletions(-) diff --git a/main.c b/main.c index e79d810..5b0c476 100644 --- a/main.c +++ b/main.c @@ -244,7 +244,9 @@ Options:\n\ -g gid Change gid \n\ -P file Create a pid file\n\ -G file Create a pgid file\n\ - -O nr Script optimization level (debugging option)\n" + -O nr Script optimization level (debugging option)\n\ + -a mode Auto aliases mode: enable with yes or on,\n\ + disable with no or off\n" #ifdef STATS " -s file File to which statistics is dumped (disabled otherwise)\n" #endif @@ -1665,7 +1667,7 @@ int main(int argc, char** argv) "DBG_MSG_QA enabled, ser may exit abruptly\n"); #endif - options= ":f:cm:dVhEb:l:L:n:vrRDTN:W:w:t:u:g:P:G:SQ:O:" + options= ":f:cm:dVhEb:l:L:n:vrRDTN:W:w:t:u:g:P:G:SQ:O:a:" #ifdef STATS "s:" #endif @@ -1764,6 +1766,7 @@ int main(int argc, char** argv) case 'G': case 'S': case 'Q': + case 'a': case 's': break; case '?': @@ -1977,6 +1980,18 @@ try_again: case 'G': pgid_file=optarg; break; + case 'a': + if(strcmp(optarg, "on")==0 || strcmp(optarg, "yes")==0) + sr_auto_aliases = 1; + else if(strcmp(optarg, "off")==0 || strcmp(optarg, "no")==0) + sr_auto_aliases = 0; + else { + fprintf(stderr, + "bad auto aliases parameter: %s (valid on, off, yes, no)\n", + optarg); + goto error; + } + break; case 's': #ifdef STATS stat_file=optarg;

14 years, 6 months

SF.net SVN: openser:[6029] trunk

by axlh＠users.sourceforge.net

Revision: 6029 http://openser.svn.sourceforge.net/openser/?rev=6029&view=rev Author: axlh Date: 2010-07-27 17:20:58 +0000 (Tue, 27 Jul 2010) Log Message: ----------- Fix the transformation tobody.params when the last value of the parameter list is quoted. The resulting string was truncated by 2 characters. Because there is no way to find out the length of the parameter list (trailing whitespace may be present), store it in the to_body struct. Modified Paths: -------------- trunk/modules/pv/pv_trans.c trunk/parser/parse_to.c This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.

14 years, 6 months

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev