Skip to content

fix(files): reject blank message file ids - #166

Closed
aiirvizionz wants to merge 1 commit into
profullstack:masterfrom
aiirvizionz:clientkit/message-files-id-guard
Closed

fix(files): reject blank message file ids#166
aiirvizionz wants to merge 1 commit into
profullstack:masterfrom
aiirvizionz:clientkit/message-files-id-guard

Conversation

@aiirvizionz

Copy link
Copy Markdown
Contributor

Summary

  • Normalize messageId in the message-files API route before validation and database filtering.
  • Reject blank/whitespace-only message ids before doing auth or Supabase work.
  • Add regression coverage for whitespace-only route params.

Validation

  • git diff --check

I attempted pnpm vitest run src/app/api/files/message/[messageId]/route.test.js, but this Windows checkout stops during dependency preparation because the package postinstall script uses POSIX shell commands (command -v ... || true). I also attempted direct Vitest execution with node node_modules\vitest\vitest.mjs run "src/app/api/files/message/[messageId]/route.test.js", but the local Vitest config fails to load because @vitejs/plugin-react imports vite/internal, which is not exported by the installed Vite package. I did not modify dependency files.

@ralyodio ralyodio closed this Jul 30, 2026
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