[Home]

Summary:ASTERISK-17880: [patch] [regression] SIP peers unregistered when asterisk restarts
Reporter:Remi Quezada (remiq)Labels:
Date Opened:2011-05-18 09:51:37Date Closed:2011-05-20 16:18:09
Priority:BlockerRegression?Yes
Status:Closed/CompleteComponents:Channels/chan_sip/Registration
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) diff.txt
Description:When asterisk restarts all SIP peers go unregistered.  The astdb shows the proper SIP Registry entires, but when I do 'sip show peers' the peer is unregistered.  
Comments:By: Leif Madsen (lmadsen) 2011-05-18 13:35:10

I think I can reproduce this. It's sometime after 318400 which I was running previously. I was talking to twilson and he thinks this could be from a change he recently made to fix another issue.

By: Leif Madsen (lmadsen) 2011-05-18 14:12:01

Yes I can confirm this, and Paul is working on a test for the test suite. This has been marked as a regression and blocker for 1.8.5.

By: Leif Madsen (lmadsen) 2011-05-18 14:24:43

Patch confirmed to work for me.

By: Remi Quezada (remiq) 2011-05-18 14:40:48

Patch is working for me as well.

By: Digium Subversion (svnbot) 2011-05-18 15:22:38

Repository: asterisk
Revision: 319552

U   branches/1.8/channels/chan_sip.c

------------------------------------------------------------------------
r319552 | twilson | 2011-05-18 15:22:37 -0500 (Wed, 18 May 2011) | 11 lines

Unbreak the storing of registrations for restart

The fix for issue 18882 broke retrieving non-realtime peers from the ast_db
on restart/reload. This patch tries to unbreak things while leaving the intent
of the original fix intact.
(closes issue ASTERISK-17880)
Reported by: remiq
Patches:
     diff.txt uploaded by twilson (license 396)
Tested by: lmadsen, remiq

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

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

By: Digium Subversion (svnbot) 2011-05-18 15:25:34

Repository: asterisk
Revision: 319564

_U  trunk/
U   trunk/channels/chan_sip.c

------------------------------------------------------------------------
r319564 | twilson | 2011-05-18 15:25:34 -0500 (Wed, 18 May 2011) | 18 lines

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

........
 r319552 | twilson | 2011-05-18 13:22:36 -0700 (Wed, 18 May 2011) | 11 lines
 
 Unbreak the storing of registrations for restart
 
 The fix for issue 18882 broke retrieving non-realtime peers from the ast_db
 on restart/reload. This patch tries to unbreak things while leaving the intent
 of the original fix intact.
 (closes issue ASTERISK-17880)
 Reported by: remiq
 Patches:
       diff.txt uploaded by twilson (license 396)
 Tested by: lmadsen, remiq
........

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

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