-
-
Notifications
You must be signed in to change notification settings - Fork 6
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 authentication to the administration area. #4
Comments
Maybe not? Lets not give people a false sense of security. |
Okay, let's add some basic auth for management. |
I keep yo-yo-ing on this. Basic auth is awful, but proper auth is complicated. Maybe later. |
This can easily be done with both Apache and Nginx. At least, that's what I did. I see no reason to include this as an enhancement. |
Maybe we could just add a snippet on securing that with rate limiting and call it implemented. |
That sounds good to me. Maybe even throw in a mock .htaccess and .htpasswd file for those using Apache. While you're at it, you could also cover how to use rewrite so instead of https://domain.tld/announce.php you could use https://domain.tld/announce since that's how most other trackers look. |
Provide some basic password security.
Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
The text was updated successfully, but these errors were encountered: