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

Sean Bright asteriskteam at digium.com
Mon Sep 25 13:02:59 CDT 2017


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


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/93/6593/1

diff --git a/res/res_pjsip/pjsip_configuration.c b/res/res_pjsip/pjsip_configuration.c
index d3ff1f3..2360af2 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/6593
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 15.0
Gerrit-MessageType: newchange
Gerrit-Change-Id: I8623973b4209b44889243716d7880274caed8a6d
Gerrit-Change-Number: 6593
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/3ce64cf0/attachment.html>


More information about the asterisk-code-review mailing list