http: remove legacy parser

Remove the legacy `http_parser` implementation as a dependency
and all code that uses it in favor of llhttp, given that the latter
has been the default for all of Node 12 with no outstanding issues.

PR-URL: https://github.com/nodejs/node/pull/29589
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Minwoo Jung <minwoo@nodesource.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

4 files changed, 7 insertions, 22 deletions

diff --git a/doc/api/cli.md b/doc/api/cli.md
index af54fbe7c0..31e5540242 100644
--- a/doc/api/cli.md
+++ b/doc/api/cli.md
@@ -294,22 +294,6 @@ Specify the file name of the heap profile generated by `--heap-prof`.
 Generates a heap snapshot each time the process receives the specified signal.
 `signal` must be a valid signal name. Disabled by default.
 
-### `--http-parser=library`
-<!-- YAML
-added: v11.4.0
--->
-
-Chooses an HTTP parser library. Available values are:
-
-* `llhttp` for https://llhttp.org/
-* `legacy` for https://github.com/nodejs/http-parser
-
-The default is `llhttp`, unless otherwise specified when building Node.js.
-
-This flag exists to aid in experimentation with the internal implementation of
-the Node.js http parser.
-This flag is likely to become a no-op and removed at some point in the future.
-
 ### `--icu-data-dir=file`
 <!-- YAML
 added: v0.11.15
diff --git a/doc/api/deprecations.md b/doc/api/deprecations.md
index 8addff8dfd..46d2467c72 100644
--- a/doc/api/deprecations.md
+++ b/doc/api/deprecations.md
@@ -2475,16 +2475,20 @@ Module.createRequireFromPath() is deprecated. Please use
 ### DEP0131: Legacy HTTP parser
 <!-- YAML
 changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/29589
+    description: This feature has been removed.
   - version: v12.3.0
     pr-url: https://github.com/nodejs/node/pull/27498
     description: Documentation-only.
 -->
 
-Type: Documentation-only
+Type: End-of-Life
 
 The legacy HTTP parser, used by default in versions of Node.js prior to 12.0.0,
-is deprecated. This deprecation applies to users of the
-[`--http-parser=legacy`][] command-line flag.
+is deprecated and has been removed in REPLACEME. Prior to REPLACEME, the
+`--http-parser=legacy` command-line flag could be used to revert to using the
+legacy parser.
 
 <a id="DEP0132"></a>
 ### DEP0132: worker.terminate() with callback
@@ -2514,7 +2518,6 @@ Type: Documentation-only
 Prefer [`response.socket`][] over [`response.connection`] and
 [`request.socket`][] over [`request.connection`].
 
-[`--http-parser=legacy`]: cli.html#cli_http_parser_library
 [`--pending-deprecation`]: cli.html#cli_pending_deprecation
 [`--throw-deprecation`]: cli.html#cli_throw_deprecation
 [`Buffer.allocUnsafeSlow(size)`]: buffer.html#buffer_class_method_buffer_allocunsafeslow_size
diff --git a/doc/api/process.md b/doc/api/process.md
index 3a414dad23..bbf0151e9b 100644
--- a/doc/api/process.md
+++ b/doc/api/process.md
@@ -2348,7 +2348,6 @@ Will generate an object similar to:
   nghttp2: '1.34.0',
   napi: '4',
   llhttp: '1.1.1',
-  http_parser: '2.8.0',
   openssl: '1.1.1b',
   cldr: '34.0',
   icu: '63.1',
diff --git a/doc/api/report.md b/doc/api/report.md
index 8819d2802b..d08de8636a 100644
--- a/doc/api/report.md
+++ b/doc/api/report.md
@@ -53,7 +53,6 @@ is provided below for reference.
       "nghttp2": "1.34.0",
       "napi": "3",
       "llhttp": "1.0.1",
-      "http_parser": "2.8.0",
       "openssl": "1.1.0j"
     },
     "release": {