[Asterisk-code-review] res sorcery config: Allow configuration section to be used b... (asterisk[13])

Joshua Colp asteriskteam at digium.com
Wed Jul 18 12:41:47 CDT 2018


Hello Jenkins2, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/9514

to look at the new patch set (#2).

Change subject: res_sorcery_config: Allow configuration section to be used based on name.
......................................................................

res_sorcery_config: Allow configuration section to be used based on name.

A problem I've seen countless times is a global or system section
for PJSIP not getting applied. This is inevitably the result of
the "type=" line missing. This change alleviates that problem.

The ability to specify an explicit section name has been
added to res_sorcery_config. If the configured section
name matches this and there are no unknown things configured
the section is taken as being for the given type.

Both the PJSIP "global" and "system" types now support this
so you can just name your section "global" or "system" and it
will be matched and used, even without a "type=" line.

ASTERISK-27972

Change-Id: Ie22723663c1ddd24f869af8c9b4c1b59e2476893
---
M res/res_pjsip.c
M res/res_pjsip/config_global.c
M res/res_pjsip/config_system.c
M res/res_sorcery_config.c
4 files changed, 100 insertions(+), 11 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/14/9514/2
-- 
To view, visit https://gerrit.asterisk.org/9514
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie22723663c1ddd24f869af8c9b4c1b59e2476893
Gerrit-Change-Number: 9514
Gerrit-PatchSet: 2
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Jenkins2
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180718/bf9e6088/attachment.html>


More information about the asterisk-code-review mailing list