src: use HandleScope in StreamReq::Done()

Fixes: #38707

PR-URL: #38720
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>

Author: RaisinTen

src/stream_base-inl.h

@@ -279,6 +279,7 @@ void StreamReq::Done(int status, const char* error_str) {
   AsyncWrap* async_wrap = GetAsyncWrap();
   Environment* env = async_wrap->env();
   if (error_str != nullptr) {
+    v8::HandleScope handle_scope(env->isolate());
     async_wrap->object()->Set(env->context(),
                               env->error_string(),
                               OneByteString(env->isolate(), error_str))

test/parallel/test-https-agent-unref-socket.js

@@ -0,0 +1,13 @@
+'use strict';
+
+const common = require('../common');
+if (!common.hasCrypto)
+  common.skip('missing crypto');
+
+const https = require('https');
+
+const request = https.get('https://example.com');
+
+request.on('socket', (socket) => {
+  socket.unref();
+});