Skip to content

Remove Cloudflare Worker image proxy#415

Draft
mohanadft wants to merge 2 commits intomainfrom
remove-cloudflare-worker
Draft

Remove Cloudflare Worker image proxy#415
mohanadft wants to merge 2 commits intomainfrom
remove-cloudflare-worker

Conversation

@mohanadft
Copy link
Copy Markdown
Collaborator

@mohanadft mohanadft commented Apr 9, 2026

Summary

  • Deletes cloudflare-worker/ and src/utils/imageProxy.ts
  • Removes proxy URL construction and cache-busting logic from notionClient.ts
  • Simplifies Events.tsx image error handler to a direct fallback to /images/default.jpg
  • Cleans up NOTION_IMAGE_PROXY_URL from .env.example, wrangler.toml, and package.json (wrangler devDep removed)
  • Updates all docs (CLAUDE.md, AGENTS.md, DEPLOYMENT.md, TECHNICAL_OVERVIEW.md, docs/EVENTS.md)

Note: Notion-hosted images will now be served directly. They may expire after ~1 hour as Notion S3 URLs are time-limited, falling back to /images/default.jpg on error.

Test plan

  • pnpm build passes (verified locally)
  • Events page loads and images render
  • Image error fallback shows default image

Notion images are now served directly. Removes the worker, the
imageProxy utility, all proxy URL construction logic in notionClient.ts,
and the proxy-aware error handler in Events.tsx. Cleans up env vars,
wrangler config, package.json (wrangler devDep), and all docs.
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Apr 9, 2026

Deploying website with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1613a03
Status: ✅  Deploy successful!
Preview URL: https://9c6e28c7.website-aun.pages.dev
Branch Preview URL: https://remove-cloudflare-worker.website-aun.pages.dev

View logs

@mohanadft mohanadft marked this pull request as draft April 9, 2026 12:27
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