[asterisk-bugs] [Asterisk 0010524]: res_features CLI revamping and removal of default builtins

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=10524 
====================================================================== 
Reported By:                caio1982
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   10524
Category:                   Resources/res_features
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:            SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 80129 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             08-22-2007 07:36 CDT
Last Modified:              08-22-2007 13:11 CDT
====================================================================== 
Summary:                    res_features CLI revamping and removal of default
builtins
Description: 
One problem already talked in previous reports about res_features
(regarding park-dial context) is that wasn't possible to avoid default
builtins for transfers and some parking options, even if you erase your
features.conf. There were hardcoded stuff in res_features.c

I changed it so it's now possible to have a clean installation where
nobody will ever use those default builtins to possible bypass the PBX
restrictions (my own case). Also, I've changed the way information are
displayed in the CLI so now it has a bunch of new parameters shown.

Random thoughts:

1. Having default builtins are wrong because the system may break its own
PBX restrictions

2. And even putting blank options in features.conf to null the builtins is
not very smart, IMHO. It's better have a way to configure them, I mean, at
least the major parameters.

3. Some key information such as timeouts were not getting shown.

4. Also, I have changed the default value for
DEFAULT_FEATURE_DIGIT_TIMEOUT from 500ms to 1000ms so it waits enough. It
was really a short ammount of time to press something in a softphone
keypad, for example.

5. I got rid of the middle column 'Default' in CLI's features show.
====================================================================== 

---------------------------------------------------------------------- 
 caio1982 - 08-22-07 13:11  
---------------------------------------------------------------------- 
Already replying a possible comment (made by Clod Patry, thanks!) is that
changing default values might be a bad idea but I believe it's a even worse
idea to keep default values builtin for actions like, for example, (both
methods of) transfers and call disconnect.

I just thought about it as a PBX admin would not want people pressing
random keys and getting not allowed resposes from Asterisk "by default" :-) 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
08-22-07 13:11  caio1982       Note Added: 0069263                          
======================================================================