sr-dev

sr-dev@lists.kamailio.org

44307 discussions

git:master: pv: fixed tobody.params transformation

by Daniel-Constantin Mierla

Module: sip-router Branch: master Commit: 6cc5f8ec90db48483d644430bc2628356833fc1e URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=6cc5f8e… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Fri Jul 30 20:15:13 2010 +0200 pv: fixed tobody.params transformation - len was wrong when having quoted parameters - patch by Alex Hermann, closes SF#3035382 --- modules_k/pv/pv_trans.c | 8 ++------ 1 files changed, 2 insertions(+), 6 deletions(-) diff --git a/modules_k/pv/pv_trans.c b/modules_k/pv/pv_trans.c index 5d60d04..e568ddb 100644 --- a/modules_k/pv/pv_trans.c +++ b/modules_k/pv/pv_trans.c @@ -1033,12 +1033,8 @@ int tr_eval_tobody(struct sip_msg *msg, tr_param_t *tp, int subtype, if(_tr_tobody.param_lst!=NULL) { val->rs.s = _tr_tobody.param_lst->name.s; - if(_tr_tobody.last_param->value.s!=NULL) - val->rs.len = _tr_tobody.last_param->value.s - +_tr_tobody.last_param->value.len - val->rs.s; - else - val->rs.len = _tr_tobody.last_param->name.s - +_tr_tobody.last_param->name.len - val->rs.s; + val->rs.len = _tr_tobody_str.s + _tr_tobody_str.len + - val->rs.s; } else val->rs = _tr_empty; break;

14 years, 6 months

[ openser-Patches-1769555 ] DISPATCH : weighted dispatching

by SourceForge.net

Patches item #1769555, was opened at 2007-08-07 20:29 Message generated for change (Comment added) made by miconda You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743022&aid=1769555&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 devel >Status: Closed >Resolution: Fixed Priority: 2 Private: No Submitted By: Emmanuel BUU (neutrino38) Assigned to: Daniel-Constantin Mierla (miconda) Summary: DISPATCH : weighted dispatching Initial Comment: The proposed patch enhances the DISPATCH modules. It enables the specification of weight for each destination. For instance if destination.list contains 1 proxy1:5060 50 1 proxy2:5060 20 1 proxy3;5060 30 50% of calls will be dispatched to proxy1, 20% of calls will be disptached to proxy2 30% to proxy3 ---------------------------------------------------------------------- >Comment By: Daniel-Constantin Mierla (miconda) Date: 2010-07-30 20:51 Message: Git master branch has weight based load balancing in dispatcher module. ---------------------------------------------------------------------- Comment By: Emmanuel BUU (neutrino38) Date: 2007-10-05 17:59 Message: Logged In: YES user_id=1862563 Originator: YES As you wish ---------------------------------------------------------------------- Comment By: Daniel-Constantin Mierla (miconda) Date: 2007-10-05 17:34 Message: Logged In: YES user_id=1246013 Originator: NO Not being sure that this enhancement will preserve old behavior when having same weight for all destination, I postpone the integration for the next release, as carrierroute module satisfy the proposed enhancement. ---------------------------------------------------------------------- Comment By: Emmanuel BUU (neutrino38) Date: 2007-08-30 01:07 Message: Logged In: YES user_id=1862563 Originator: YES Ok I submitted a patch based on trunk version. It compiles but it is not tested yet although it is based on the same algorithm than the previous one. I updated the DB schema as well. Note: I discovered the existence of the CARRIERROUTE module that basically makes much more than this modified DISPATCH. THis is great. Emmanuel BUU http://www.ives.fr/ ---------------------------------------------------------------------- Comment By: Daniel-Constantin Mierla (miconda) Date: 2007-08-10 10:20 Message: Logged In: YES user_id=1246013 Originator: NO Please provide the patch against devel version, there is support for storing destination list in database as well. Thanks. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743022&aid=1769555&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 10:04 Message generated for change (Comment added) made by miconda 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: Closed >Resolution: Accepted Priority: 5 Private: No Submitted By: Alex Hermann (axlh) >Assigned to: Daniel-Constantin Mierla (miconda) 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. ---------------------------------------------------------------------- >Comment By: Daniel-Constantin Mierla (miconda) Date: 2010-07-30 20:48 Message: Patch committed on git master branch and svn 1.5. Thanks! ---------------------------------------------------------------------- Comment By: Iñaki Baz Castillo (ibc_sf) Date: 2010-07-30 12:31 Message: Reopened as it's clear now the purpose of this patch. ---------------------------------------------------------------------- Comment By: Alex Hermann (axlh) Date: 2010-07-30 11:59 Message: Sorry, the patch is already a bit old and i got the description wrong. The patch is actually an addendum to bug 2851214. It makes sure kamailio doesn't return someone else's contact in case there is a failure. The contacts are stored in a global variable and needs to be cleared if it isn't filled with valid contacts. Please reopen. ---------------------------------------------------------------------- Comment By: Iñaki Baz Castillo (ibc_sf) Date: 2010-07-28 13:29 Message: A "Contact" header with no value is *not* valid according to SIP BNF grammar so in case a UA fetches its bindging and there are not bindings then the registrar should reply a 200 without "Contact" header. Taking this into account I suspect that this patch is wrong. As a side note, there is a bug/issue open for RFC 3261 related to this topic as the RFC 3261 seems to state that the registrar should reply a 200 with a Contact header containing all the bindings. But it's clear (also in the RFC bug report) that the BNF grammar cannot be violated so there should be NO Contact header if there are no bindings. I close this bug as it not valid. However as Daniel said please use http://sip-router.org/tracker/ for future reports :) ---------------------------------------------------------------------- Comment By: Daniel-Constantin Mierla (miconda) Date: 2010-07-28 10:07 Message: Can you use the tracker at sip-router.org? http://sip-router.org/tracker/ SF tracker should not be used anymore for new items. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743022&aid=3035778&group_…

