Skip to content

@remotion/media: add <Video crossOrigin> for parser fetch credentials#6838

Merged
JonnyBurger merged 4 commits intoremotion-dev:mainfrom
alec-watts:alec/video-crossorigin-mediabunny-credentials
Mar 17, 2026
Merged

@remotion/media: add <Video crossOrigin> for parser fetch credentials#6838
JonnyBurger merged 4 commits intoremotion-dev:mainfrom
alec-watts:alec/video-crossorigin-mediabunny-credentials

Conversation

@alec-watts
Copy link
Copy Markdown

Summary

  • add a top-level crossOrigin prop to @remotion/media Video props
  • thread the prop through Video -> VideoForPreview -> MediaPlayer
  • map crossOrigin to Mediabunny UrlSource requestInit.credentials (use-credentials -> include, anonymous/empty -> same-origin)
  • keep default behavior unchanged when crossOrigin is not provided

Test plan

  • load <Video src=... crossOrigin=\"use-credentials\" /> against a cookie-protected media URL and confirm format detection no longer 401s
  • verify <Video src=... /> without crossOrigin preserves existing behavior
  • verify fallback path (fallbackOffthreadVideoProps.crossOrigin) continues to work as before

Made with Cursor

Thread a top-level crossOrigin prop through Video preview playback and map it to UrlSource request credentials so consumers can opt into cookie-backed fetches without forcing credentials for all requests.

Made-with: Cursor
@pullfrog
Copy link
Copy Markdown
Contributor

pullfrog bot commented Mar 13, 2026

no API key found. Pullfrog requires at least one LLM provider API key.

to fix this, add the required secret to your GitHub repository:

  1. go to: https://github.com/remotion-dev/remotion/settings/secrets/actions
  2. click "New repository secret"
  3. set the name to your provider's key (e.g., ANTHROPIC_API_KEY, OPENAI_API_KEY, GEMINI_API_KEY)
  4. set the value to your API key
  5. click "Add secret"

configure your model at https://pullfrog.com/console/remotion-dev/remotion

Pullfrog  | Rerun failed job ➔View workflow run | Triggered by Pullfrogpullfrog.com𝕏

@vercel
Copy link
Copy Markdown
Contributor

vercel bot commented Mar 13, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
bugs Ready Ready Preview, Comment Mar 17, 2026 10:54am
remotion Ready Ready Preview, Comment Mar 17, 2026 10:54am

Request Review

Replace the crossOrigin prop (which mapped HTML video element values to
fetch credentials) with a direct credentials: RequestCredentials prop.
Thread credentials through getSinks/getSink and document the new prop.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
… Audio

Pass credentials prop through extractFrame, extractAudio,
extractFrameAndAudio, extractFrameViaBroadcastChannel, and the
broadcast channel request type. Add credentials to AudioProps and
audio-for-preview. Add AvailableFrom tag to Video docs and add
credentials documentation for Audio.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@JonnyBurger JonnyBurger changed the title @remotion/media: add <Video crossOrigin> for parser fetch credentials @remotion/media: add <Video crossOrigin> for parser fetch credentials Mar 17, 2026
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@JonnyBurger JonnyBurger merged commit 70da1b9 into remotion-dev:main Mar 17, 2026
10 of 12 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.

3 participants