THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.
The following task has a new comment added:
FS#324 - Entity URI in NOTIFY XML body
User who did this - Pawel Sternal (Sternik)
----------
Yes, you're right. I forgot to update len check.
And yes, this work with all phones which I test. Polycom seems to be rfc pedantic piece of hardware.
Module parameter would be ok because I don't know how to mark this subscription without hardcoding "polycom" as User-Agent.
----------
More information can be found at the following URL:
http://sip-router.org/tracker/index.php?do=details&task_id=324#comment998
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 - Pawel Sternal (Sternik)
Attached to Project - sip-router
Summary - Entity URI in NOTIFY XML body
Task Type - Improvement
Category - Module
Status - Unconfirmed
Assigned To -
Operating System - Linux
Severity - Low
Priority - Normal
Reported Version - 4.0
Due in Version - Undecided
Due Date - Undecided
Details - Hi. We had a small problem with polycom phones to realize blf functionality. On yealink, cisco, aastra etc works like a charm. We discover that in xml body of NOTIFY, entity uri doesn't contain a "sip:" value. In RFC 4235 however in xml body "sip:" is on every example. I've modify notify_body.c in presence_dialoginfo module to enable BLF functionality for polycom phone by adding "sip:" to entity pres and it works. I wonder it's ok?
I've attach sip trace before the change and diff of notify_body.
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=324
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#324 - Entity URI in NOTIFY XML body
User who did this - Daniel-Constantin Mierla (miconda)
----------
Does this work with all the phones? I wonder if there has to be a flag to do it per subscription dialog or maybe a module parameter to control this.
The patch is missing the update of the uri len check, not to overflow the buffer it is built in.
----------
More information can be found at the following URL:
http://sip-router.org/tracker/index.php?do=details&task_id=324#comment997
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#324 - Entity URI in NOTIFY XML body
User who did this - Pawel Sternal (Sternik)
----------
And this is a log before I've patch notify_body.c
----------
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=324#comment996
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#323 - error when calling uac_replace_from from branch route
User who did this - Andrew Pogrebennyk (marduk)
----------
restore_mode is set to auto (as it was in our config for 3.3.x) and restore_dlg is not set with 0 being the default. The config is as follows:
modparam("uac", "rr_from_store_param", "vsf")
modparam("uac", "restore_mode", "auto")
modparam("uac", "restore_passwd", "XYZ")
----------
More information can be found at the following URL:
https://sip-router.org/tracker/index.php?do=details&task_id=323#comment995
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.
Module: sip-router
Branch: master
Commit: 9a25e712529cb7aacbae8e64a2e1be4da3c9a8c3
URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=9a25e71…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: Fri Jul 12 10:22:33 2013 +0200
dispatcher: allow pv in ds_is_from_list(setid) parameter
---
modules/dispatcher/README | 4 +++-
modules/dispatcher/dispatcher.c | 10 ++++++++--
modules/dispatcher/doc/dispatcher_admin.xml | 4 +++-
3 files changed, 14 insertions(+), 4 deletions(-)
diff --git a/modules/dispatcher/README b/modules/dispatcher/README
index 4736445..9f8e73c 100644
--- a/modules/dispatcher/README
+++ b/modules/dispatcher/README
@@ -829,7 +829,9 @@ failure_route[tryagain] {
Parameter groupid is optional, when it is missing, then the matching
will be done against all addresses in all groups. Upon a match, the
variable specified by 'setid_pvname' parameter will be set to groupid
- of matching address.
+ of matching address and the attributes will be set in variable
+ specified by 'attrs_pvname'. The parameter can be an integer or a
+ variable holding an integer value.
This function can be used from ANY_ROUTE.
diff --git a/modules/dispatcher/dispatcher.c b/modules/dispatcher/dispatcher.c
index 4af6757..9cb5ed5 100644
--- a/modules/dispatcher/dispatcher.c
+++ b/modules/dispatcher/dispatcher.c
@@ -183,7 +183,7 @@ static cmd_export_t cmds[]={
{"ds_is_from_list", (cmd_function)w_ds_is_from_list0, 0,
0, 0, REQUEST_ROUTE|FAILURE_ROUTE|ONREPLY_ROUTE|BRANCH_ROUTE},
{"ds_is_from_list", (cmd_function)w_ds_is_from_list1, 1,
- fixup_uint_null, 0, ANY_ROUTE},
+ fixup_igp_null, 0, ANY_ROUTE},
{"ds_load_unset", (cmd_function)w_ds_load_unset, 0,
0, 0, ANY_ROUTE},
{"ds_load_update", (cmd_function)w_ds_load_update, 0,
@@ -807,7 +807,13 @@ static int w_ds_is_from_list0(struct sip_msg *msg, char *str1, char *str2)
static int w_ds_is_from_list1(struct sip_msg *msg, char *set, char *str2)
{
- return ds_is_from_list(msg, (int)(long)set);
+ int s;
+ if(fixup_get_ivalue(msg, (gparam_p)set, &s)!=0)
+ {
+ LM_ERR("cannot get set id value\n");
+ return -1;
+ }
+ return ds_is_from_list(msg, s);
}
static int ds_parse_reply_codes() {
diff --git a/modules/dispatcher/doc/dispatcher_admin.xml b/modules/dispatcher/doc/dispatcher_admin.xml
index 7115225..b63a523 100644
--- a/modules/dispatcher/doc/dispatcher_admin.xml
+++ b/modules/dispatcher/doc/dispatcher_admin.xml
@@ -986,7 +986,9 @@ failure_route[tryagain] {
Parameter groupid is optional, when it is missing, then the matching
will be done against all addresses in all groups. Upon a match, the
variable specified by 'setid_pvname' parameter will be set to groupid
- of matching address.
+ of matching address and the attributes will be set in variable
+ specified by 'attrs_pvname'. The parameter can be an integer or a
+ variable holding an integer value.
</para>
<para>
This function can be used from ANY_ROUTE.