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

Deno destroys our npm private registry by making too many requests at the same time #21052

Open
RoXuS opened this issue Nov 2, 2023 · 4 comments

Comments

@RoXuS
Copy link

RoXuS commented Nov 2, 2023

Hi Team,

We have migrated one of our front on Deno + vite + lit, it works like a charm, thanks.

We use a private npm registry (verdaccio 5.x).

When we launch Deno run, all npm dependencies are fetched and our registry explode randomly (CPU 100% then it crashes).
We have tried to use a big instance (8 CPU) it better but not perfect (sometimes crash too)...
When we disable HTTP2 on our load balancer all is ok...

Not sure that is a Deno issue but it seems Deno request all dependencies too quickly and too hardly on our registry.

There is a way to slow down the speed at which Deno requests its dependencies?

The error:

Caused by:
    Error getting response at https://OUR-REGISTRY-URL/finalhandler for package "finalhandler": error sending request for url (https://OUR-REGISTRY-URL/finalhandler): http2 error: stream error received: refused stream before processing any application logic: http2 error: stream error received: refused stream before processing any application logic: stream error received: refused stream before processing any application logic
@RoXuS RoXuS changed the title Deno destroy our private registry Deno destroys our private registry Nov 2, 2023
@RoXuS
Copy link
Author

RoXuS commented Nov 2, 2023

maybe link to seanmonstar/reqwest#976 ...

@dsherret dsherret changed the title Deno destroys our private registry Deno destroys our npm private registry by making too many requests at the same time Nov 2, 2023
@magurotuna
Copy link
Member

I believe this is the same issue as #21789. Once hyperium/h2#731 lands and we incorporate that change, this issue should be fixed.

@char8x
Copy link

char8x commented Mar 26, 2024

It seems that all the PRs have been merged, shall we upgrade these dependencies?

@RoXuS
Copy link
Author

RoXuS commented Apr 3, 2024

That would be fantastic

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

No branches or pull requests

4 participants