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

Fix/use hashed password login #258

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from

Conversation

thejamesgore
Copy link

  • Replaced clear text password comparison with hashed password comparison using bcrypt.
  • Enhanced security by avoiding the storage and comparison of plain text passwords.

Fixes issue #255.

thejamesgore and others added 5 commits June 21, 2024 02:16
- Replaced clear text password comparison with hashed password comparison using bcrypt.
- Updated environment variable to store the hashed password instead of the clear text password.
- Enhanced security by avoiding the storage and comparison of plain text passwords.

Fixes issue fusion44#255.
@fusion44
Copy link
Owner

fusion44 commented Jun 27, 2024

Thank you! I'll test and merge it the coming weekend.

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