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

API requests for non-existent status pages hang forever #4535

Closed
1 task done
MikeBishop opened this issue Feb 27, 2024 · 0 comments Β· Fixed by #4537
Closed
1 task done

API requests for non-existent status pages hang forever #4535

MikeBishop opened this issue Feb 27, 2024 · 0 comments Β· Fixed by #4537
Labels
area:api concearning the api or automation area:status-page Everything related to the status page bug Something isn't working

Comments

@MikeBishop
Copy link

πŸ“‘ I have found these related issues/pull requests

No issues appear directly related; https://github.com/louislam/uptime-kuma/issues?q=label%3Aarea%3Astatus-page are related in being about status pages more generally.

πŸ›‘οΈ Security Policy

Description

When a query is made to a status page API endpoint for a status page which doesn't exist, the server sends no response.

πŸ‘Ÿ Reproduction steps

  1. When a status page named foobar does not exist, curl http://uptime/api/status-page/foobar.

πŸ‘€ Expected behavior

HTTP 404 would seem logical, since the requested resource doesn't exist. Alternatively, http://uptime/api/status-page/heartbeat/foobar returns 200 with an empty list of monitors, which would also be reasonable.

πŸ˜“ Actual Behavior

Request hangs indefinitely.

🐻 Uptime-Kuma Version

1.23.11

πŸ’» Operating System and Arch

Ubuntu 22.04 x64

🌐 Browser

Curl

πŸ–₯️ Deployment Environment

Deployed in Docker

πŸ“ Relevant log output

No log output related to this request.
@MikeBishop MikeBishop added the bug Something isn't working label Feb 27, 2024
@chakflying chakflying added area:status-page Everything related to the status page area:api concearning the api or automation labels Feb 27, 2024
@CommanderStorm CommanderStorm added help wanted May need your help to test or answer good first issue Good for newcomers labels Feb 27, 2024
@chakflying chakflying removed help wanted May need your help to test or answer good first issue Good for newcomers labels Feb 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:api concearning the api or automation area:status-page Everything related to the status page bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants