sr-dev May 2012

sr-dev@lists.kamailio.org

29 participants
173 discussions

git:master: pkg/kamailio/fedora/f16: Fixed some broken paths in kamailio. spec

by Peter Dunkley

Module: sip-router Branch: master Commit: c467017f858ff382fb44830b2073bebc88d7345b URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=c467017… Author: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Committer: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Date: Thu May 17 23:11:52 2012 +0100 pkg/kamailio/fedora/f16: Fixed some broken paths in kamailio.spec --- pkg/kamailio/fedora/f16/kamailio.spec | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/kamailio/fedora/f16/kamailio.spec b/pkg/kamailio/fedora/f16/kamailio.spec index e69d8a3..754e918 100644 --- a/pkg/kamailio/fedora/f16/kamailio.spec +++ b/pkg/kamailio/fedora/f16/kamailio.spec @@ -347,11 +347,11 @@ make install-modules-all skip_modules="auth_identity db_cassandra iptrtpproxy\ xmlops" mkdir -p $RPM_BUILD_ROOT/%{_unitdir} -install -m644 pkg/kamailio/fedora/kamailio.service \ +install -m644 pkg/kamailio/fedora/f16/kamailio.service \ $RPM_BUILD_ROOT/%{_unitdir}/kamailio.service mkdir -p $RPM_BUILD_ROOT/%{_sysconfdir}/sysconfig -install -m644 pkg/kamailio/fedora/kamailio.sysconfig \ +install -m644 pkg/kamailio/fedora/f16/kamailio.sysconfig \ $RPM_BUILD_ROOT/%{_sysconfdir}/sysconfig/kamailio

12 years, 7 months

git:master: pkg/kamailio/fedora/f16: Added BoxGrinder appliance definition for a Fedora 16 Kamailio test machine

by Peter Dunkley

Module: sip-router Branch: master Commit: e5a1c6fc0d773d0b12286cae309566685a3eb846 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=e5a1c6f… Author: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Committer: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Date: Thu May 17 22:33:06 2012 +0100 pkg/kamailio/fedora/f16: Added BoxGrinder appliance definition for a Fedora 16 Kamailio test machine - BoxGrinder automatically creates up-to-date CentOS/Fedora/RHEL appliances to a specified configuration. See http://boxgrinder.org/ for more information. - This configuration is the minimum required to test F16 Kamailio 3.3 RPMs. - To build the appliance: 1) Install BoxGrinder: "yum install rubygem-boxgrinder-build" on Fedora 2) Run BoxGrinder: "boxgrinder-build kamailio.appl" - If you run BoxGrinder on a 32-bit host you get a 32-bit appliance. - If you run BoxGrinder on a 64-bit host you get a 64-bit appliance by default. You can create a 32-bit appliance on a 64-bit host by using the command: "setarch i386 boxgrinder-build kamailio.appl" - WARNING: This is an unsecure (no firewall, default password) and minimal appliance. It is not suitable for actual deployment but it does have all of the Kamailio Fedora RPM dependencies installed. --- pkg/kamailio/fedora/f16/kamailio.appl | 37 +++++++++++++++++++++++++++++++++ 1 files changed, 37 insertions(+), 0 deletions(-) diff --git a/pkg/kamailio/fedora/f16/kamailio.appl b/pkg/kamailio/fedora/f16/kamailio.appl new file mode 100644 index 0000000..0b88d5a --- /dev/null +++ b/pkg/kamailio/fedora/f16/kamailio.appl @@ -0,0 +1,37 @@ +name: "f16-kamailio-#BASE_ARCH#" +summary: Fedora 16 for Installing Kamailio RPMs +os: + name: fedora + version: 16 + password: kamailio +hardware: + memory: 1024 + partitions: + "/": + size: 2 +packages: + - @core + - db4 + - expat + - GeoIP + - glib + - hiredis + - json-c + - libconfuse + - libcurl + - libevent + - libpurple + - libxml2 + - mod_perl + - mono-core + - mysql-libs + - net-snmp-agent-libs + - openldap + - openssl + - pcre + - postgresql-libs + - python + - radiusclient-ng + - sqlite + - unixODBC + - zlib

