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

If set default page is invalid, thousands of XHR requests are made #1496

Open
pointydev opened this issue Aug 3, 2021 · 2 comments
Open
Labels
🐛 Bug Issues marked with this label indicate unintended program behaviour that needs correction. 🟢 Low priority Issues marked with this label are actively being worked on if nothing serious is on the list. 👍 PR-ok Issues marked with this label are good candidates for being accepted in a pull request.

Comments

@pointydev
Copy link

Description

Related to #1455. If the default page set in Local Storage is set to an invalid value (such as an old installation), navigating to an invalid page causes thousands of XHR requests to be made.

Steps to reproduce

  1. (To emulate an old installation, run localStorage["asf-ui:settings:default-view"] = "\"home\"" in console)
  2. Navigate to any invalid ASF-ui page, such as the root: http://localhost:1242
  3. Observe network activity

Expected behavior

The invalid value is replaced by the default value and network activity is normal.

Current behavior

The invalid value remains, the user is not automatically redirected to a working page, and ~10000 requests to the IPC API can be made in around a minute, even if the user navigates to a working page from there.

Screenshots


Additional information

This behaviour does not present itself on the IPC password screen, only once logged in. Even though this can be easily remedied by manually changing the setting, it may not be apparent to all users, leading to excess resource consumption.

Thanks,
Elliott

@pointydev pointydev added 🐛 Bug Issues marked with this label indicate unintended program behaviour that needs correction. 🧐 Evaluation Issues marked with this label are currently being evaluated if they're going to be considered. labels Aug 3, 2021
@Abrynos
Copy link
Contributor

Abrynos commented Aug 3, 2021

Finally a way to reproduce the issue. 🎉

@Abrynos
Copy link
Contributor

Abrynos commented Aug 3, 2021

Clarification: We discussed the "thousands of requests" on our discord server previously, but never found a way to reliably reproduce it. Only consensus was that it happend after updating.

@MrBurrBurr MrBurrBurr added 👍 PR-ok Issues marked with this label are good candidates for being accepted in a pull request. 🔴 High priority Issues marked with this label indicate the most serious problems, especially security-related. 🟢 Low priority Issues marked with this label are actively being worked on if nothing serious is on the list. and removed 🧐 Evaluation Issues marked with this label are currently being evaluated if they're going to be considered. 🔴 High priority Issues marked with this label indicate the most serious problems, especially security-related. labels Aug 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug Issues marked with this label indicate unintended program behaviour that needs correction. 🟢 Low priority Issues marked with this label are actively being worked on if nothing serious is on the list. 👍 PR-ok Issues marked with this label are good candidates for being accepted in a pull request.
Projects
None yet
Development

No branches or pull requests

3 participants