[Asterisk-code-review] Documentation: A couple of trivial fixes in sip.conf.sample ... (asterisk[13])

Rusty Newton asteriskteam at digium.com
Mon Jul 20 12:45:40 CDT 2015


Rusty Newton has uploaded a new change for review.

  https://gerrit.asterisk.org/936

Change subject: Documentation: A couple of trivial fixes in sip.conf.sample and func_cdr.c
......................................................................

Documentation: A couple of trivial fixes in sip.conf.sample and func_cdr.c

 * In sip.conf.sample fix sentence where we said that WS or WSS are supported
   transports for use in an outbound register definition. They are not
   supported in that case.
 * In func_cdr.c made it clear that the Disable option for CDR_PROP can be used
   to enable CDR on a channel.

ASTERISK-24867 #close
Reported by: Rusty Newton

ASTERISK-24853 #close
Reported by: PSDK

Change-Id: I3d698bc6302b9d00a0a995b5c4ad9a42d69b48ca
---
M configs/samples/sip.conf.sample
M funcs/func_cdr.c
2 files changed, 3 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/36/936/1

diff --git a/configs/samples/sip.conf.sample b/configs/samples/sip.conf.sample
index a34b5fa..44d2d43 100644
--- a/configs/samples/sip.conf.sample
+++ b/configs/samples/sip.conf.sample
@@ -802,7 +802,7 @@
 ;
 ;register => tls://username:xxxxxx@sip-tls-proxy.example.org
 ;
-;    The 'transport' part defaults to 'udp' but may also be 'tcp', 'tls', 'ws', or 'wss'.
+;    The 'transport' part defaults to 'udp' but may also be 'tcp' or 'tls'.
 ;    Using 'udp://' explicitly is also useful in case the username part
 ;    contains a '/' ('user/name').
 
diff --git a/funcs/func_cdr.c b/funcs/func_cdr.c
index ca54b1e..95d047e 100644
--- a/funcs/func_cdr.c
+++ b/funcs/func_cdr.c
@@ -180,7 +180,8 @@
 						<para>Write-Only</para>
 					</enum>
 					<enum name="disable">
-						<para>Disable CDRs for this channel.</para>
+						<para>Setting to 1 will disable CDRs for this channel.
+						Setting to 0 will enable CDRs for this channel.</para>
 						<para>Write-Only</para>
 					</enum>
 				</enumlist>

-- 
To view, visit https://gerrit.asterisk.org/936
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3d698bc6302b9d00a0a995b5c4ad9a42d69b48ca
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Rusty Newton <rnewton at digium.com>



More information about the asterisk-code-review mailing list