14 years, 6 months

SF.net SVN: openser:[6035] branches/1.5/modules

by Daniel-Constantin Mierla

Revision: 6035 http://openser.svn.sourceforge.net/openser/?rev=6035&view=rev Author: miconda Date: 2010-07-30 17:47:03 +0000 (Fri, 30 Jul 2010) Log Message: ----------- - backport git commit 776d94 - when an UA fetches the bindings for a user and the user isn't registered, properly return an empty contact. - closes SF#3035778, patch by Alex Hermann Modified Paths: -------------- branches/1.5/modules/registrar/save.c Property Changed: ---------------- branches/1.5/modules/pv/pv_shv.c branches/1.5/modules/pv/pv_shv.h branches/1.5/modules/pv/pv_svar.c branches/1.5/modules/pv/pv_svar.h branches/1.5/modules/pv/pv_trans.h This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.

14 years, 6 months

git:master: registrar(k): reset local contact holder

by Daniel-Constantin Mierla

Module: sip-router Branch: master Commit: 776d941bef2e97089493a896bf0b57607078c52d URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=776d941… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Fri Jul 30 19:40:50 2010 +0200 registrar(k): reset local contact holder - when an UA fetches the bindings for a user and the user isn't registered, properly return an empty contact. - closes SF#3035778, patch by Alex Hermann --- modules_k/registrar/save.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/modules_k/registrar/save.c b/modules_k/registrar/save.c index d22def0..9955034 100644 --- a/modules_k/registrar/save.c +++ b/modules_k/registrar/save.c @@ -457,6 +457,8 @@ static inline int insert_contacts(struct sip_msg* _m, contact_t* _c, if (r->contacts) build_contact(r->contacts); ul.release_urecord(r); + } else { /* No contacts found */ + build_contact(NULL); } #ifdef USE_TCP

14 years, 6 months

git:master: presence_xml: xml PV moved to xmlops module

by Daniel-Constantin Mierla