12 years, 7 months

git:master: pkg/kamailio/fedora/f16: Added BoxGrinder appliance definition for a Fedora 16 Kamailio build machine

by Peter Dunkley

Module: sip-router Branch: master Commit: f3171908e3ba5a4b7d6d3154991bdb66f783ceb1 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=f317190… Author: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Committer: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Date: Thu May 17 22:25:47 2012 +0100 pkg/kamailio/fedora/f16: Added BoxGrinder appliance definition for a Fedora 16 Kamailio build machine - BoxGrinder automatically creates up-to-date CentOS/Fedora/RHEL appliances to a specified configuration. See http://boxgrinder.org/ for more information. - This configuration is the minimum required to build F16 Kamailio 3.3 RPMs. - To build the appliance: 1) Install BoxGrinder: "yum install rubygem-boxgrinder-build" on Fedora 2) Run BoxGrinder: "boxgrinder-build kamailio-build.appl" - If you run BoxGrinder on a 32-bit host you get a 32-bit appliance. - If you run BoxGrinder on a 64-bit host you get a 64-bit appliance by default. You can create a 32-bit appliance on a 64-bit host by using the command: "setarch i386 boxgrinder-build kamailio-build.appl" --- pkg/kamailio/fedora/f16/kamailio-build.appl | 44 +++++++++++++++++++++++++++ 1 files changed, 44 insertions(+), 0 deletions(-) diff --git a/pkg/kamailio/fedora/f16/kamailio-build.appl b/pkg/kamailio/fedora/f16/kamailio-build.appl new file mode 100644 index 0000000..f007485 --- /dev/null +++ b/pkg/kamailio/fedora/f16/kamailio-build.appl @@ -0,0 +1,44 @@ +name: "f16-kamailio-build-#BASE_ARCH#" +summary: Fedora 16 for Building Kamailio RPMs +os: + name: fedora + version: 16 + password: kamailio +hardware: + memory: 1024 + partitions: + "/": + size: 4 +packages: + - @core + - bison + - db4-devel + - expat-devel + - flex + - GeoIP-devel + - gcc + - glib-devel + - hiredis-devel + - json-c-devel + - libconfuse-devel + - libcurl-devel + - libevent-devel + - libpurple-devel + - libxml2-devel + - lm_sensors-devel + - lua-devel + - make + - mod_perl-devel + - mono-devel + - mysql-devel + - net-snmp-devel + - openldap-devel + - openssl-devel + - pcre-devel + - postgresql-devel + - python-devel + - radiusclient-ng-devel + - rpm-build + - sqlite-devel + - unixODBC-devel + - zlib-devel

12 years, 7 months

git:master: pkg/kamailio/fedora/f16: Updated changelog in kamailio.spec

by Peter Dunkley

Module: sip-router Branch: master Commit: 037b263f3fb95c802de40b5a560b544287c40ad9 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=037b263… Author: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Committer: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Date: Thu May 17 22:25:04 2012 +0100 pkg/kamailio/fedora/f16: Updated changelog in kamailio.spec --- pkg/kamailio/fedora/f16/kamailio.spec | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/pkg/kamailio/fedora/f16/kamailio.spec b/pkg/kamailio/fedora/f16/kamailio.spec index 273af67..e69d8a3 100644 --- a/pkg/kamailio/fedora/f16/kamailio.spec +++ b/pkg/kamailio/fedora/f16/kamailio.spec @@ -859,6 +859,8 @@ fi %changelog +* Thu May 17 2012 Peter Dunkley <peter(a)dunkley.me.uk> + - Updated rel to pre2. * Mon May 7 2012 Peter Dunkley <peter(a)dunkley.me.uk> - Changed to use systemd instead of SysV init. * Sun May 6 2012 Peter Dunkley <peter(a)dunkley.me.uk>

