[asterisk-bugs] [Asterisk 0018597]: [regression] Snom BLF subscription does not work

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=18597 
====================================================================== 
Reported By:                thsgmbh
Assigned To:                twilson
====================================================================== 
Project:                    Asterisk
Issue ID:                   18597
Category:                   Channels/chan_sip/Subscriptions
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
Asterisk Version:           SVN 
JIRA:                       SWP-2892 
Regression:                 Yes 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): 1.6.2 
SVN Revision (number only!): 301220 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2011-01-11 06:59 CST
Last Modified:              2011-01-12 15:24 CST
====================================================================== 
Summary:                    [regression] Snom BLF subscription does not work
Description: 
Since Asterisk Branch 1.6.2 SVN Revision 300298, our SNOM phones cannot
subscribe any more to device state changes via xml dialog-info. After
sending the subscribe request, the asterisk servers answers with 401
unauthenticated!?!

I have attached a asterisk/sip debug file...

A possible reason could be a change in chan_sip.c (svn rev. 300298):
"Don't authenticate SUBSCRIBE re-transmissions" !??

But even with NEW subscribe requests and correct authentication values in
the snom sip header, asterisk ALWAYS sends an 401 unauth message back to
the phone.
====================================================================== 

---------------------------------------------------------------------- 
 (0130432) svnbot (reporter) - 2011-01-12 15:24
 https://issues.asterisk.org/view.php?id=18597#c130432 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 301684

_U  trunk/
U   trunk/channels/chan_sip.c

------------------------------------------------------------------------
r301684 | twilson | 2011-01-12 15:24:19 -0600 (Wed, 12 Jan 2011) | 22
lines

Merged revisions 301683 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.8

................
  r301683 | twilson | 2011-01-12 15:19:48 -0600 (Wed, 12 Jan 2011) | 15
lines
  
  Merged revisions 301682 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.6.2
  
  ........
    r301682 | twilson | 2011-01-12 15:05:02 -0600 (Wed, 12 Jan 2011) | 9
lines
    
    Don't reject all SUBSCRIBE auth requests
    
    When merging another SUBSCRIBE fix from 1.4, some braces were put in
    the wrong place. This patch fixes that.
    
    (closes issue https://issues.asterisk.org/view.php?id=18597)
    Reported by: thsgmbh
  ........
................

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=301684 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2011-01-12 15:24 svnbot         Checkin                                      
2011-01-12 15:24 svnbot         Note Added: 0130432                          
======================================================================