Skip to content

Fix legacy eVault File dereference - #1138

Open
serobkhachatryan-cpu wants to merge 3 commits into
MetaState-Prototype-Project:mainfrom
serobkhachatryan-cpu:fix/legacy-evault-file-dereference
Open

serobkhachatryan-cpu wants to merge 3 commits into
MetaState-Prototype-Project:mainfrom
serobkhachatryan-cpu:fix/legacy-evault-file-dereference

Conversation

@serobkhachatryan-cpu

Copy link
Copy Markdown

Fixes the native /files/:metaEnvelopeId fast path for the documented legacy File-record ontology (a1b2c3d4-e5f6-7890-abcd-ef1234567890).

The route keeps eName-scoped lookup and HTTP(S)-only redirect validation, prefers publicUrl, and uses url only for that legacy ontology. This prevents valid shared recordings from falling back to the slower GraphQL metadata path.

Validated with focused Fastify injection tests (6 passing) and pnpm --filter evault-core build.

@coderabbitai

coderabbitai Bot commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: cd423b99-8c78-486a-be81-6ec3b9a2ab50


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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