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

[FEATURE_REQUEST] Limit Redirect in Status Check #494

Closed
EVOTk opened this issue Feb 18, 2022 · 3 comments
Closed

[FEATURE_REQUEST] Limit Redirect in Status Check #494

EVOTk opened this issue Feb 18, 2022 · 3 comments
Labels
🦄 Feature Request [ISSUE] Suggestion for new feature, update or change 📌 Keep Open [ISSUE][PR] Prevent auto-closing

Comments

@EVOTk
Copy link
Contributor

EVOTk commented Feb 18, 2022

Is your feature request related to a problem? If so, please describe.

Hello,
I use the StatusCheck to check the availability of my services.
But for some of these services, when it is not available, my reverse proxy automatically redirects visitors to my status page ( uptime kuma ). Because of this, even if the service is offline, Dashy does not detect it!

Describe the solution you'd like

Would it be possible to add an option to configure the maximum number of redirects before considering the service as offline?

Thank you

Priority

Low (Nice-to-have)

Is this something you would be keen to implement

No

@EVOTk EVOTk added the 🦄 Feature Request [ISSUE] Suggestion for new feature, update or change label Feb 18, 2022
@Lissy93
Copy link
Owner

Lissy93 commented Feb 19, 2022

Yes, this is possible, and a good idea :)
Note to self: In Axios has a config item called maxRedirects which will do this.

@liss-bot
Copy link
Collaborator

This issue has gone 6 weeks without an update. To keep the ticket open, please indicate that it is still relevant in a comment below. Otherwise it will be closed in 5 working days.

@liss-bot liss-bot added the ⚰️ Stale [ISSUE] [PR] No activity for over 1 month label Mar 22, 2022
@Lissy93 Lissy93 added 📌 Keep Open [ISSUE][PR] Prevent auto-closing and removed ⚰️ Stale [ISSUE] [PR] No activity for over 1 month labels Mar 22, 2022
@Lissy93
Copy link
Owner

Lissy93 commented Mar 22, 2022

Keep open, has been implemented, just waiting merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🦄 Feature Request [ISSUE] Suggestion for new feature, update or change 📌 Keep Open [ISSUE][PR] Prevent auto-closing
Projects
None yet
Development

No branches or pull requests

3 participants