Skip to content

fix(conversations): reject blank join ids - #172

Merged
ralyodio merged 1 commit into
profullstack:masterfrom
aiirvizionz:clientkit/join-conversation-id-trim
Jul 30, 2026
Merged

fix(conversations): reject blank join ids#172
ralyodio merged 1 commit into
profullstack:masterfrom
aiirvizionz:clientkit/join-conversation-id-trim

Conversation

@aiirvizionz

Copy link
Copy Markdown
Contributor

Summary

  • Trim join conversationId values before validation.
  • Reject whitespace-only ids before user lookup or SSE room join.
  • Add regression coverage for blank ids and trimmed valid ids.

Validation

  • git diff --check
  • node node_modules\vitest\vitest.mjs run --config %TEMP%\qryptchat-vitest-minimal.config.mjs "src/app/api/conversations/join/route.test.js" (2 tests passed)

I used the same temporary minimal Vitest config needed locally because the repo's default Vitest config currently fails to load through the installed Vite/plugin combination; no dependency files were modified.

@ralyodio
ralyodio merged commit c90293a 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