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

Enhancement: Include user session cookies when performing monitor requests #323

Open
2q2code opened this issue Jan 24, 2025 · 3 comments
Open

Comments

@2q2code
Copy link
Contributor

2q2code commented Jan 24, 2025

At present, all monitor requests are performed on the server-side. It may be beneficial to do this from the client side, as it would automatically handle inclusion of user session cookies for secure services.

I secure most of my services behind Authelia, so that means that most of the time, the response for the monitor widget is "Unauthorised". Even if I am logged in on the client browser. I have considered other options for this - perhaps allowing access to each site's favicon unrestricted - but not all sites have a usable favicon, either.

Might need to turn this into a discussion?

@svilenmarkov
Copy link
Member

alt-status-codes was implemented in #240 for this exact reason. It's not out yet, but it should work in your case since a 401 is technically a "working state" and you'd still be able to catch when a service goes down. Alternatively, there will also be #261 as an option so you wouldn't need to deal with HTTP requests.

@2q2code
Copy link
Contributor Author

2q2code commented Jan 26, 2025 via email

@2q2code
Copy link
Contributor Author

2q2code commented Jan 26, 2025 via email

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

2 participants