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

Add support for basic authentication for the web server. #366

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Add support for basic authentication for the web server. #366

wants to merge 3 commits into from

Conversation

psa-jforestier
Copy link

For my personal use, I added the "basic-auth" authentication mecasism in RPi-Monitor. When enabled, if you go to the web interface, you will be prompted for a user and password (HTTP 401 response). If the user/pass match the one in config file, you will get access to the ressource. Otherwise, you will get a new HTTP 401 response requesting a user/pass.
All browsers support "basic" auth, where the user and pass is send to the server in plain text (base64 encoded), so it is not suitable if you dont trust the network.

@XavierBerger
Copy link
Owner

Thanks for the pull request. I need to validate it before merging but this is a nice feature for RPi-Monitor.

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

Successfully merging this pull request may close these issues.

2 participants