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

Ensure contentLookup never has more than 5 active requests #711

Open
acolytec3 opened this issue Jan 16, 2025 · 0 comments
Open

Ensure contentLookup never has more than 5 active requests #711

acolytec3 opened this issue Jan 16, 2025 · 0 comments

Comments

@acolytec3
Copy link
Collaborator

acolytec3 commented Jan 16, 2025

I've noticed in the course of testing stuff on the live network that our contentLookup is queueing up more than 5 requests and gets stuck in this pending state where there are up to 20 open requests and none of them have resolved.

One example. Easily reproducible if you request the LightClientBootstrap on the beacon network with a local node.

Image

We should try and reproduce this in a test locally and then find a fix.

The likely solution involves adding a timeout around in this section of code that will delete the request if the other node doesn't respond within a certain amount of time.

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

1 participant