diff options
author | Anna Henningsen <anna@addaleax.net> | 2019-11-20 00:15:17 +0300 |
---|---|---|
committer | Anna Henningsen <anna@addaleax.net> | 2019-11-30 05:57:58 +0300 |
commit | fc11db18fec5b4632ef26d084bd978335fa80664 (patch) | |
tree | f08970d605bddf48a5c6eb44292ed2088e8548fa /src/node_crypto.cc | |
parent | 7bd587ef0cd574cd6e6e44888217b8d817d0d912 (diff) |
src: inline SetSNICallback
Refs: https://github.com/nodejs/node/pull/30548#discussion_r348168855
PR-URL: https://github.com/nodejs/node/pull/30548
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Diffstat (limited to 'src/node_crypto.cc')
-rw-r--r-- | src/node_crypto.cc | 17 |
1 files changed, 1 insertions, 16 deletions
diff --git a/src/node_crypto.cc b/src/node_crypto.cc index 91704732d18..4b5e5121028 100644 --- a/src/node_crypto.cc +++ b/src/node_crypto.cc @@ -142,7 +142,6 @@ static bool extra_root_certs_loaded = false; template void SSLWrap<TLSWrap>::AddMethods(Environment* env, Local<FunctionTemplate> t); template void SSLWrap<TLSWrap>::ConfigureSecureContext(SecureContext* sc); -template void SSLWrap<TLSWrap>::SetSNIContext(SecureContext* sc); template int SSLWrap<TLSWrap>::SetCACerts(SecureContext* sc); template void SSLWrap<TLSWrap>::MemoryInfo(MemoryTracker* tracker) const; template SSL_SESSION* SSLWrap<TLSWrap>::GetSessionCallback( @@ -2993,12 +2992,7 @@ void SSLWrap<Base>::CertCbDone(const FunctionCallbackInfo<Value>& args) { if (cons->HasInstance(ctx)) { SecureContext* sc = Unwrap<SecureContext>(ctx.As<Object>()); CHECK_NOT_NULL(sc); - // XXX: There is a method w->SetSNIContext(sc), and you might think that - // it makes sense to call that here and make setting w->sni_context_ part - // of it. In fact, that passes the test suite, although SetSNIContext() - // performs a lot more operations. - // If anybody is familiar enough with the TLS code to know whether it makes - // sense, please do so or document why it doesn't. + // Store the SNI context for later use. w->sni_context_ = BaseObjectPtr<SecureContext>(sc); int rv; @@ -3058,15 +3052,6 @@ void SSLWrap<Base>::DestroySSL() { template <class Base> -void SSLWrap<Base>::SetSNIContext(SecureContext* sc) { - ConfigureSecureContext(sc); - CHECK_EQ(SSL_set_SSL_CTX(ssl_.get(), sc->ctx_.get()), sc->ctx_.get()); - - SetCACerts(sc); -} - - -template <class Base> int SSLWrap<Base>::SetCACerts(SecureContext* sc) { int err = SSL_set1_verify_cert_store(ssl_.get(), SSL_CTX_get_cert_store(sc->ctx_.get())); |