Skip to content

changelog: serve aus-mobile.mp4 from middlecache - #32588

Open
mvvmm wants to merge 1 commit into
productionfrom
aus-mobile-middlecache
Open

changelog: serve aus-mobile.mp4 from middlecache#32588
mvvmm wants to merge 1 commit into
productionfrom
aus-mobile-middlecache

Conversation

@mvvmm

@mvvmm mvvmm commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

What

Updates the Media Transformations changelog to reference the aus-mobile.mp4 video 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 the aus_mobile_video pipeline in middlecache-housekeeping.

Changes

  • src/content/changelog/stream/2025-03-06-media-transformations.mdx
    • <video> source URL: pub-d9fcbc1abcd244c1821f38b99017347f.r2.dev/aus-mobile.mp4middlecache.ced.cloudflare.com/v1/aus-mobile/aus-mobile.mp4
    • Example URL in code block: same replacement

Replace external R2 public bucket URL with middlecache URL for the
Media Transformations changelog video asset.
@mvvmm
mvvmm requested review from a team and hoan-pom as code owners August 6, 2026 18:47
@cloudflare-docs-bot

cloudflare-docs-bot Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Review

⚠️ Rebase: AI conflict resolution stopped — confidence not high enough to auto-resolve.

Could not identify specific conflicting files. Please resolve manually.
⚠️ 2 warnings found in commit b3b5e39.

👉 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 Review

This code review is in beta and may not always be helpful — use your judgment.

Warnings (2)
File Issue
changelog/stream/2025-03-06-media-transformations.mdx line 37 Internal-only origin used in public-facing content — The embedded
changelog/stream/2025-03-06-media-transformations.mdx line 42 Internal-only origin used in public-facing content — 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.

Conventions

No convention issues found.

Style Guide Review

No style-guide issues found.

Commands

Only codeowners can run commands. Post a comment with the command to trigger it.

Command Description
/review Runs a review now. Incremental if a prior review exists, full if not.
/full-review Re-reviews the entire PR diff from scratch, ignoring incremental history. Useful after a rebase, when you want a fresh review, or if the bot gets out of sync and reports issues that no longer exist.
/ignore-review-limit Permanently lifts the 2-review automatic limit for this PR. Future pushes will trigger reviews as normal.
/disable-auto-review Stops automatic reviews from triggering on future pushes to this PR. Codeowners can still run /review or /full-review manually.
/rebase Rebases the PR branch against production. On conflict, attempts to resolve automatically using AI. Stops with an explanation if confidence is not high enough.

@mvvmm

mvvmm commented Aug 6, 2026

Copy link
Copy Markdown
Contributor Author

Skipping CR-5b20563edd67 and CR-833b8cc59bf0

Both findings flag middlecache.ced.cloudflare.com as an internal-only origin. It is not — the domain is a public Cloudflare custom domain on the middlecache R2 bucket and is publicly accessible:

$ curl -sIL https://middlecache.ced.cloudflare.com/v1/aus-mobile/aus-mobile.mp4
HTTP/2 200
content-type: video/mp4
content-length: 28978866

The ced subdomain references the Content Engineering team that owns the bucket, but the URL is stable and served to any public reader. If a cleaner public domain name is desired, that is a separate infrastructure task (new R2 custom domain) outside the scope of this content change.

@mvvmm

mvvmm commented Aug 6, 2026

Copy link
Copy Markdown
Contributor Author

/rebase

1 similar comment
@mvvmm

mvvmm commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

/rebase

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.

2 participants