Skip to content

security: fix DUMMY_HASH, timing jitter, and sub-token scan limit#25

Open
mosobande wants to merge 1 commit intoorifrom
claude/security-audit-ogiri-3kbEb
Open

security: fix DUMMY_HASH, timing jitter, and sub-token scan limit#25
mosobande wants to merge 1 commit intoorifrom
claude/security-audit-ogiri-3kbEb

Conversation

@mosobande
Copy link
Collaborator

  • Replace invalid 36-char DUMMY_HASH with valid 60-char BCrypt hash to
    prevent user enumeration via timing side-channel
  • Replace fixed 100ms Thread.sleep floor with randomized 80-120ms jitter
    to improve timing normalization and reduce thread exhaustion risk
  • Limit sub-token validation fallback scan to maxClients entries to
    bound worst-case BCrypt comparisons
  • Remove SECURITY-AUDIT.md
  • Update security and performance docs to reflect changes

- Replace invalid 36-char DUMMY_HASH with valid 60-char BCrypt hash to
  prevent user enumeration via timing side-channel
- Replace fixed 100ms Thread.sleep floor with randomized 80-120ms jitter
  to improve timing normalization and reduce thread exhaustion risk
- Limit sub-token validation fallback scan to maxClients entries to
  bound worst-case BCrypt comparisons
- Remove SECURITY-AUDIT.md
- Update security and performance docs to reflect changes
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