Skip to content

fix(files): trim message file ids - #176

Open
aiirvizionz wants to merge 1 commit into
profullstack:masterfrom
aiirvizionz:clientkit/message-file-id-trim
Open

fix(files): trim message file ids#176
aiirvizionz wants to merge 1 commit into
profullstack:masterfrom
aiirvizionz:clientkit/message-file-id-trim

Conversation

@aiirvizionz

Copy link
Copy Markdown
Contributor

Summary

  • trim message file route params before validation and lookup
  • reject whitespace-only message ids before authentication or database work
  • add focused route coverage for trimmed and blank message ids

Tests

  • node node_modules\vitest\vitest.mjs run --config $env:TEMP\qryptchat-vitest-minimal.config.mjs "src/app/api/files/message/[messageId]/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.

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.

1 participant