| Summary: | ASTERISK-15189: [patch] After upgrading to asterisk 1.4.27 Optipoint SIP phone can no longer register | ||
| Reporter: | Michael Gernoth (mgernoth) | Labels: | |
| Date Opened: | 2009-11-21 07:30:01.000-0600 | Date Closed: | 2009-12-07 12:10:38.000-0600 |
| Priority: | Major | Regression? | Yes |
| Status: | Closed/Complete | Components: | Channels/chan_sip/Registration |
| Versions: | Frequency of Occurrence | ||
| Related Issues: | |||
| Environment: | Attachments: | ( 0) debug-broken-register-1.4.27.txt ( 1) debug-working-register-1.4.26.2.txt ( 2) reg_parse_issue_1.4.diff | |
| Description: | SVN revision 218578 (Send request contact header field with response to registrer queries instead of the address of record) introduced a change in the generation of SIP Contact-Header fields. After this change, my Optipoint 400 SIP phone thinks it is not registered to asterisk, but asterisk has a working registration. In sip debug traces (attached), I see that 1.4.26 sends the Contact-Header in the final REGISTER response as "Contact: <sip:13@192.168.50.254:5060;transport=udp>;expires=605" whereas 1.4.27 sends "Contact: sip:13@192.168.50.254:5060;transport=udp;expires=605" which misses the angle brackets and seems to confuse the phone. Reverting that one patch leads to a working 1.4.27 where Optipoint phones are able to register again. | ||
| Comments: | By: Digium Subversion (svnbot) 2009-12-07 12:07:39.000-0600 Repository: asterisk Revision: 233471 U branches/1.4/channels/chan_sip.c ------------------------------------------------------------------------ r233471 | dvossel | 2009-12-07 12:07:38 -0600 (Mon, 07 Dec 2009) | 9 lines fixes missing Contact header angle brackets (closes issue ASTERISK-15189) Reported by: mgernoth Patches: reg_parse_issue_1.4.diff uploaded by dvossel (license 671) Tested by: dvossel ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=233471 By: Digium Subversion (svnbot) 2009-12-07 12:08:47.000-0600 Repository: asterisk Revision: 233472 _U trunk/ U trunk/channels/chan_sip.c ------------------------------------------------------------------------ r233472 | dvossel | 2009-12-07 12:08:47 -0600 (Mon, 07 Dec 2009) | 15 lines Merged revisions 233471 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r233471 | dvossel | 2009-12-07 12:07:38 -0600 (Mon, 07 Dec 2009) | 9 lines fixes missing Contact header angle brackets (closes issue ASTERISK-15189) Reported by: mgernoth Patches: reg_parse_issue_1.4.diff uploaded by dvossel (license 671) Tested by: dvossel ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=233472 By: Digium Subversion (svnbot) 2009-12-07 12:09:27.000-0600 Repository: asterisk Revision: 233473 _U branches/1.6.2/ U branches/1.6.2/channels/chan_sip.c ------------------------------------------------------------------------ r233473 | dvossel | 2009-12-07 12:09:26 -0600 (Mon, 07 Dec 2009) | 22 lines Merged revisions 233472 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r233472 | dvossel | 2009-12-07 12:08:46 -0600 (Mon, 07 Dec 2009) | 15 lines Merged revisions 233471 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r233471 | dvossel | 2009-12-07 12:07:38 -0600 (Mon, 07 Dec 2009) | 9 lines fixes missing Contact header angle brackets (closes issue ASTERISK-15189) Reported by: mgernoth Patches: reg_parse_issue_1.4.diff uploaded by dvossel (license 671) Tested by: dvossel ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=233473 By: Digium Subversion (svnbot) 2009-12-07 12:10:02.000-0600 Repository: asterisk Revision: 233474 _U branches/1.6.1/ U branches/1.6.1/channels/chan_sip.c ------------------------------------------------------------------------ r233474 | dvossel | 2009-12-07 12:10:02 -0600 (Mon, 07 Dec 2009) | 22 lines Merged revisions 233472 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r233472 | dvossel | 2009-12-07 12:08:46 -0600 (Mon, 07 Dec 2009) | 15 lines Merged revisions 233471 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r233471 | dvossel | 2009-12-07 12:07:38 -0600 (Mon, 07 Dec 2009) | 9 lines fixes missing Contact header angle brackets (closes issue ASTERISK-15189) Reported by: mgernoth Patches: reg_parse_issue_1.4.diff uploaded by dvossel (license 671) Tested by: dvossel ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=233474 By: Digium Subversion (svnbot) 2009-12-07 12:10:37.000-0600 Repository: asterisk Revision: 233475 _U branches/1.6.0/ U branches/1.6.0/channels/chan_sip.c ------------------------------------------------------------------------ r233475 | dvossel | 2009-12-07 12:10:37 -0600 (Mon, 07 Dec 2009) | 22 lines Merged revisions 233472 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r233472 | dvossel | 2009-12-07 12:08:46 -0600 (Mon, 07 Dec 2009) | 15 lines Merged revisions 233471 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r233471 | dvossel | 2009-12-07 12:07:38 -0600 (Mon, 07 Dec 2009) | 9 lines fixes missing Contact header angle brackets (closes issue ASTERISK-15189) Reported by: mgernoth Patches: reg_parse_issue_1.4.diff uploaded by dvossel (license 671) Tested by: dvossel ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=233475 | ||