|Summary:||ASTERISK-08746: clear up the description of 'sendvoicemail' and 'dialout' parameters|
|Reporter:||Caio Begotti (caio1982)||Labels:|
|Date Opened:||2007-02-07 06:44:23.000-0600||Date Closed:||2007-06-30 09:20:07|
|Environment:||Attachments:||( 0) tb_voicemail_conf_dump.txt|
( 1) vm_forward_notok_trunk.txt
|Description:||If I configure a SIP peer to not be allowed to forward voicemail messages (sendvoicemail=no) in my Realtime *Static* scheme, they can do that anyway. The attached logs shows my voicemail.conf table with the 2 peers involved and the actual log output from CLI with full debug etc.|
Both sendvoicemail parameters from the general context and from the peer context have been set to "no".
I cannot easily change my layout to test it using plain text files, but if there are anything more possible to test, please let me know.
****** ADDITIONAL INFORMATION ******
I have tested it on the 1.4.0 release and now in trunk (checkout made on 20070206).
|Comments:||By: Caio Begotti (caio1982) 2007-03-06 10:56:48.000-0600|
A lonely ping just in case someone can read it, a peer review would be nice...
By: Serge Vecher (serge-v) 2007-03-06 11:18:03.000-0600
ok, I did some looking at the code; sendvoicemail is not doing what you think. It controls whether or not a user is allowed to send a voicemail from within mailbox's advanced options, not message's. There appears to be no option to control whether a user is allowed to forward the message or not. Adding this option qualifies as a "feature request"
I propose changing the verbage of voicemail.conf.sample within the scale of this issue, taking care of 'dialout' while we are at it:
; dialout=fromvm ; Context to dial out from [option 4 from mailbox's advanced menu]. If not specified, option 4 will not be listed and dialing out from within VoiceMailMain() will not be permitted.
sendvoicemail=yes ; Allow the user to compose and send a voicemail while inside VoiceMailMain() [option 5 from mailbox's advanced menu]. If set to 'no', option 5 will not be listed.
By: Caio Begotti (caio1982) 2007-03-06 12:54:58.000-0600
Sounds good for me, thanks for clarifying this serge-v.
By: Russell Bryant (russell) 2007-03-06 17:01:19.000-0600
I have merged the documentation update in revision 58120 and 58119. Thanks!