Skip to content

Reject whitespace-only auth passwords#4187

Open
bozicovichsantiago20-oss wants to merge 1 commit into
SecureBananaLabs:mainfrom
bozicovichsantiago20-oss:codex/reject-whitespace-passwords
Open

Reject whitespace-only auth passwords#4187
bozicovichsantiago20-oss wants to merge 1 commit into
SecureBananaLabs:mainfrom
bozicovichsantiago20-oss:codex/reject-whitespace-passwords

Conversation

@bozicovichsantiago20-oss

Copy link
Copy Markdown

Summary

  • Add a shared auth password validator requiring at least eight non-whitespace characters.
  • Apply it to both registration and login schemas without trimming or rewriting valid submitted values.
  • Add focused validator tests for whitespace-only and mixed-space passwords.
  • Update the API test script to target test files so the package test command runs correctly.

Verification

  • npm.cmd test -w apps/api
  • node --test apps/api/src/tests/auth.test.js
  • git diff --check

Closes #4186
Refs #743
/claim #4186

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Auth validators should reject whitespace-only passwords

1 participant