Skip to content

fix(users): trim profile username params - #174

Merged
ralyodio merged 1 commit into
profullstack:masterfrom
aiirvizionz:clientkit/by-username-trim
Jul 30, 2026
Merged

fix(users): trim profile username params#174
ralyodio merged 1 commit into
profullstack:masterfrom
aiirvizionz:clientkit/by-username-trim

Conversation

@aiirvizionz

Copy link
Copy Markdown
Contributor

Summary

  • trim and lowercase public profile username route params before lookup
  • reject whitespace-only usernames before database work
  • update route tests to cover trimming and blank username handling

Tests

  • node node_modules\vitest\vitest.mjs run --config $env:TEMP\qryptchat-vitest-minimal.config.mjs "src/app/api/users/by-username/[username]/route.test.js"
  • git diff --check

Note: I used the minimal Vitest config already present locally because the repo's default Vitest setup imports @vitejs/plugin-react, which currently fails against the installed Vite package export map in this checkout.

@ralyodio
ralyodio merged commit ddbe70d into profullstack:master Jul 30, 2026
8 checks passed
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