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

Nodejs v13.0.1 HTTP2 NGHTTP2_ENHANCE_YOUR_CALM & NGHTTP2_INTERNAL_ERROR #30193

Closed
Rantoledo opened this issue Oct 31, 2019 · 4 comments
Closed
Labels
http2 Issues or PRs related to the http2 subsystem.

Comments

@Rantoledo
Copy link

Rantoledo commented Oct 31, 2019

  • Version: 13.0.1
  • Platform: Linux

I've tried to create a simple client-server app using http/2, that downloads directories.
On September 1st I posted that nodejs crashes at some point.
My previous bug report:
#29393

Now, nodejs doesn't crash. But I still get NGHTTP2_ENHANCE_YOUR_CALM & NGHTTP2_INTERNAL_ERROR at some point.

Here's my code to reproduce, hope it helps:
https://github.com/Rantoledo/http2_nodejs_client_server_example2.git

BTW, I've tried to set MaxSessionMemory to 10, 100, 1000 and even 10000. I don't think it made any difference.

@Rantoledo
Copy link
Author

Hey guys, any updates?

@lundibundi
Copy link
Member

Is it possibly similar to #29223 (though that only deals with NGHTTP2_ENHANCE_YOUR_CALM) but has a fix already that will soon be landed.

@lundibundi lundibundi added http2 Issues or PRs related to the http2 subsystem. v13.x labels Nov 27, 2019
@Rantoledo
Copy link
Author

@lundibundi I hope it's the same problem... thanks!

@lundibundi
Copy link
Member

Should be fixed in all release lines since 13.3.0, 12.14.1 and 10.18.1 via #30684.

Feel free to reopen if the issue is still present while using 'fixed' Node.js version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
http2 Issues or PRs related to the http2 subsystem.
Projects
None yet
Development

No branches or pull requests

2 participants