Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DEP0111] deprecation warning from restify's dependency on node 16 #1876

Open
3 tasks done
pcwiek opened this issue May 19, 2021 · 16 comments
Open
3 tasks done

[DEP0111] deprecation warning from restify's dependency on node 16 #1876

pcwiek opened this issue May 19, 2021 · 16 comments

Comments

@pcwiek
Copy link

pcwiek commented May 19, 2021

  • Used appropriate template for the issue type
  • Searched both open and closed issues for duplicates of this issue
  • Title adequately and concisely reflects the feature or the bug

Restify Version: 8.5.1
Node.js Version: 16.1.0

Expected behaviour

No node deprecation warnings. This is mostly important in the longer term, because if the subsequent LTS version drops the deprecated functionality, that means that restify will be incompatible with node 18.

Actual behaviour

When starting the server:

(node:1751) [DEP0111] DeprecationWarning: Access to process.binding('http_parser') is deprecated.
    at process.binding (node:internal/bootstrap/loaders:133:17)
    at Object.<anonymous> (/dist/node_modules/http-deceiver/lib/deceiver.js:22:24)
    // cut short

Repro case

Installing restify 8.5.1 and spinning up a restify server on node 16 seems to be enough to trigger the warning.

Cause

Dependency chain: restify -> spdy -> http-deceiver

Are you willing and able to fix this?

I don't think I'm able to. The core functionality that accesses process.binding lives in the http-deceiver package, which is part of spdy.
Unless the behavior can be changed there, or the whole dependency can be replaced, I'm not sure what else can be done.

@GaikwadPratik
Copy link

adding link to dependent issue in spdy 380

@Romulo-S
Copy link

I have this problem when do ng serve

Your global Angular CLI version (12.2.6) is greater than your local version (10.2.3). The local Angular CLI version is used.

To disable this warning use "ng config -g cli.warnings.versionMismatch false".
10% building 3/3 modules 0 active(node:19144) [DEP0111] DeprecationWarning: Access to process.binding('http_parser') is deprecated.
(Use node --trace-deprecation ... to show where the warning was created)

@dibari
Copy link

dibari commented Nov 8, 2021

Any anticipated movement on this? It would be helpful to know if this project is still generally supported.

@nikeee
Copy link

nikeee commented Feb 15, 2022

Is this resolved in v9? The 9.x branch has Node.js 16 listed in the tested CI environments.

@mcjambi
Copy link

mcjambi commented Feb 18, 2022

my log file has so many line that contant: "Access to process.binding('http_parser') is deprecated." and for a year you have not fixed it!

@bunays
Copy link

bunays commented Apr 2, 2022

10% building 3/3 modules 0 active(node:15988) [DEP0111] DeprecationWarning: Access to process.binding('http_parser') is deprecated.
(Use node --trace-deprecation ... to show where the warning was created)
anybody solved this?

@NathBabs
Copy link

NathBabs commented Apr 7, 2022

please any fix for this, it keeps showing in my console and i don't like it

@Phero49
Copy link

Phero49 commented Apr 17, 2022

is it safe to use or i should just use express because i don`t like warning

@NathBabs
Copy link

I don't know why anyone will use this framework in production, this issue has been around for years and nothing has been done about it.
Still flustered as to why this company I just joined is using it, when Express is there.
@Phero49 I think you should just use Express

@shairontoledo
Copy link

I ended up adding --no-deprecation to node and node-based commands to silent the warning:

  • nodemon --no-deprecation ...
  • ts-node --no-deprecation index.ts
  • node --no-deprecation index.js

@djui
Copy link

djui commented Jun 1, 2022

It seems Restify 8 supports SPDY as well as HTTP2, by setting options.spdy = ...; see https://github.com/restify/node-restify/blob/master/lib/server.js#L152-L154

As SPDY is considered experimental and deprecated, replaced by HTTP2, it would be an easy fix to just remove SPDY support from e.g. 9+.

@mcjambi
Copy link

mcjambi commented Jun 20, 2022

i have to change to use http2 in NginX, nodejs should not use spdy.

@mainfraame
Copy link

so any plans to just remove spdy support & package?

@nexdrew
Copy link

nexdrew commented Nov 17, 2022

According to PR #1929, which adds support for Node 18 but doesn't remove spdy, the deprecation warning still exists.

So the good news is that the deprecation does not block upgrading to Node 18 (once a new version of restify is published after the PR is merged). The bad news is that the deprecation warning will still be there.

@NormandoHall
Copy link

Restify 10, Node 16, and still the warning...

@jwyglendowski-precisionlender

Have you looked into https://www.npmjs.com/package/spdy-fixes which looks like it depends on http-deceiver-fixes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests