Avoid using deprecated process.binding('http_parser')

spdy-http2 / http-deceiver

Deceive HTTP Parser

17 stars 8 forks source link

Avoid using deprecated process.binding('http_parser') #7

Open lamweili opened 3 years ago

lamweili commented 3 years ago

Closes https://github.com/spdy-http2/http-deceiver/issues/6 Refs: https://nodejs.org/api/deprecations.html#DEP0111

lamweili commented 3 years ago

Related to a sub-issue in https://github.com/spdy-http2/node-spdy/issues/380 Approving this pull request will close #6 and #7.

jaketrimble commented 2 years ago

Any movement on this PR?

alexander-akait commented 2 years ago

/cc @indutny Can you review this? Will be great to fix deprecation issue

tdd commented 2 years ago

Folks, it's been 6 months, and http-deceiver's dep warning here on Node >= 10 causes warnings across the board (spdy, hence restify etc.). We have a working PR, what gives? @indutny I understand you've got a full gynormous plate, but this would really help :)

nikeee commented 2 years ago

Any chance to get this merged? I'd be happy to help.

Xenope commented 2 years ago

Any news? it's been a while now!

aIbarrab commented 2 years ago

waiting for this to be merged hehe

Cc @indutny @ILOveyouo

lamweili commented 2 years ago

cc: @jacobheun @daviddias @indutny

beenotung commented 1 year ago

I've forked this package get this PR (https://github.com/spdy-http2/http-deceiver/pull/7) and https://github.com/spdy-http2/http-deceiver/pull/5 patched and published to npm as http-deceiver-fixes