nodejs
diff --git a/Collapse file
‎src/crypto/crypto_common.cc‎
Copy file name to clipboardExpand all lines: src/crypto/crypto_common.cc
+4-13Lines changed: 4 additions & 13 deletions b/Collapse file
‎src/crypto/crypto_common.cc‎
Copy file name to clipboardExpand all lines: src/crypto/crypto_common.cc
+4-13Lines changed: 4 additions & 13 deletions
diff --git a/Collapse file
‎src/crypto/crypto_common.h‎
Copy file name to clipboardExpand all lines: src/crypto/crypto_common.h
+2-3Lines changed: 2 additions & 3 deletions b/Collapse file
‎src/crypto/crypto_common.h‎
Copy file name to clipboardExpand all lines: src/crypto/crypto_common.h
+2-3Lines changed: 2 additions & 3 deletions
diff --git a/Collapse file
‎src/crypto/crypto_tls.cc‎
Copy file name to clipboardExpand all lines: src/crypto/crypto_tls.cc
+2-1Lines changed: 2 additions & 1 deletion b/Collapse file
‎src/crypto/crypto_tls.cc‎
Copy file name to clipboardExpand all lines: src/crypto/crypto_tls.cc
+2-1Lines changed: 2 additions & 1 deletion
@@ -26,7 +26,6 @@ namespace node {
 
 using v8::Array;
 using v8::ArrayBuffer;
-using v8::ArrayBufferView;
 using v8::BackingStore;
 using v8::Context;
 using v8::EscapableHandleScope;
@@ -87,18 +86,10 @@ void LogSecret(
   keylog_cb(ssl.get(), line.c_str());
 }
 
-bool SetALPN(const SSLPointer& ssl, const std::string& alpn) {
-  return SSL_set_alpn_protos(
-      ssl.get(),
-      reinterpret_cast<const uint8_t*>(alpn.c_str()),
-      alpn.length()) == 0;
-}
-
-bool SetALPN(const SSLPointer& ssl, Local<Value> alpn) {
-  if (!alpn->IsArrayBufferView())
-    return false;
-  ArrayBufferViewContents<unsigned char> protos(alpn.As<ArrayBufferView>());
-  return SSL_set_alpn_protos(ssl.get(), protos.data(), protos.length()) == 0;
+bool SetALPN(const SSLPointer& ssl, std::string_view alpn) {
+  return SSL_set_alpn_protos(ssl.get(),
+                             reinterpret_cast<const uint8_t*>(alpn.data()),
+                             alpn.length()) == 0;
 }
 
 MaybeLocal<Value> GetSSLOCSPResponse(
 
@@ -33,9 +33,8 @@ void LogSecret(
     const unsigned char* secret,
     size_t secretlen);
 
-bool SetALPN(const SSLPointer& ssl, const std::string& alpn);
-
-bool SetALPN(const SSLPointer& ssl, v8::Local<v8::Value> alpn);
+// TODO(tniessen): use std::u8string_view when we switch to C++20.
+bool SetALPN(const SSLPointer& ssl, std::string_view alpn);
 
 v8::MaybeLocal<v8::Value> GetSSLOCSPResponse(
     Environment* env,
 
@@ -1530,7 +1530,8 @@ void TLSWrap::SetALPNProtocols(const FunctionCallbackInfo<Value>& args) {
     return env->ThrowTypeError("Must give a Buffer as first argument");
 
   if (w->is_client()) {
-    CHECK(SetALPN(w->ssl_, args[0]));
+    ArrayBufferViewContents<char> protos(args[0].As<ArrayBufferView>());
+    CHECK(SetALPN(w->ssl_, {protos.data(), protos.length()}));
   } else {
     CHECK(
         w->object()->SetPrivate(