[Home]

Summary:ASTERISK-17653: [patch] Add 'description' field for SIP peers
Reporter:Leif Madsen (lmadsen)Labels:
Date Opened:2011-04-06 14:12:20Date Closed:2011-04-13 10:49:36
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Channels/chan_sip/NewFeature
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) __20110406_sip_peer_description.txt
( 1) __20110407-channel_description.txt
( 2) __20110408-channel-description.txt
Description:Using some of the best practices names for SIP peers provides the quandary of knowing exactly what peer is where, or who it is assigned to.

If you name all your peers as the MAC address of the device, then it's difficult to determine what device is assigned to whom, or at least knowing where the device physically resides.

I've added a description field that can help. When you do 'sip show peers' then a new column called Description outputs 32 chars of a description to the Asterisk console.
Comments:By: antonio (antonio) 2011-04-07 01:29:04

Well I would applaud this, see https://issues.asterisk.org/view.php?id=16466.
By: Leif Madsen (lmadsen) 2011-04-07 13:03:59

Added a new patch that provides support for chan_dahdi.conf, sip.conf, and iax.conf, along with a Description field for manager options IAXpeers, SIPpeers, SIPshowpeer, and DAHDIshowchannels.
By: Digium Subversion (svnbot) 2011-04-13 10:49:35

Repository: asterisk
Revision: 313528

U   trunk/CHANGES
U   trunk/channels/chan_dahdi.c
U   trunk/channels/chan_iax2.c
U   trunk/channels/chan_sip.c
U   trunk/channels/sip/include/sip.h
U   trunk/configs/chan_dahdi.conf.sample
U   trunk/configs/iax.conf.sample
U   trunk/configs/sip.conf.sample
U   trunk/configs/users.conf.sample

------------------------------------------------------------------------
r313528 | lmadsen | 2011-04-13 10:49:34 -0500 (Wed, 13 Apr 2011) | 9 lines

Add 'description' field for CLI and Manager output

(closes issue ASTERISK-17653)
Reported by: lmadsen
Patches:
     __20110408-channel-description.txt uploaded by lmadsen (license 10)
Tested by: lmadsen

Review: https://reviewboard.asterisk.org/r/1163/
------------------------------------------------------------------------

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