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

Authentication #61

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Authentication #61

wants to merge 6 commits into from

Conversation

nzjoel1234
Copy link

@nzjoel1234 nzjoel1234 commented May 22, 2016

Hi,

I have implemented a simple authentication scheme; for unauthenticated users index.php shows the LCD display and the Graph. Any request other than getting graph or LCD data is not be allowed if user is not logged in.

The password checking is very crude (a single username and password pair can be specified in user config file).

The lockout mechanism (after too many failed password attempts) is also crude. I save the number of failed attempts and the lock-out expiry time in a file (this file can be accessed by anyone but not edited).

Cheers

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.

1 participant