feat(caddy): support prerelease manifest redirects - #1
Conversation
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
![kilo-code-bot[bot]](https://avatars.githubusercontent.com/in/2193792?s=60&v=4){kind=small}
| run_test "$HOST" "Jellyfin-Server/10.11.0" "/manifest.json" "cdn:10.11" "10.11 → branch" | ||
| run_test "$HOST" "Jellyfin-Server/12.0.0" "/manifest.json" "cdn:12.0" "12.0 → branch" | ||
| run_test "$HOST" "Jellyfin-Server/10.9.2" "/" "literal:https://github.com/intro-skipper/" "Not /manifest.json → GitHub" | ||
| run_test "$HOST" "Jellyfin-Server/12.0.0" "/manifest.json" "cdn:12.0:manifest.json" "12.0 → branch" |
There was a problem hiding this comment.
SUGGESTION: 10.x routing is no longer covered by tests
This PR removes all 10.x redirect test cases (previously 10.8, 10.9, 10.10, 10.11), but docker/Caddyfile:35 still advertises and routes 10.x via the VER header regex (10\.\d+|12\.\d+). If 10.x routing regresses, CI will no longer catch it. Consider keeping at least one 10.x manifest case (e.g. cdn:10.11:manifest.json) so the live 10.x support stays verified.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (1 file)
Previous Review Summaries (2 snapshots, latest commit 044b084)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit 044b084)Status: 1 Issue Found | Recommendation: Address before merge Overview
Issue Details (click to expand)SUGGESTION
Files Reviewed (2 files)
Fix these issues in Kilo Cloud Previous review (commit 5c39d91)Status: 1 Issue Found | Recommendation: Address before merge Overview
Issue Details (click to expand)SUGGESTION
Files Reviewed (2 files)
Reviewed by step-3.7-flash · Input: 76.2K · Output: 2.5K · Cached: 170.5K |
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
2 participants
Adds
manifest-prerelease.jsonto the Caddy redirect flow so Jellyfin clients are sent to the matching versioned prerelease manifest on jsDelivr while non-manifest paths and non-Jellyfin user agents still fall back to GitHub.manifest.jsonandmanifest-prerelease.json, preserving the current CDN commit pinning behavior.