diff options
author | Anna Henningsen <anna@addaleax.net> | 2019-09-17 14:17:08 +0200 |
---|---|---|
committer | Anna Henningsen <anna@addaleax.net> | 2019-09-20 20:18:35 +0200 |
commit | ac59dc42edb721ede2e5ddc6d1e4945ee2bf1e9c (patch) | |
tree | b73ea409a56d4efc04dc452713d46222faed1d05 /lib | |
parent | ba3be578d8bbca1bafc391984b0e6f037507dcbc (diff) | |
download | android-node-v8-ac59dc42edb721ede2e5ddc6d1e4945ee2bf1e9c.tar.gz android-node-v8-ac59dc42edb721ede2e5ddc6d1e4945ee2bf1e9c.tar.bz2 android-node-v8-ac59dc42edb721ede2e5ddc6d1e4945ee2bf1e9c.zip |
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>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/_http_common.js | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/lib/_http_common.js b/lib/_http_common.js index c271ec54eb..732f4f29c4 100644 --- a/lib/_http_common.js +++ b/lib/_http_common.js @@ -24,11 +24,7 @@ const { Math } = primordials; const { setImmediate } = require('timers'); -const { getOptionValue } = require('internal/options'); - -const { methods, HTTPParser } = - getOptionValue('--http-parser') === 'legacy' ? - internalBinding('http_parser') : internalBinding('http_parser_llhttp'); +const { methods, HTTPParser } = internalBinding('http_parser'); const FreeList = require('internal/freelist'); const incoming = require('_http_incoming'); |