[asterisk-dev] [Code Review] 2773: UDPTL: Allow UDPTL to use default configurations when no udptl.conf is present

[Mark Michelson]

> On Aug. 21, 2013, 7:24 a.m., wdoekes wrote:
> > /branches/11/main/udptl.c, lines 1424-1425
> > <https://reviewboard.asterisk.org/r/2773/diff/1/?file=44286#file44286line1424>
> >
> >     Note that we now don't get notified when "the configuration is not updated/reloaded because no udptl.conf could be read".
> >     
> >     But 1.8 was even less verbose.

aco_process_config() will print one of "Unable to load config file 'udptl.conf'" or "udptl.conf is missing! Cannot load udptl.conf" if the file is not present.


- Mark


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2773/#review9449
-----------------------------------------------------------


On Aug. 20, 2013, 9:22 p.m., jrose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2773/
> -----------------------------------------------------------
> 
> (Updated Aug. 20, 2013, 9:22 p.m.)
> 
> 
> Review request for Asterisk Developers, kmoore, Matt Jordan, and Mark Michelson.
> 
> 
> Bugs: ASTERISK-22349
>     https://issues.asterisk.org/jira/browse/ASTERISK-22349
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> If UDPTL failed to load properly due to lack of a configuration file, Asterisk would log a failure to load the configuration, but would continue to function normally until UDPTL configuration was needed by a consumer at which point it would fail to find the global UDPTL configuration and fail, often without displaying a good reason for the failure.
> 
> 
> Diffs
> -----
> 
>   /branches/11/main/udptl.c 397192 
> 
> Diff: https://reviewboard.asterisk.org/r/2773/diff/
> 
> 
> Testing
> -------
> 
> Compared the differences before and after the patch with the following:
> 
> 1. No udptl.conf present in /etc/asterisk
> 2. load Asterisk
> 3. issue comand udptl show config
> 
> prior to patch:
> 
> *CLI> udptl show config
> Command 'udptl show config ' failed.
> 
> 
> after the patch:
> 
> *CLI> udptl show config 
> UDPTL Global options
> --------------------
> udptlstart:      4000
> udptlend:        4999
> udptlfecentries: 0
> udptlfecspan:    0
> use_even_ports:  No
> udptlchecksums: Yes
> 
> 
> Seems pretty open and shut to me.
> 
> 
> Thanks,
> 
> jrose
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130821/8af10688/attachment.htm>