Summary:ASTERISK-03709: [patch] SUBSCRIBE causes "Internal Server Error" on Polycom
Reporter:kuj (kuj)Labels:
Date Opened:2005-03-19 12:54:12.000-0600Date Closed:2008-01-15 15:29:05.000-0600
Versions:Frequency of
Environment:Attachments:( 0) sip_xpidf.txt
( 1) subscribe.txt
Description:A Polycom 500 subscribing to buddy status of other extensions would generate a "500 Internal Server Error" upon receiving the xpidf NOTIFY message from *.


From googling around: For the the xpidf format, ""... The priority tag contains an integer that indicates the relative
preference of this address over other addresses. It is a floating-point value between 0 and 1, with 1 being the highest preference."

Changing the priority tag to use "dotted" notation instead of a comma in the float value, I got rid of the server error. (Have sip debug traces if needed)

(I doubt this is related to ASTERISK-3708, even though that one also generates an internal server error)

DISCLAIMER on file :)
Comments:By: Kevin P. Fleming (kpfleming) 2005-03-19 13:26:18.000-0600

This also seems obviously correct.

By: kuj (kuj) 2005-03-19 13:36:40.000-0600

Darn, that wasn't it. The 500 Internal Server Error reappeared. See the attached subscribe.txt

By: kuj (kuj) 2005-03-19 13:46:04.000-0600

However, this together with ASTERISK-3711 seems to be the cure.

By: kuj (kuj) 2005-03-19 19:55:50.000-0600

One or all of the other SIP changes happening today seem to have fixed my Internal Server Error issue. Not sure whether this phone can parse a 'priority="0,800000"' tag correctly or if it simply ignores it.

While changing the comma to a decimal point seems to make sense, this one can be closed.

By: Mark Spencer (markster) 2005-03-19 20:54:53.000-0600

Fixed in CVS head.  Thanks!

By: Russell Bryant (russell) 2005-03-28 00:47:41.000-0600

fixed in 1.0

By: Digium Subversion (svnbot) 2008-01-15 15:29:05.000-0600

Repository: asterisk
Revision: 5288

U   branches/v1-0/channels/chan_sip.c

r5288 | russell | 2008-01-15 15:29:05 -0600 (Tue, 15 Jan 2008) | 2 lines

change ',' to '.' (bug ASTERISK-3709)