[Asterisk-code-review] webrtc: Allow 'webrtc' to be set on endpoints without dtls c... (asterisk[15])

[Sean Bright]

Sean Bright has uploaded this change for review. ( https://gerrit.asterisk.org/6592


Change subject: webrtc: Allow 'webrtc' to be set on endpoints without dtls_ca_file
......................................................................

webrtc: Allow 'webrtc' to be set on endpoints without dtls_ca_file

If using a legitimate from a trusted certificate authority, you don't
need to provide CA file.

Change-Id: I8623973b4209b44889243716d7880274caed8a6d
---
M res/res_pjsip/pjsip_configuration.c
1 file changed, 2 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/92/6592/1

diff --git a/res/res_pjsip/pjsip_configuration.c b/res/res_pjsip/pjsip_configuration.c
index cba8a7e..3a752c9 100644
--- a/res/res_pjsip/pjsip_configuration.c
+++ b/res/res_pjsip/pjsip_configuration.c
@@ -1363,10 +1363,9 @@
 		endpoint->media.rtp.dtls_cfg.default_setup = AST_RTP_DTLS_SETUP_ACTPASS;
 		endpoint->media.rtp.dtls_cfg.verify = AST_RTP_DTLS_VERIFY_FINGERPRINT;
 
-		if (ast_strlen_zero(endpoint->media.rtp.dtls_cfg.certfile) ||
-			(ast_strlen_zero(endpoint->media.rtp.dtls_cfg.cafile))) {
+		if (ast_strlen_zero(endpoint->media.rtp.dtls_cfg.certfile)) {
 			ast_log(LOG_ERROR, "WebRTC can't be enabled on endpoint '%s' - a DTLS cert "
-				"or ca file has not been specified", ast_sorcery_object_get_id(endpoint));
+				"has not been specified", ast_sorcery_object_get_id(endpoint));
 			return -1;
 		}
 	}

-- 
To view, visit https://gerrit.asterisk.org/6592
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 15
Gerrit-MessageType: newchange
Gerrit-Change-Id: I8623973b4209b44889243716d7880274caed8a6d
Gerrit-Change-Number: 6592
Gerrit-PatchSet: 1
Gerrit-Owner: Sean Bright <sean.bright at gmail.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20170925/e3f68d0d/attachment.html>