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-0600Date Closed:2007-06-30 09:20:07
Versions:Frequency of
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.


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!