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

Warmup command functionality broken #130

Open
Qup42 opened this issue Feb 8, 2025 · 0 comments
Open

Warmup command functionality broken #130

Qup42 opened this issue Feb 8, 2025 · 0 comments

Comments

@Qup42
Copy link
Member

Qup42 commented Feb 8, 2025

  • Most of qlever-ui's commands warmup commands issue require an access token. Warmup commands don't include an access token in the requests to the back end even if one is defined. This applies to the management command as well as the HTTP API.
  • The HTTP API for calling warmup commands uses a hard coded token for access control. This has no security impact, because the underlying functions don't work. Still this is bad practice and should be fixed.
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