sr-dev September 2009

sr-dev@lists.kamailio.org

32 participants
207 discussions

siprouter.org domain
by Greger Viken Teigre 19 Sep '09

19 Sep '09

I just realized today that I have the siprouter.org domain (for pre-historic reasons that I don't even remember). It didn't even have proper DNS, but I have set that up and pointed www. to the same ip as www (but the server does not recognize the siprouter.org domain). Or maybe I should use CNAME? Who is the keeper of DNS for sip-router.org? Let's sync up so at least siprouter.org redirects correctly the most important addresses. g-)

2 2

git:master: xmlrpc doc: use set_reply_close()
by Andrei Pelinescu-Onciul 18 Sep '09

18 Sep '09

Module: sip-router Branch: master Commit: b2ba3a96b33e4bac14a9de6b615665af9f598c78 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=b2ba3a9… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Fri Sep 18 23:56:08 2009 +0200 xmlrpc doc: use set_reply_close() Updated doc & examples to use set_reply_close() and set_reply_no_connect() in the XMLRPC route instead of return -1. (this allows … [View More]using rpc async commands with the broken python xmlrpclib). --- modules_s/xmlrpc/README | 19 ++++++++++++++++--- modules_s/xmlrpc/doc/functions.xml | 6 ++++++ modules_s/xmlrpc/doc/xmlrpc.xml | 14 +++++++++++--- 3 files changed, 33 insertions(+), 6 deletions(-) diff --git a/modules_s/xmlrpc/README b/modules_s/xmlrpc/README index b497e66..d1abb54 100644 --- a/modules_s/xmlrpc/README +++ b/modules_s/xmlrpc/README @@ -456,14 +456,21 @@ Content-Length: 276 xmlrpclib with a Transport class that works. For the second solution (force closing tcp connections after answering) - the XMLRPC route should end with a return -1, exit -1 or drop -1. + the XMLRPC route should have a set_reply_close() command before + dispatch_rpc(). set_reply_no_connect() is also recommended (avoid + trying to open tcp connection to xmlrpc clients that closed it). + Alternatively ending the XMLRPC route with return -1, exit -1 or drop + -1 can also be used, but note that this will not work for async rpcs + (it will close the connection immeidately and not on the async + response). Example 1. route[XMLRPC]{ - dispatch_rpc(); # close connection only for xmlrpclib user agents if search("^User-Agent:.*xmlrpclib")) - return -1 ; + set_reply_close(); + set_reply_no_connect(); # optional + dispatch_rpc(); } 1.4. Client Examples @@ -543,6 +550,9 @@ modparam("xmlrpc", "autoconversion", 1) modparam("xmlrpc", "route", "XMLRPC"); #... route[XMLRPC]{ + if search("^User-Agent:.*xmlrpclib")) + set_reply_close(); + set_reply_no_connect(); # optional dispatch_rpc(); } @@ -566,5 +576,8 @@ route[XMLRPC]{ xmlrpc_reply("400", "Unauthorized"); return; } + if search("^User-Agent:.*xmlrpclib")) + set_reply_close(); + set_reply_no_connect(); # optional dispatch_rpc(); } diff --git a/modules_s/xmlrpc/doc/functions.xml b/modules_s/xmlrpc/doc/functions.xml index bc3820f..adacbac 100644 --- a/modules_s/xmlrpc/doc/functions.xml +++ b/modules_s/xmlrpc/doc/functions.xml @@ -43,6 +43,9 @@ modparam("xmlrpc", "route", "XMLRPC"); #... route[XMLRPC]{ + if search("^User-Agent:.*xmlrpclib")) + set_reply_close(); + set_reply_no_connect(); # optional dispatch_rpc(); } </programlisting> @@ -74,6 +77,9 @@ route[XMLRPC]{ xmlrpc_reply("400", "Unauthorized"); return; } + if search("^User-Agent:.*xmlrpclib")) + set_reply_close(); + set_reply_no_connect(); # optional dispatch_rpc(); } </programlisting> diff --git a/modules_s/xmlrpc/doc/xmlrpc.xml b/modules_s/xmlrpc/doc/xmlrpc.xml index c893e97..2cdf422 100644 --- a/modules_s/xmlrpc/doc/xmlrpc.xml +++ b/modules_s/xmlrpc/doc/xmlrpc.xml @@ -677,15 +677,23 @@ Content-Length: 276 </para> <para> For the second solution (force closing tcp connections after answering) - the XMLRPC route should end with a return -1, exit -1 or drop -1. + the XMLRPC route should have a <function>set_reply_close()</function> + command before <function>dispatch_rpc()</function>. + <function>set_reply_no_connect()</function> is also recommended + (avoid trying to open tcp connection to xmlrpc clients that closed it). + Alternatively ending the XMLRPC route with return -1, exit -1 or + drop -1 can also be used, but note that this will not work for + async rpcs (it will close the connection immeidately and not on the + async response). <example> <programlisting> <![CDATA[ route[XMLRPC]{ - dispatch_rpc(); # close connection only for xmlrpclib user agents if search("^User-Agent:.*xmlrpclib")) - return -1 ; + set_reply_close(); + set_reply_no_connect(); # optional + dispatch_rpc(); } ]]> </programlisting> [View Less]

1 0

git:master: core: send_flags support in the onsend_route
by Andrei Pelinescu-Onciul 18 Sep '09

18 Sep '09

Module: sip-router Branch: master Commit: 8ed9165103cfe93172ee18904a2f210674edfdce URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=8ed9165… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Fri Sep 18 20:58:20 2009 +0200 core: send_flags support in the onsend_route send_flags can now be set also in the onsend_route (via set_forward_no_connect() or set_forward_close()). E.g.: … [View More]onsend_route{ if (dsp_ip!=10.0.0.0/8) set_forward_no_connect(); # don't open new connection, # only reuse } --- cfg.y | 4 ++++ onsend.h | 12 ++++++++++++ 2 files changed, 16 insertions(+), 0 deletions(-) diff --git a/cfg.y b/cfg.y index af5335c..528a70a 100644 --- a/cfg.y +++ b/cfg.y @@ -2053,6 +2053,10 @@ fcmd: case ISFLAGSET_T: case IF_T: case MODULE_T: + case SET_FWD_NO_CONNECT_T: + case SET_RPL_NO_CONNECT_T: + case SET_FWD_CLOSE_T: + case SET_RPL_CLOSE_T: $$=$1; break; default: diff --git a/onsend.h b/onsend.h index 433100d..deec874 100644 --- a/onsend.h +++ b/onsend.h @@ -55,6 +55,7 @@ extern struct onsend_info* p_onsend; /* * returns: 0 drop the message, >= ok, <0 error (but forward the message) + * it also migh change dst->send_flags! * WARNING: buf must be 0 terminated (to allow regex matches on it) */ static inline int run_onsend(struct sip_msg* orig_msg, struct dest_info* dst, char* buf, int len) @@ -63,6 +64,8 @@ static inline int run_onsend(struct sip_msg* orig_msg, struct dest_info* dst, int ret; struct run_act_ctx ra_ctx; int backup_route_type; + snd_flags_t fwd_snd_flags_bak; + snd_flags_t rpl_snd_flags_bak; ret=1; if (onsend_rt.rlist[DEFAULT_RT]){ @@ -74,8 +77,17 @@ static inline int run_onsend(struct sip_msg* orig_msg, struct dest_info* dst, backup_route_type=get_route_type(); set_route_type(ONSEND_ROUTE); if (exec_pre_script_cb(orig_msg, ONSEND_CB_TYPE)>0) { + /* backup orig_msg send flags */ + fwd_snd_flags_bak=orig_msg->fwd_send_flags; + rpl_snd_flags_bak=orig_msg->rpl_send_flags; + orig_msg->fwd_send_flags=dst->send_flags; /* intial value */ init_run_actions_ctx(&ra_ctx); ret=run_actions(&ra_ctx, onsend_rt.rlist[DEFAULT_RT], orig_msg); + /* update dst send_flags */ + dst->send_flags=orig_msg->fwd_send_flags; + /* restore orig_msg flags */ + orig_msg->fwd_send_flags=fwd_snd_flags_bak; + orig_msg->rpl_send_flags=rpl_snd_flags_bak; exec_post_script_cb(orig_msg, ONSEND_CB_TYPE); } else { ret=0; /* drop the message */ [View Less]

1 0

git:master: tm: send_flags can be used in branch & reply routes
by Andrei Pelinescu-Onciul 18 Sep '09

18 Sep '09

Module: sip-router Branch: master Commit: cc256eba86a81f61ead4e00c39d81858b5e5e56c URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=cc256eb… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Fri Sep 18 21:03:26 2009 +0200 tm: send_flags can be used in branch & reply routes Forwarding send flags can now be used in branch routes (and they will affect only the current branch) and in … [View More]reply routes (if the reply will be forwarded, either as provisional or final reply, the send_flags will affect it, but they will not affect other replies). --- modules/tm/t_fwd.c | 30 +++++++++++++++++++++++++++++- modules/tm/t_reply.c | 5 ++++- 2 files changed, 33 insertions(+), 2 deletions(-) diff --git a/modules/tm/t_fwd.c b/modules/tm/t_fwd.c index 0590c10..2ef3025 100644 --- a/modules/tm/t_fwd.c +++ b/modules/tm/t_fwd.c @@ -144,6 +144,23 @@ unsigned int get_on_branch(void) } +/** create a branch "buffer". + * Creates the buffer used in the branch rb and runs the on_branch route. + * @param t - transaction + * @param i_req - corresponding sip_msg, must be non-null, flags might be + * be modified (on_branch route) + * @param branch - branch no + * @param uri + * @param path - path vector (list of route like destination in text form, + * e.g.: "<sip:1.2.3.4;lr>, <sip:5.6.7.8;lr>") + * @param len - resul parameter, it will be filled with the created buffer + * lenght. + * @param dst - value/result parameter. It will be used to generate the + * message. All the values except the send_flags are read-only. + * send_flags it's updated. + * @return pointer to shm alloc'ed buffer on success (*len and dst->send_flags + * are filled/modified), 0 on failure + */ static char *print_uac_request( struct cell *t, struct sip_msg *i_req, int branch, str *uri, str* path, unsigned int *len, struct dest_info* dst) { @@ -156,6 +173,8 @@ static char *print_uac_request( struct cell *t, struct sip_msg *i_req, str path_bak; int path_backed_up; int backup_route_type; + snd_flags_t fwd_snd_flags_bak; + snd_flags_t rpl_snd_flags_bak; shbuf=0; msg_uri_bak.s=0; /* kill warnings */ @@ -208,11 +227,20 @@ static char *print_uac_request( struct cell *t, struct sip_msg *i_req, set_route_type(BRANCH_ROUTE); tm_ctx_set_branch_index(branch+1); if (exec_pre_script_cb(i_req, BRANCH_CB_TYPE)>0) { + /* backup ireq msg send flags */ + fwd_snd_flags_bak=i_req->fwd_send_flags;; + rpl_snd_flags_bak=i_req->rpl_send_flags; + i_req->fwd_send_flags=dst->send_flags /* intial value */; if (run_top_route(branch_rt.rlist[branch_route], i_req, 0) < 0) { LOG(L_ERR, "Error in run_top_route\n"); } + /* update dst send_flags */ + dst->send_flags=i_req->fwd_send_flags; + /* restore ireq_msg flags */ + i_req->fwd_send_flags=fwd_snd_flags_bak; + i_req->rpl_send_flags=rpl_snd_flags_bak; exec_post_script_cb(i_req, BRANCH_CB_TYPE); - } + } tm_ctx_set_branch_index(0); set_route_type(backup_route_type); } diff --git a/modules/tm/t_reply.c b/modules/tm/t_reply.c index 981efd2..fa99650 100644 --- a/modules/tm/t_reply.c +++ b/modules/tm/t_reply.c @@ -1607,6 +1607,9 @@ enum rps relay_reply( struct cell *t, struct sip_msg *p_msg, int branch, if (branch!=relay) { free_via_clen_lump(&relayed_msg->add_rm); } + /* update send_flags with possible additions from the + reply route */ + uas_rb->dst.send_flags|=relayed_msg->rpl_send_flags; } } update_reply_stats( relayed_code ); @@ -1978,7 +1981,7 @@ int reply_received( struct sip_msg *p_msg ) backup_domain_to = set_avp_list(AVP_TRACK_TO | AVP_CLASS_DOMAIN, &t->domain_avps_to ); setbflagsval(0, uac->branch_flags); /* Pre- and post-script callbacks have already - * been execueted by the core. (Miklos) + * been executed by the core. (Miklos) */ if (run_top_route(onreply_rt.rlist[t->on_reply], p_msg, 0)<0) LOG(L_ERR, "ERROR: on_reply processing failed\n"); [View Less]

1 0

git:master: tcp: fix fd leak on async error or quick connect
by Andrei Pelinescu-Onciul 18 Sep '09

18 Sep '09

Module: sip-router Branch: master Commit: b8d528b657bd52fce0b342e01380001b4ffa15fe URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=b8d528b… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Fri Sep 18 20:52:37 2009 +0200 tcp: fix fd leak on async error or quick connect In some error cases (from which the most likely to appear in real-life would be maximum global send queue size exceeded)… [View More]

1 0

git:master: Merge commit 'origin/andrei/path'
by Andrei Pelinescu-Onciul 18 Sep '09

18 Sep '09

Module: sip-router Branch: master Commit: 0fa4b2d5a4d9700eed970d36ed1bb803c1f07d21 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=0fa4b2d… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Fri Sep 18 13:36:33 2009 +0200 Merge commit 'origin/andrei/path' * commit 'origin/andrei/path': core: add path route headers after local via core: path support when forwarding tm: path support … [View More]

1 0

git:master: Merge branch 'rpc_async'
by Andrei Pelinescu-Onciul 18 Sep '09

18 Sep '09

Module: sip-router Branch: master Commit: 5b1cd719918a51ca8f4b4a05dbc34e27fd1ae58c URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=5b1cd71… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Fri Sep 18 13:16:26 2009 +0200 Merge branch 'rpc_async' * rpc_async: tm: async t_uac (t_uac_wait) support xmlrpc(s): basic support for delayed replies core: rpc capabilities and delayed reply … [View More]

1 0

git:master: Merge commit 'origin/andrei/send_flags'
by Andrei Pelinescu-Onciul 18 Sep '09

18 Sep '09

Module: sip-router Branch: master Commit: 643491da0b3cb9a87d278c08ed5e0879511ddfc1 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=643491d… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Fri Sep 18 12:57:42 2009 +0200 Merge commit 'origin/andrei/send_flags' * commit 'origin/andrei/send_flags': NEWS: update core: new close after send and reuse only script functions tcp: … [View More]

1 0

git:master: Merge commit 'origin/andrei/to_parser_fix'
by Andrei Pelinescu-Onciul 18 Sep '09

18 Sep '09

Module: sip-router Branch: master Commit: 43d869ccd2ae997338e4009aa760862589feca3d URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=43d869c… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Fri Sep 18 12:54:50 2009 +0200 Merge commit 'origin/andrei/to_parser_fix' * commit 'origin/andrei/to_parser_fix': core: to parser fix ---

1 0

tags
by Andrei Pelinescu-Onciul 18 Sep '09

18 Sep '09

I've tried tagging the current code version, before doing some merges, but by mistake I've pushed more tags back to the repository. They look like old ser tags which somehow did not make it in the intial import. There's no harm done, just some extra tags. The commit messages that appear as being on a tag, where already on the repo, they get extra reported probably because of a minor problem with the script generating the emails. Andrei

1 0

← Newer
1
...
8
9
10
11
12
13
14
...
21
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev September 2009