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

Two API requests done towards api.github.com anytime someone visits ACS UI #10522

Open
Sparc0 opened this issue Mar 6, 2025 · 8 comments · May be fixed by #10533
Open

Two API requests done towards api.github.com anytime someone visits ACS UI #10522

Sparc0 opened this issue Mar 6, 2025 · 8 comments · May be fixed by #10533

Comments

@Sparc0
Copy link

Sparc0 commented Mar 6, 2025

problem

Anytime anyone visit our cloudstack webui you get two notifications in top right corner saying:

Forbidden
API rate limit exceeded for REDACTED_IP. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)

It does not matter if you are logged in our not. It's because for some reason my browser do two API requests towards Github.

versions

ACS: 4.20.0

The steps to reproduce the bug

  1. Visit the webui
  2. Open browser console
  3. F5
  4. Two API requests towards Github is made to https://api.github.com/repos/apache/cloudstack/releases

What to do about it?

Can this behavior be disabled somehow?

@Sparc0 Sparc0 changed the title Two API requests done towards api.github.com anytime i someone visits ACS UI Two API requests done towards api.github.com anytime someone visits ACS UI Mar 6, 2025
@rohityadavcloud rohityadavcloud added this to the 4.20.1 milestone Mar 7, 2025
@DaanHoogland
Copy link
Contributor

@Sparc0 , I see only one call, but that IMNSHO is one too many. I will look into the functionality requiring this and into disabling this by default.

@DaanHoogland
Copy link
Contributor

@Sparc0 the reason you might see two calls to api.github is because you have oauth enabled, is that true?

@Sparc0
Copy link
Author

Sparc0 commented Mar 7, 2025

I dont use oauth.
This is a complete fresh and new mgmt server installed and i was met by the error on first login.

Image

Since it does it towards https://api.github.com/repos/apache/cloudstack/releases does the UI check i run the latest version or something?

@DaanHoogland
Copy link
Contributor

@Sparc0 , another question, you should only get this when you log on as "Root Admin". Are you saying you get it as any user?

Since it does it towards https://api.github.com/repos/apache/cloudstack/releases does the UI check i run the latest version or something?

Yes, it should tell you if there is a newer version.

@Sparc0
Copy link
Author

Sparc0 commented Mar 7, 2025

We only have one user at the moment, have not setup LDAP or any other local users yet. I could try that next week see if that helps.

@DaanHoogland DaanHoogland linked a pull request Mar 9, 2025 that will close this issue
14 tasks
@DaanHoogland DaanHoogland linked a pull request Mar 9, 2025 that will close this issue
14 tasks
@DaanHoogland
Copy link
Contributor

@Sparc0 , I created a PR that would reduce the double call to api.github, but I could not reproduce that part, so it is purely theoratical. Can you try #10533 ?

@Sparc0
Copy link
Author

Sparc0 commented Mar 10, 2025

Sure, do you want me to clone your PR and build it using this instructions?
OR is the EL9 packages that blue mentions here available somewhere?

@DaanHoogland
Copy link
Contributor

The packages are in a private lab to be used for integration tests. If you can build, please do. If all else fails we can arrange some file transfer.

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

Successfully merging a pull request may close this issue.

3 participants