Module: sip-router Branch: master Commit: 5d5f342267e6ca70f2af7722e6179fd19ec78945 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=5d5f342… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Thu Jul 29 20:23:19 2010 +0200 presence_xml: xml PV moved to xmlops module --- modules_k/presence_xml/README | 36 +- modules_k/presence_xml/doc/presence_xml_admin.xml | 15 +- modules_k/presence_xml/presence_xml.c | 9 +- modules_k/presence_xml/pv_xml.c | 550 --------------------- modules_k/presence_xml/pv_xml.h | 38 -- 5 files changed, 13 insertions(+), 635 deletions(-) Diff: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=5d5…

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 (Comment added) made by ibc_sf 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. ---------------------------------------------------------------------- >Comment By: Iñaki Baz Castillo (ibc_sf) Date: 2010-07-30 11:31 Message: Reopened as it's clear now the purpose of this patch. ---------------------------------------------------------------------- Comment By: Alex Hermann (axlh) Date: 2010-07-30 10:59 Message: Sorry, the patch is already a bit old and i got the description wrong. The patch is actually an addendum to bug 2851214. It makes sure kamailio doesn't return someone else's contact in case there is a failure. The contacts are stored in a global variable and needs to be cleared if it isn't filled with valid contacts. Please reopen. ---------------------------------------------------------------------- Comment By: Iñaki Baz Castillo (ibc_sf) Date: 2010-07-28 12:29 Message: A "Contact" header with no value is *not* valid according to SIP BNF grammar so in case a UA fetches its bindging and there are not bindings then the registrar should reply a 200 without "Contact" header. Taking this into account I suspect that this patch is wrong. As a side note, there is a bug/issue open for RFC 3261 related to this topic as the RFC 3261 seems to state that the registrar should reply a 200 with a Contact header containing all the bindings. But it's clear (also in the RFC bug report) that the BNF grammar cannot be violated so there should be NO Contact header if there are no bindings. I close this bug as it not valid. However as Daniel said please use http://sip-router.org/tracker/ for future reports :) ---------------------------------------------------------------------- Comment By: Daniel-Constantin Mierla (miconda) Date: 2010-07-28 09:07 Message: Can you use the tracker at sip-router.org? http://sip-router.org/tracker/ SF tracker should not be used anymore for new items. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743022&aid=3035778&group_…

14 years, 6 months

Cleaning up my patch queue

by Alex Hermann

Hello all, i'm cleaning my patch queue and have pushed all patched that might be relevant to others to the tracker on sourceforge. To prevent mishaps like my previous commit, i would like an ACK or NACK on each patch before i commit them. These are all meant for 1.5, but most of them apply to 3.0 too. Thanks, -- Alex Hermann SpeakUp BV

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 (Comment added) 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: Closed Resolution: Invalid 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. ---------------------------------------------------------------------- Comment By: Alex Hermann (axlh) Date: 2010-07-30 10:59 Message: Sorry, the patch is already a bit old and i got the description wrong. The patch is actually an addendum to bug 2851214. It makes sure kamailio doesn't return someone else's contact in case there is a failure. The contacts are stored in a global variable and needs to be cleared if it isn't filled with valid contacts. Please reopen. ---------------------------------------------------------------------- Comment By: Iñaki Baz Castillo (ibc_sf) Date: 2010-07-28 12:29 Message: A "Contact" header with no value is *not* valid according to SIP BNF grammar so in case a UA fetches its bindging and there are not bindings then the registrar should reply a 200 without "Contact" header. Taking this into account I suspect that this patch is wrong. As a side note, there is a bug/issue open for RFC 3261 related to this topic as the RFC 3261 seems to state that the registrar should reply a 200 with a Contact header containing all the bindings. But it's clear (also in the RFC bug report) that the BNF grammar cannot be violated so there should be NO Contact header if there are no bindings. I close this bug as it not valid. However as Daniel said please use http://sip-router.org/tracker/ for future reports :) ---------------------------------------------------------------------- Comment By: Daniel-Constantin Mierla (miconda) Date: 2010-07-28 09:07 Message: Can you use the tracker at sip-router.org? http://sip-router.org/tracker/ SF tracker should not be used anymore for new items. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743022&aid=3035778&group_…

14 years, 6 months

git:master: xmlops: new module for xml operations

by Daniel-Constantin Mierla

Module: sip-router Branch: master Commit: e9438550e076de32fc0ebd51c296efba099d82c2 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=e943855… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Thu Jul 29 20:37:50 2010 +0200 xmlops: new module for xml operations - implements $xml(name=>spec) pseudo-variable (previously was in presence_xml) - new parameter buf_size to control the max size of internal buffer for xml document management --- modules/xmlops/Makefile | 19 ++ modules/xmlops/README | 80 ++++++ modules/xmlops/doc/Makefile | 4 + modules/xmlops/doc/functions.xml | 29 ++ modules/xmlops/doc/params.xml | 41 +++ modules/xmlops/doc/xmlops.xml | 66 +++++ modules/xmlops/pv_xml.c | 550 ++++++++++++++++++++++++++++++++++++++ modules/xmlops/pv_xml.h | 38 +++ modules/xmlops/xmlops_mod.c | 74 +++++ 9 files changed, 901 insertions(+), 0 deletions(-) Diff: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=e94…

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