[asterisk-bugs] [Asterisk 0014276]: [patch] allow storage of vmsecret in users voicemail spool directory

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=14276 
====================================================================== 
Reported By:                klaus3000
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   14276
Category:                   Applications/app_voicemail/NewFeature
Reproducibility:            have not tried
Severity:                   minor
Priority:                   normal
Status:                     ready for testing
Asterisk Version:           SVN 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!): 169203 
Request Review:              
====================================================================== 
Date Submitted:             2009-01-19 09:33 CST
Last Modified:              2009-08-26 13:06 CDT
====================================================================== 
Summary:                    [patch] allow storage of vmsecret in users voicemail
spool directory
Description: 
When secretinspool=yes in voicemail.conf, voicemail reads and writes the
secret into a file in the users voicemail spool directory. For example if
voicemail is configured for user 1234 in voicemail context foobar, then the
secret will be stored in /var/spool/asterisk/voicemail/foobar/1234/secret.

When secretinspool=no, it uses the old behavior and stores secrets into
the configuration files.

Note: realtime voiceboxes are never influenced by this parameter
====================================================================== 

---------------------------------------------------------------------- 
 (0109695) tilghman (administrator) - 2009-08-26 13:06
 https://issues.asterisk.org/view.php?id=14276#c109695 
---------------------------------------------------------------------- 
A couple of suggestions:

1) It would be better if this option were specified as something other
than a boolean, because I can see the extension of storing the secret
elsewhere, possibly in astdb or another database, independently of storing
other voicemail configuration data.  passwordlocation=spooldir might be one
way of doing this.

2) I'm uncomfortable with having yet another file format for user options.
 I'd suggest that you create an ast_config structure and use the
ast_config_load and ast_config_save functions, respectively, for loading
and saving the password. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-08-26 13:06 tilghman       Note Added: 0109695                          
======================================================================