Skip to content

feat: bcrypt password hashing in authService - #11560

Open
0xhermes-28 wants to merge 5 commits into
SecureBananaLabs:mainfrom
0xhermes-28:11559-bcrypt
Open

feat: bcrypt password hashing in authService#11560
0xhermes-28 wants to merge 5 commits into
SecureBananaLabs:mainfrom
0xhermes-28:11559-bcrypt

Conversation

@0xhermes-28

Copy link
Copy Markdown

Fixes #11559

  • registerUser hashes password with bcryptjs (12 rounds), stores in-memory map
  • loginUser verifies password against hash, throws "Invalid email or password"
  • refreshToken(req.user.sub, req.user.role) uses verified identity
  • Added bcryptjs dependency + authService tests (register, login, wrong pass, not found)
    /bounty $50

github-actions Bot added a commit that referenced this pull request Aug 2, 2026
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