[Asterisk-code-review] res pjsip: Ignore empty TLS configuration (asterisk[13])

Sean Bright asteriskteam at digium.com
Mon Nov 6 08:15:17 CST 2017


Hello Joshua Colp, 

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

    https://gerrit.asterisk.org/7004

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

Change subject: res_pjsip: Ignore empty TLS configuration
......................................................................

res_pjsip: Ignore empty TLS configuration

When using realtime, fields that are not explicitly set by an
administrator are still presented to sorcery as empty strings. Handle
this case explicitly.

In this particular case, if any of these fields are required for TLS
support, their existence should be validated in the 'apply' handler once
we have a complete transport definition.

ASTERISK-27032 #close
Reported by: seanchann.zhou

Change-Id: Ie3b5fb421977ccdb33e415d4ec52c3fd192601b7
---
M res/res_pjsip/config_transport.c
1 file changed, 5 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/04/7004/2
-- 
To view, visit https://gerrit.asterisk.org/7004
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie3b5fb421977ccdb33e415d4ec52c3fd192601b7
Gerrit-Change-Number: 7004
Gerrit-PatchSet: 2
Gerrit-Owner: Sean Bright <sean.bright at gmail.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20171106/6a507228/attachment.html>


More information about the asterisk-code-review mailing list