changelog: serve aus-mobile.mp4 from middlecache - #32588
Conversation
Replace external R2 public bucket URL with middlecache URL for the Media Transformations changelog video asset.
Review
👉 Fix in your agent 👈Fix the following review findings in PR #32588 (https://github.com/cloudflare/cloudflare-docs/pull/32588).
Before making changes, review each finding and present a brief summary table:
- For each finding, state whether you agree, disagree, or need clarification
- If you disagree (e.g. the fix requires disproportionate effort for minimal benefit,
or the finding is factually incorrect), explain why
- If you need clarification before deciding, ask those questions
- Then share your plan for which issues to tackle and in what order
After triaging, follow this order:
1. Post a comment on this PR for any findings you are skipping, with the finding ID and your reasoning.
2. Then commit the fixes for the legitimate findings.
The comment must come before the commit — the bot reads PR comments when a new
push triggers a review, so skip comments posted after the push will be missed.
---
## Code Review
### Warnings (2)
#### CR-5b20563edd67 · Internal-only origin used in public-facing content
- **File:** `src/content/changelog/stream/2025-03-06-media-transformations.mdx` line 37
- **Issue:** The embedded <video> source and example URL point to https://middlecache.ced.cloudflare.com/v1/aus-mobile/aus-mobile.mp4. This is an internal Cloudflare domain, which may be unreachable for public readers and is not a stable public origin for documentation assets.
- **Fix:** Host the example MP4 on a public origin (e.g., in public/ or an R2/public bucket) or use a placeholder/example URL that readers can substitute with their own source video.
#### CR-833b8cc59bf0 · Internal-only origin used in public-facing content
- **File:** `src/content/changelog/stream/2025-03-06-media-transformations.mdx` line 42
- **Issue:** The example URL repeats the internal middlecache.ced.cloudflare.com origin, exposing the same public accessibility concern on another changed line.
- **Fix:** Use a public example source URL or a clear placeholder so the documented example remains reproducible for external users.
Code ReviewThis code review is in beta and may not always be helpful — use your judgment. Warnings (2)
ConventionsNo convention issues found. Style Guide ReviewNo style-guide issues found. CommandsOnly codeowners can run commands. Post a comment with the command to trigger it.
|
Skipping CR-5b20563edd67 and CR-833b8cc59bf0Both findings flag The |
|
/rebase |
1 similar comment
|
/rebase |
What
Updates the Media Transformations changelog to reference the
aus-mobile.mp4video from middlecache instead of the external R2 public bucket.Why
The video was previously served from
pub-d9fcbc1abcd244c1821f38b99017347f.r2.dev— an external R2 public bucket outside our control. It is now mirrored in the middlecache R2 bucket via theaus_mobile_videopipeline in middlecache-housekeeping.Changes
src/content/changelog/stream/2025-03-06-media-transformations.mdx<video>source URL:pub-d9fcbc1abcd244c1821f38b99017347f.r2.dev/aus-mobile.mp4→middlecache.ced.cloudflare.com/v1/aus-mobile/aus-mobile.mp4