Stop the upgrade stream pretending to be a socket

Author: pimterry

doc/api/http.md

@@ -1725,9 +1725,8 @@ before starting to read from the upgraded stream.
 
 The stream argument will typically be the {net.Socket} instance used by the
 request, but in some cases (such as with a request body) it may be a duplex
-stream (though socket properties are exposed for backward compatibility). If
-required, you can access the raw connection underlying the request via
-[`request.socket`][], which is guaranteed to be an instance of {net.Socket}
+stream. If required, you can access the raw connection underlying the request
+via [`request.socket`][], which is guaranteed to be an instance of {net.Socket}
 unless the user specified another socket type.
 
 ### `server.close([callback])`

lib/_http_server.js

@@ -987,35 +987,6 @@ class UpgradeStream extends Duplex {
 
     // Other events (most notably, reading) all only
     // activate after requestBodyCompleted is called.
-
-    // This stream wraps itself in a proxy which forwards all non-stream
-    // property lookups back to the underlying socket, for backward
-    // compatibility.
-    // eslint-disable-next-line no-constructor-return
-    return new Proxy(this, {
-      __proto__: null,
-      has(target, prop) {
-        return prop in target || prop in socket;
-      },
-      get(target, prop, receiver) {
-        if (prop in target) {
-          return ReflectGet(target, prop, receiver);
-        }
-
-        return socket[prop];
-      },
-      set(target, prop, value, receiver) {
-        if (prop in target) {
-          return ReflectSet(target, prop, value, receiver);
-        }
-
-        socket[prop] = value;
-        return true;
-      },
-      getPrototypeOf(target) {
-        return ObjectGetPrototypeOf(socket);
-      },
-    });
   }
 
   requestBodyCompleted(upgradeHead) {