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

chore: Suppress deprecation warning caused by http-proxy #46

Closed
wants to merge 1 commit into from

Conversation

danjoa
Copy link
Contributor

@danjoa danjoa commented May 9, 2024

Suppress deprecation warning in Node 22 due to http-proxy using util._extend() → see:

As http-proxy seems to be not well maintained, this hack would suppress those annoying warnings.

@danjoa
Copy link
Contributor Author

danjoa commented May 9, 2024

P.S. Could I get access to this repo? → would avoid the fork overhead for me

@oklemenz2
Copy link
Contributor

Can't we wait for node-http-proxy to solve this issue?

@rlindner81
Copy link
Contributor

rlindner81 commented May 10, 2024

Can't we wait for node-http-proxy to solve this issue?

@danjoa is right, the library is no longer actively maintained. I would not expect them to fix this in any reasonable timeframe.

I think you could try and patch-package their code. That would fix it when it is used within our library.

@oklemenz2
Copy link
Contributor

Part of 2f98c34

@oklemenz2 oklemenz2 closed this May 17, 2024
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

Successfully merging this pull request may close these issues.

None yet

3 participants