12 years, 7 months

git:master: pkg/kamailio/fedora: restructured so that the .spec, .service, .sysconfig files for different Fedora versions can be kept

by Peter Dunkley

Module: sip-router Branch: master Commit: 419c36a87fb522e4b33dcd31d550490b7d125ff4 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=419c36a… Author: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Committer: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Date: Thu May 17 22:08:53 2012 +0100 pkg/kamailio/fedora: restructured so that the .spec, .service, .sysconfig files for different Fedora versions can be kept - Updated the release number in kamailio.spec to pre2 --- pkg/kamailio/fedora/{ => f16}/kamailio.service | 0 pkg/kamailio/fedora/{ => f16}/kamailio.spec | 2 +- pkg/kamailio/fedora/{ => f16}/kamailio.sysconfig | 0 3 files changed, 1 insertions(+), 1 deletions(-) diff --git a/pkg/kamailio/fedora/kamailio.service b/pkg/kamailio/fedora/f16/kamailio.service similarity index 100% rename from pkg/kamailio/fedora/kamailio.service rename to pkg/kamailio/fedora/f16/kamailio.service diff --git a/pkg/kamailio/fedora/kamailio.spec b/pkg/kamailio/fedora/f16/kamailio.spec similarity index 99% rename from pkg/kamailio/fedora/kamailio.spec rename to pkg/kamailio/fedora/f16/kamailio.spec index 3f42f2b..273af67 100644 --- a/pkg/kamailio/fedora/kamailio.spec +++ b/pkg/kamailio/fedora/f16/kamailio.spec @@ -1,6 +1,6 @@ %define name kamailio %define ver 3.3.0 -%define rel pre1 +%define rel pre2 %define _sharedir %{_prefix}/share diff --git a/pkg/kamailio/fedora/kamailio.sysconfig b/pkg/kamailio/fedora/f16/kamailio.sysconfig similarity index 100% rename from pkg/kamailio/fedora/kamailio.sysconfig rename to pkg/kamailio/fedora/f16/kamailio.sysconfig

12 years, 7 months

git:master: modules_k/presence: Fixes to cleanup functions

by Peter Dunkley

Module: sip-router Branch: master Commit: 1a49037e15f13afec7524431cd2b4805134cfe20 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=1a49037… Author: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Committer: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Date: Thu May 17 15:37:17 2012 +0100 modules_k/presence: Fixes to cleanup functions - The presentity cleanup function now uses fetch_rows - Fixed a problem with array indexing (only cleaned one row) in the active_watchers cleanup function for notifier processes --- modules_k/presence/presence.c | 3 +- modules_k/presence/presence.h | 2 + modules_k/presence/presentity.c | 2 - modules_k/presence/publish.c | 249 ++++++++++++++------------------------- modules_k/presence/subscribe.c | 4 +- 5 files changed, 94 insertions(+), 166 deletions(-) Diff: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=1a4…

12 years, 7 months

git:master: modules_k/presence: More fixes to issues in notifier processes

by Peter Dunkley

Module: sip-router Branch: master Commit: c6fe62a61ce059aafb15703e94fea358fab64378 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=c6fe62a… Author: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Committer: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Date: Thu May 17 15:40:02 2012 +0100 modules_k/presence: More fixes to issues in notifier processes - Fixes to some small issues/race conditions found during soak and load testing. --- modules_k/presence/notify.c | 67 +++++++++++++++++++++++++++---------------- 1 files changed, 42 insertions(+), 25 deletions(-) diff --git a/modules_k/presence/notify.c b/modules_k/presence/notify.c index 9240244..8228af9 100644 --- a/modules_k/presence/notify.c +++ b/modules_k/presence/notify.c @@ -2435,11 +2435,11 @@ error: return ret; } -static int notifier_notify(subs_t *sub, int *updated) +static int notifier_notify(subs_t *sub, int *updated, int *end_transaction) { str *nbody = NULL; watcher_t *watchers = NULL; - int ret = -1, attempt_delete_presentities = 0; + int ret = 0, attempt_delete_presentities = 0; *updated = 0; @@ -2452,10 +2452,20 @@ static int notifier_notify(subs_t *sub, int *updated) { if (unset_watchers_updated_winfo(&sub->pres_uri) < 0) { - /* Make sure this gets tried again next time */ LM_WARN("resetting updated_winfo flags\n"); + + if (pa_dbf.abort_transaction) + { + if (pa_dbf.abort_transaction(pa_db) < 0) + { + LM_ERR("in abort_transaction\n"); + goto error; + } + } + *end_transaction = 0; + + /* Make sure this gets tried again next time */ *updated = 1; - ret = 0; goto done; } } @@ -2489,7 +2499,6 @@ static int notifier_notify(subs_t *sub, int *updated) if (sub->updated_winfo == UPDATED_TYPE && num_winfos > 0) { *updated = 1; - ret = 0; goto done; } } @@ -2501,10 +2510,20 @@ static int notifier_notify(subs_t *sub, int *updated) { if (unset_watchers_updated_winfo(&sub->pres_uri) < 0) { - /* Make sure this gets tried again next time */ LM_WARN("resetting updated_winfo flags\n"); + + if (pa_dbf.abort_transaction) + { + if (pa_dbf.abort_transaction(pa_db) < 0) + { + LM_ERR("in abort_transaction\n"); + goto error; + } + } + *end_transaction = 0; + + /* Make sure this gets tried again next time */ *updated = 1; - ret = 0; goto done; } @@ -2556,10 +2575,7 @@ static int notifier_notify(subs_t *sub, int *updated) attempt_delete_presentities = 1; } else if (!send_fast_notify) - { - ret = 0; goto done; - } } if (notify(sub, NULL, nbody, 0) < 0) @@ -2580,11 +2596,23 @@ done: } } -error: free_notify_body(nbody, sub->event); free_watcher_list(watchers); return ret; + +error: + free_notify_body(nbody, sub->event); + free_watcher_list(watchers); + + if (pa_dbf.abort_transaction) + { + if (pa_dbf.abort_transaction(pa_db) < 0) + LM_ERR("in abort_transaction\n"); + } + *end_transaction = 0; + + return -1; } int process_dialogs(int round, int presence_winfo) @@ -2604,7 +2632,6 @@ int process_dialogs(int round, int presence_winfo) int end_transaction = 0; subs_t sub; str ev_sname, winfo = str_init("presence.winfo"); - event_t parsed_event; int now = (int)time(NULL); int updated = 0; @@ -2700,7 +2727,7 @@ int process_dialogs(int round, int presence_winfo) EXTRACT_STRING(sub.to_tag, VAL_STRING(&values[to_tag_col])); EXTRACT_STRING(sub.from_tag, VAL_STRING(&values[from_tag_col])); EXTRACT_STRING(ev_sname, VAL_STRING(&values[event_col])); - sub.event = contains_event(&ev_sname, &parsed_event); + sub.event = contains_event(&ev_sname, NULL); if (sub.event == NULL) { LM_ERR("event not found and set to NULL\n"); @@ -2813,20 +2840,10 @@ int process_dialogs(int round, int presence_winfo) sub.updated = round; - if ((notify_sent = notifier_notify(&sub, &updated)) < 0) + if ((notify_sent = notifier_notify(&sub, &updated, &end_transaction)) < 0) { LM_ERR("sending NOTIFY request\n"); - if (pa_dbf.abort_transaction) - { - if (pa_dbf.abort_transaction(pa_db) < 0) - { - LM_ERR("in abort_transaction\n"); - goto error; - } - } - end_transaction = 0; - if (cleanup_missing_dialog(&sub) < 0) LM_ERR("cleaning up after error sending NOTIFY" "request\n"); @@ -2841,7 +2858,7 @@ int process_dialogs(int round, int presence_winfo) goto error; } - if (sub.expires > 0 && sub.status != TERMINATED_STATUS) + if ((sub.expires > 0 && sub.status != TERMINATED_STATUS) || updated) { if (sub.updated_winfo != cached_updated_winfo) {

12 years, 7 months

git:master: modules_k/pua: Removed some unnecessary checks from pua_db.c

by Peter Dunkley

Module: sip-router Branch: master Commit: 162c7531cc414020f533a7a934fea0870f9cd503 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=162c753… Author: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Committer: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Date: Thu May 17 15:32:19 2012 +0100 modules_k/pua: Removed some unnecessary checks from pua_db.c --- modules_k/pua/pua_db.c | 62 +++-------------------------------------------- 1 files changed, 4 insertions(+), 58 deletions(-) diff --git a/modules_k/pua/pua_db.c b/modules_k/pua/pua_db.c index 2e6c398..c064c25 100644 --- a/modules_k/pua/pua_db.c +++ b/modules_k/pua/pua_db.c @@ -922,20 +922,7 @@ int delete_record_puadb(ua_pres_t *pres) return -1; } - if (pua_dbf.affected_rows == NULL) - return 1; - - switch(pua_dbf.affected_rows(pua_db)) - { - case 0: - LM_ERR("no rows deleted\n"); - return -1; - case 1: - return 1; - default: - LM_ERR("too many rows deleted\n"); - return -1; - } + return 1; } /******************************************************************************/ @@ -1002,22 +989,7 @@ int update_record_puadb(ua_pres_t *pres, int expires, str *etag) return -1; } - if (pua_dbf.affected_rows == NULL) - { - return 0; - } - - switch(pua_dbf.affected_rows(pua_db)) - { - case 0: - LM_INFO("no rows updated\n"); - return -1; - case 1: - return 1; - default: - LM_ERR("too many rows updated\n"); - return -1; - } + return 1; } /******************************************************************************/ @@ -1302,20 +1274,7 @@ int delete_dialog_puadb(ua_pres_t *pres) return -1; } - if (pua_dbf.affected_rows == NULL) - return 1; - - switch(pua_dbf.affected_rows(pua_db)) - { - case 0: - LM_ERR("no rows deleted\n"); - return -1; - case 1: - return 1; - default: - LM_ERR("too many rows deleted\n"); - return -1; - } + return 1; } /******************************************************************************/ @@ -1388,20 +1347,7 @@ int update_dialog_puadb(ua_pres_t *pres, int expires, str *contact) return -1; } - if (pua_dbf.affected_rows == NULL) - return 1; - - switch(pua_dbf.affected_rows(pua_db)) - { - case 0: - LM_ERR("no rows updated\n"); - return -1; - case 1: - return 1; - default: - LM_ERR("too many rows updated\n"); - return -1; - } + return 1; } /******************************************************************************/

