Summary: | ASTERISK-16701: [patch] asterisk could not register to asterisk with pedantic=yes | ||
Reporter: | Stefan Schmidt (schmidts) | Labels: | |
Date Opened: | 2010-09-20 09:28:55 | Date Closed: | 2010-09-20 16:35:48 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Channels/chan_sip/Registration |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) issues_18017_v1.diff ( 1) pedanticlog ( 2) pedanticsip.conf | |
Description: | After Beta5 sip pedantic check is active by default, so i have seen that registering from one asterisk (tried 1.6.2svn, 1.8b5 and trunk) to another will not work if pedantic is active. ****** ADDITIONAL INFORMATION ****** sip.conf of both servers is attached sip debug, verbose and debug level 15 is attached | ||
Comments: | By: David Vossel (dvossel) 2010-09-20 10:02:18 It looks to me like the REGISTER request containing the authentication credentials after the first 401 message isn't being matched correctly. By: Stefan Schmidt (schmidts) 2010-09-20 12:05:05 i havent looked deep enough into this part of code, but my feeling tolds me it could something have to do with the ao2 callback and the args flag for authentication, so the wrong or no dialog would be found. By: David Vossel (dvossel) 2010-09-20 16:04:21 please test and verify this patch fixes the issue for you. By: Stefan Schmidt (schmidts) 2010-09-20 16:08:07 this patch solves the problem. thank you david! By: Digium Subversion (svnbot) 2010-09-20 16:34:15 Repository: asterisk Revision: 287645 U branches/1.8/channels/chan_sip.c ------------------------------------------------------------------------ r287645 | dvossel | 2010-09-20 16:34:15 -0500 (Mon, 20 Sep 2010) | 9 lines Fixes issue with registrations not working properly with pedantic=yes. (closes issue ASTERISK-16701) Reported by: schmidts Patches: issues_18017_v1.diff uploaded by dvossel (license 671) Tested by: schmidts ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=287645 By: Digium Subversion (svnbot) 2010-09-20 16:35:47 Repository: asterisk Revision: 287646 _U trunk/ U trunk/channels/chan_sip.c ------------------------------------------------------------------------ r287646 | dvossel | 2010-09-20 16:35:47 -0500 (Mon, 20 Sep 2010) | 15 lines Merged revisions 287645 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.8 ........ r287645 | dvossel | 2010-09-20 16:34:15 -0500 (Mon, 20 Sep 2010) | 9 lines Fixes issue with registrations not working properly with pedantic=yes. (closes issue ASTERISK-16701) Reported by: schmidts Patches: issues_18017_v1.diff uploaded by dvossel (license 671) Tested by: schmidts ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=287646 |