crypto: don't call SSL_CTX_set_ciphersuites on boringssl

PR-URL: #26365 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Sam Roberts <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]>
nodejs · Mar 12, 2019 · 7155aaf · 7155aaf
1 parent f99349d
commit 7155aaf
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/node_crypto.cc b/src/node_crypto.cc
@@ -937,7 +937,7 @@ void SecureContext::SetCiphers(const FunctionCallbackInfo<Value>& args) {
   // TLSv1.3 cipher suites, so we get backwards compatible synchronous errors.
   const node::Utf8Value ciphers(args.GetIsolate(), args[0]);
   if (
-#ifdef TLS1_3_VERSION
+#if defined(TLS1_3_VERSION) && !defined(OPENSSL_IS_BORINGSSL)
       !SSL_CTX_set_ciphersuites(sc->ctx_.get(), "") ||
 #endif
       !SSL_CTX_set_cipher_list(sc->ctx_.get(), *ciphers)) {