[asterisk-bugs] [Asterisk 0014248]: [patch] Add Contact field to sip show registry

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=14248 
====================================================================== 
Reported By:                Nick_Lewis
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   14248
Category:                   Channels/chan_sip/Registration
Reproducibility:            always
Severity:                   feature
Priority:                   normal
Status:                     ready for testing
Target Version:             1.6.2
Asterisk Version:           1.6.0 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2009-01-15 04:15 CST
Last Modified:              2009-01-29 05:18 CST
====================================================================== 
Summary:                    [patch] Add Contact field to sip show registry
Description: 
Currently the contact field is not shown on the registry display. It would
be useful if it were included in the response to the sip show registry cli
command. It is often more readily informative than the username or host.

The contact field is also known as r->callback and is derived from the
/contact portion of the registration string.
====================================================================== 

---------------------------------------------------------------------- 
 (0099035) Nick_Lewis (reporter) - 2009-01-29 05:18
 http://bugs.digium.com/view.php?id=14248#c99035 
---------------------------------------------------------------------- 
I did think about using the AOR as the unique identifier but concluded that
it was quicker to type an id. The id should be the order in which the
registration strings appear in sip.conf but reload does confuse matters a
bit.

I will rework the patch to use the AOR instead.

Regarding "multiple registrations" I do not think forking should be
encouraged especially to different interfaces on the same asterisk pbx. I
suggest that any such registrations are merged with a log message such as 
"A registration to AOR '%s' already exists - Ignoring registration string
at line '%d'\n" 
or 
"A registration to AOR '%s' already exists - Ignoring register=yes for
peer '%s' at line '%d'\n" 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-01-29 05:18 Nick_Lewis     Note Added: 0099035                          
======================================================================