refactor: update CODEOWNERS and benchmark files to use internal HTTP modules

Author: bjohansebas

.github/CODEOWNERS

@@ -75,6 +75,7 @@
 /lib/https.js @nodejs/crypto @nodejs/net @nodejs/http
 /src/node_http_common* @nodejs/http @nodejs/http2 @nodejs/net
 /src/node_http_parser.cc @nodejs/http @nodejs/net
+/lib/internal/http/* @nodejs/http @nodejs/net
 
 # http2
 

benchmark/http/check_invalid_header_char.js

@@ -1,8 +1,6 @@
 'use strict';
 
 const common = require('../common.js');
-// TODO: benchmark has access to internal modules?
-const _checkInvalidHeaderChar = require('_http_common')._checkInvalidHeaderChar;
 
 const groupedInputs = {
   // Representative set of inputs from an AcmeAir benchmark run:
@@ -52,9 +50,13 @@ const inputs = [
 const bench = common.createBenchmark(main, {
   input: inputs.concat(Object.keys(groupedInputs)),
   n: [1e6],
+}, {
+  flags: ['--expose-internals', '--no-warnings'],
 });
 
 function main({ n, input }) {
+  const checkInvalidHeaderChar = require('internal/http/common')._checkInvalidHeaderChar;
+
   let inputs = [input];
   if (Object.hasOwn(groupedInputs, input)) {
     inputs = groupedInputs[input];
@@ -63,7 +65,7 @@ function main({ n, input }) {
   const len = inputs.length;
   bench.start();
   for (let i = 0; i < n; i++) {
-    _checkInvalidHeaderChar(inputs[i % len]);
+    checkInvalidHeaderChar(inputs[i % len]);
   }
   bench.end(n);
 }

benchmark/http/check_is_http_token.js

@@ -1,8 +1,6 @@
 'use strict';
 
 const common = require('../common.js');
-// TODO: benchmark has access to internal modules?
-const _checkIsHttpToken = require('_http_common')._checkIsHttpToken;
 
 const bench = common.createBenchmark(main, {
   key: [
@@ -39,12 +37,16 @@ const bench = common.createBenchmark(main, {
     'alternate-protocol:', // slow bailout
   ],
   n: [1e6],
+}, {
+  flags: ['--expose-internals', '--no-warnings'],
 });
 
 function main({ n, key }) {
+  const checkIsHttpToken = require('internal/http/common')._checkIsHttpToken;
+
   bench.start();
   for (let i = 0; i < n; i++) {
-    _checkIsHttpToken(key);
+    checkIsHttpToken(key);
   }
   bench.end(n);
 }

doc/api/deprecations.md

@@ -4329,6 +4329,9 @@ Creating SHAKE-128 and SHAKE-256 digests without an explicit `options.outputLeng
 
 <!-- YAML
 changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/58535
+    description: Runtime deprecation.
   - version:
      - v24.6.0
      - v22.19.0
@@ -4388,21 +4391,6 @@ import { opendir } from 'node:fs/promises';
 }
 ```
 
-### DEP0195: `require('node:_http_*')`
-
-<!-- YAML
-changes:
-  - version: REPLACEME
-    pr-url: https://github.com/nodejs/node/pull/58535
-    description: Runtime deprecation.
--->
-
-Type: Runtime
-
-The `node:_http_agent`, `node:_http_client`, `node:_http_common`, `node:_http_incoming`,
-`node:_http_outgoing` and `node:_http_server` modules are deprecated as they should be considered
-an internal nodejs implementation rather than a public facing API, use `node:http` instead.
-
 [DEP0142]: #dep0142-repl_builtinlibs
 [NIST SP 800-38D]: https://nvlpubs.nist.gov/nistpubs/Legacy/SP/nistspecialpublication800-38d.pdf
 [RFC 6066]: https://tools.ietf.org/html/rfc6066#section-3