Skip to content

fix: --base-url breaks og:thumbnail generation path (marimo-team#8304)#8329

Open
mstanowski wants to merge 1 commit intomarimo-team:mainfrom
mstanowski:fix-8304
Open

fix: --base-url breaks og:thumbnail generation path (marimo-team#8304)#8329
mstanowski wants to merge 1 commit intomarimo-team:mainfrom
mstanowski:fix-8304

Conversation

@mstanowski
Copy link

📝 Summary

On a gallery page, the default card thumbnail was using an absolute /og/thumbnail path, which breaks thumbnail generation when the --base-url option is used in run mode.

Closes #8304

🔍 Test Plan

  • Open marimo in run mode, providing a directory and the --base-url argument.
  • Verify that there are no missing default thumbnails for notebooks.

On a gallery page, the default card thumbnail was using an absolute /og/thumbnail path, which breaks thumbnail generation when the --base-url option is used in run mode.
@vercel
Copy link

vercel bot commented Feb 16, 2026

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

Project Deployment Actions Updated (UTC)
marimo-docs Ready Ready Preview, Comment Feb 16, 2026 3:42pm

Request Review

@mstanowski
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

@github-actions
Copy link

github-actions bot commented Feb 16, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

? opengraphImage
: asURL(
`/og/thumbnail?file=${encodeURIComponent(relativePath)}`,
`og/thumbnail?file=${encodeURIComponent(relativePath)}`,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great catch @mstanowski, thank you!

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.

--base-url breaks og:thumbnail generation path

3 participants