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

switch status checks to WS:// #44

Open
CouleeApps opened this issue Apr 28, 2022 · 2 comments
Open

switch status checks to WS:// #44

CouleeApps opened this issue Apr 28, 2022 · 2 comments
Labels
enhancement New feature or request

Comments

@CouleeApps
Copy link
Collaborator

Currently the frontend polls requests every second for every decompiler, which ends up being 8 req/sec/user, which is a lot. We should reduce this number by using only 1 request per binary job, and remember to cancel outstanding requests if the user uploads a new binary. We may be able to use websockets instead, to really tone down the volume of requests

@CouleeApps CouleeApps added the enhancement New feature or request label Apr 28, 2022
@psifertex
Copy link
Contributor

At the very least could probably just crank down the timeout to several seconds without (much) noticeable user impact?

@CouleeApps
Copy link
Collaborator Author

This has been mitigated for the meantime, though we still want to switch to websockets in the medium term.

@psifertex psifertex changed the title Reduce request volume switch status checks to WS:// Jul 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants