Skip to content

fix(files): authorize metadata with internal user ids - #217

Merged
ralyodio merged 1 commit into
profullstack:masterfrom
rissrice2105-agent:fix/file-metadata-internal-user
Jul 31, 2026
Merged

fix(files): authorize metadata with internal user ids#217
ralyodio merged 1 commit into
profullstack:masterfrom
rissrice2105-agent:fix/file-metadata-internal-user

Conversation

@rissrice2105-agent

Copy link
Copy Markdown
Contributor

Summary

  • resolve the internal profile ID before authorizing HEAD and POST /api/files/:fileId
  • traverse the valid messages -> conversations -> conversation_participants relationship used by the working download handler
  • preserve ownership detection for records written with either auth UUIDs or legacy internal IDs

Tests

  • all file-route tests: 21 passed across 5 files
  • ESLint on both changed files
  • git diff --check

Local tooling note

The repository's default Vitest config currently fails before collection because locked @vitejs/plugin-react@6 imports an unexported Vite 7 subpath. The non-JSX file-route tests were run with the same environment, setup file, and aliases without that plugin. The configured Prettier plugin is also not declared locally; no dependency files were changed.

Closes #216

@ralyodio
ralyodio merged commit 67c7291 into profullstack:master Jul 31, 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.

File metadata endpoints authorize with the wrong user ID

2 participants