12 years, 7 months

[tracker] Comment added: ACK not in transaction when t_relay or t_check_trans (Attachment added)

by sip-router

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#229 - ACK not in transaction when t_relay or t_check_trans User who did this - Pawel Sternal (Sternik) ---------- Yes, when I set ack_report to 1, ACK is correctly matched. My fault is not to check in documentation. About "Record-Route", again my fault, because I run "record_route()" if(!$tt). When we migrate kamailio 1.5 to 3.x we notice that !$tt must be change to $tt==$null. And this time I forgot about it, with another migration, and dump on not matched ACK. Again - my fault. I tried to check another scenario A-> SER -> ASTERISK | ASTERISK -> SER -> PSTN simulation. We don't use here acc. In kamailio 1.5.2 (test_1.5.cap) I have in: 1 ACK: May 17 14:48:20 gx00 /usr/sbin/kamailio[32283]: DBG:tm:t_newtran: transaction on entrance=0xffffffff May 17 14:48:20 gx00 /usr/sbin/kamailio[32282]: DBG:core:parse_headers: flags=80 May 17 14:48:20 gx00 /usr/sbin/kamailio[32283]: DBG:core:parse_headers: flags=ffffffffffffffff May 17 14:48:20 gx00 /usr/sbin/kamailio[32283]: DBG:core:parse_headers: flags=78 May 17 14:48:20 gx00 /usr/sbin/kamailio[32283]: DBG:tm:t_lookup_request: start searching: hash=5186, isACK=1 May 17 14:48:20 gx00 /usr/sbin/kamailio[32282]: DBG:core:print_rr_body: current rr is <sip:10.0.6.20:5060;ftag=4c30f07ddedec1e3o3;lr> May 17 14:48:20 gx00 /usr/sbin/kamailio[32282]: DBG:core:print_rr_body: current rr is <sip:10.0.6.40;lr> May 17 14:48:20 gx00 /usr/sbin/kamailio[32283]: DBG:core:parse_headers: flags=38 May 17 14:48:20 gx00 /usr/sbin/kamailio[32282]: DBG:core:print_rr_body: skipping 2 route records May 17 14:48:20 gx00 /usr/sbin/kamailio[32283]: DBG:tm:t_lookup_request: RFC3261 ACK matched May 17 14:48:20 gx00 /usr/sbin/kamailio[32282]: DBG:core:print_rr_body: out rr [] May 17 14:48:20 gx00 /usr/sbin/kamailio[32283]: DBG:tm:t_lookup_request: REF_UNSAFE: after is 1 May 17 14:48:20 gx00 /usr/sbin/kamailio[32282]: DBG:core:print_rr_body: we have 2 records May 17 14:48:20 gx00 /usr/sbin/kamailio[32283]: DBG:tm:t_lookup_request: e2e proxy ACK found May 17 14:48:20 gx00 /usr/sbin/kamailio[32282]: DBG:dialog:populate_leg_info: route_set , contact sip:NodesProdIn1@10.0.6.20:5062, cseq 102 and bind_addr udp:10.0.6.20:5060 May 17 14:48:20 gx00 /usr/sbin/kamailio[32283]: DBG:tm:t_newtran: building branch for end2end ACK May 17 14:48:20 gx00 /usr/sbin/kamailio[32283]: DBG:tm:t_relay_to: forwarding ACK 2 ACK: May 17 14:48:20 gx00 /usr/sbin/kamailio[32283]: DBG:tm:t_lookup_request: start searching: hash=5186, isACK=1 May 17 14:48:20 gx00 /usr/sbin/kamailio[32282]: DBG:core:print_rr_body: current rr is <sip:10.0.6.20:5060;ftag=4c30f07ddedec1e3o3;lr> May 17 14:48:20 gx00 /usr/sbin/kamailio[32282]: DBG:core:print_rr_body: current rr is <sip:10.0.6.40;lr> May 17 14:48:20 gx00 /usr/sbin/kamailio[32283]: DBG:core:parse_headers: flags=38 May 17 14:48:20 gx00 /usr/sbin/kamailio[32282]: DBG:core:print_rr_body: skipping 2 route records May 17 14:48:20 gx00 /usr/sbin/kamailio[32283]: DBG:tm:t_lookup_request: RFC3261 ACK matched May 17 14:48:20 gx00 /usr/sbin/kamailio[32282]: DBG:core:print_rr_body: out rr [] May 17 14:48:20 gx00 /usr/sbin/kamailio[32283]: DBG:tm:t_lookup_request: REF_UNSAFE: after is 1 May 17 14:48:20 gx00 /usr/sbin/kamailio[32282]: DBG:core:print_rr_body: we have 2 records May 17 14:48:20 gx00 /usr/sbin/kamailio[32283]: DBG:tm:t_lookup_request: e2e proxy ACK found May 17 14:48:20 gx00 /usr/sbin/kamailio[32282]: DBG:dialog:populate_leg_info: route_set , contact sip:NodesProdIn1@10.0.6.20:5062, cseq 102 and bind_addr udp:10.0.6.20:5060 May 17 14:48:20 gx00 /usr/sbin/kamailio[32283]: DBG:tm:t_newtran: building branch for end2end ACK May 17 14:48:20 gx00 /usr/sbin/kamailio[32283]: DBG:tm:t_relay_to: forwarding ACK This same scenario in kamailio 3.2.2 (test_3.2.cap): 1 ACK: May 17 13:54:26 gx00 /usr/sbin/kamailio[13546]: DEBUG: tm [t_lookup.c:1379]: DEBUG: t_newtran: msg id=16134 , global msg id=16133 , T on entrance=0xffffffff May 17 13:54:26 gx00 /usr/sbin/kamailio[13546]: DEBUG: tm [t_lookup.c:527]: t_lookup_request: start searching: hash=37195, isACK=1 May 17 13:54:26 gx00 /usr/sbin/kamailio[13546]: DEBUG: tm [t_lookup.c:485]: DEBUG: RFC3261 transaction matching failed May 17 13:54:26 gx00 /usr/sbin/kamailio[13546]: DEBUG: tm [t_lookup.c:709]: DEBUG: t_lookup_request: no transaction found May 17 13:54:26 gx00 /usr/sbin/kamailio[13546]: DEBUG: tm [t_funcs.c:315]: SER: forwarding ACK statelessly 2 ACK: May 17 13:54:26 gx00 /usr/sbin/kamailio[13546]: DEBUG: tm [t_lookup.c:1379]: DEBUG: t_newtran: msg id=16134 , global msg id=16133 , T on entrance=0xffffffff May 17 13:54:26 gx00 /usr/sbin/kamailio[13546]: DEBUG: tm [t_lookup.c:527]: t_lookup_request: start searching: hash=37195, isACK=1 May 17 13:54:26 gx00 /usr/sbin/kamailio[13546]: DEBUG: tm [t_lookup.c:485]: DEBUG: RFC3261 transaction matching failed May 17 13:54:26 gx00 /usr/sbin/kamailio[13546]: DEBUG: tm [t_lookup.c:709]: DEBUG: t_lookup_request: no transaction found May 17 13:54:26 gx00 /usr/sbin/kamailio[13546]: DEBUG: tm [t_funcs.c:315]: SER: forwarding ACK statelessly :-) ---------- 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=229#comment647 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.

12 years, 7 months

[tracker] Comment added: ACK not in transaction when t_relay or t_check_trans

by sip-router

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#229 - ACK not in transaction when t_relay or t_check_trans User who did this - Daniel-Constantin Mierla (miconda) ---------- The ACK seems ok, can you get any debug info from the device not accepting the ACK? ACK comes to kamailio and it is routed to callee, based on source IP and port of 200ok, because the callee is behind nat. The code you are referring to I guess is the one related to matching ACK to an existing INVITE transaction, used for accounting purposes. But the ACK itself for a 200OK is a separate transaction, being a request within dialog and has to be routed base on Route header. In the initial description of this item you say that Record-Route is added to ACK, I could not spot that in the trace you attached. Even it would be, that is a matter of config, transaction management/matching has nothing to do with Record-Route headers. If you want to get ACK matched to INVITE transaction, set acc module to record the ACK events and try again to see if there is any difference. Can you post the trace taken when using 1.5.x? I would like to see if there is any difference. ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=229#comment646 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.

12 years, 7 months

← Newer
1
...
5
6
7
8
9
10
11
...
18
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev May 2012