Skip to content

Refactor password auth to use session secret and update docs#25

Merged
BackGwa merged 1 commit intodevelopfrom
refactor/remove-password-hash
Jan 13, 2026
Merged

Refactor password auth to use session secret and update docs#25
BackGwa merged 1 commit intodevelopfrom
refactor/remove-password-hash

Conversation

@BackGwa
Copy link
Owner

@BackGwa BackGwa commented Jan 13, 2026

Description

Clarifies the password verification flow by removing the SHA-256 pre-hash step, treating the password as the session secret used for HMAC, and updating naming to reflect this intent. Documentation has been aligned with the updated flow.

Type of Change

  • Bug fix (bugs/)
  • New feature (features/)
  • Refactoring (refactor/)
  • Hotfix (hotfix/)
  • Chore (chore/)

Changes Made

  • Rename session password handling to session secret across context and hooks
  • Remove SHA-256 pre-hash step. HMAC now uses the raw password/secret
  • Update architecture diagrams and flow descriptions to match the new semantics

Checklist

  • Code follows the project's coding style
  • Self-review completed
  • Documentation updated (if necessary)

- Updated ARCHITECTURE.md for password terminology changes.
- Replaced `sessionPassword` with `sessionSecret` across the app.
- Merged hashing and HMAC utilities into passwordCrypto.
- Removed password hashing in favor of direct secret use.
- Simplified password handling in StreamContext, hooks, and utils.
@BackGwa BackGwa added documentation Improvements or additions to documentation enhancement New feature or request labels Jan 13, 2026
@BackGwa BackGwa merged commit 3a336c3 into develop Jan 13, 2026
4 checks passed
@BackGwa BackGwa deleted the refactor/remove-password-hash branch January 13, 2026 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant