[asterisk-commits] kpfleming: branch 1.4 r44433 - in /branches/1.4: ./ channels/chan_sip.c

[asterisk-commits at lists.digium.com]

Author: kpfleming
Date: Wed Oct  4 20:30:05 2006
New Revision: 44433

URL: http://svn.digium.com/view/asterisk?rev=44433&view=rev
Log:
Merged revisions 44432 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r44432 | kpfleming | 2006-10-04 20:27:57 -0500 (Wed, 04 Oct 2006) | 2 lines

fix Polycom presence notification again

........

Modified:
    branches/1.4/   (props changed)
    branches/1.4/channels/chan_sip.c

Propchange: branches/1.4/
------------------------------------------------------------------------------
Binary property 'branch-1.2-merged' - no diff available.

Modified: branches/1.4/channels/chan_sip.c
URL: http://svn.digium.com/view/asterisk/branches/1.4/channels/chan_sip.c?rev=44433&r1=44432&r2=44433&view=diff
==============================================================================
--- branches/1.4/channels/chan_sip.c (original)
+++ branches/1.4/channels/chan_sip.c Wed Oct  4 20:30:05 2006
@@ -13880,10 +13880,12 @@
 		if (!strcmp(event, "presence") || !strcmp(event, "dialog")) { /* Presence, RFC 3842 */
 
 			/* Header from Xten Eye-beam Accept: multipart/related, application/rlmi+xml, application/pidf+xml, application/xpidf+xml */
-			/* don't supply pidf+xml to Polycom phones until they fix their firmware to
-			   parse it properly
+			/* Polycom phones only handle xpidf+xml, even if they say they can
+			   handle pidf+xml as well
 			*/
-			if (strstr(accept, "application/pidf+xml") && !strstr(p->useragent, "Polycom")) {
+			if (strstr(p->useragent, "Polycom")) {
+				p->subscribed = XPIDF_XML;
+			} else if (strstr(accept, "application/pidf+xml")) {
  				p->subscribed = PIDF_XML;         /* RFC 3863 format */
  			} else if (strstr(accept, "application/dialog-info+xml")) {
  				p->subscribed = DIALOG_INFO_XML;