Skip to content

Commit de19c1b

Browse files
authored
fix(ci): smoke test accepts release-assets CDN hostname (#509)
v2.0.4 rerun of update-apt-repo made it past hops 0 and 1 (the smoke test scheme fix in #506 worked — Pages' http:// redirect no longer trips the chain walker), but failed on hop 2: Hop 2: 302 .../releases/download/v2.0.4+claude1.3883.0/...deb -> https://release-assets.githubusercontent.com/... ::error::Hop 2 mismatch: expected https://objects\.githubusercontent\.com/, got https://release-assets.githubusercontent.com/... GitHub migrated the Release asset CDN from objects.githubusercontent.com to release-assets.githubusercontent.com (both have been serving in the past; release-assets is the current canonical hostname). Accept either hostname via alternation. Verified against the actual v2.0.4 Release: $ curl -Is https://github.com/aaddrick/claude-desktop-debian/releases/download/v2.0.4+claude1.3883.0/claude-desktop_1.3883.0-2.0.4_amd64.deb \ | grep -i location location: https://release-assets.githubusercontent.com/github-production-release-asset/... Same fix in three sites: - .github/workflows/ci.yml (update-apt-repo smoke test) - .github/workflows/ci.yml (update-dnf-repo smoke test) - .github/workflows/apt-repo-heartbeat.yml (daily heartbeat) docs/worker-apt-plan.md has historical references to objects.githubusercontent.com too; those can be updated in a follow-up docs sweep — the architectural claim (binary bytes flow direct from GitHub CDN, never through Cloudflare) is unchanged. Refs #493, #503
1 parent 0319c1d commit de19c1b

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

.github/workflows/apt-repo-heartbeat.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ jobs:
9191
expected_hops=(
9292
"https?://${WORKER_DOMAIN}/"
9393
"https://github\\.com/aaddrick/claude-desktop-debian/releases/download/"
94-
"https://objects\\.githubusercontent\\.com/"
94+
"https://(objects|release-assets)\\.githubusercontent\\.com/"
9595
)
9696
url="$URL"
9797
for i in "${!expected_hops[@]}"; do

.github/workflows/ci.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -537,7 +537,7 @@ jobs:
537537
expected_hops=(
538538
"https?://${WORKER_DOMAIN}/"
539539
"https://github\\.com/aaddrick/claude-desktop-debian/releases/download/v${repoVer}\\+claude${claudeVer}/"
540-
"https://objects\\.githubusercontent\\.com/"
540+
"https://(objects|release-assets)\\.githubusercontent\\.com/"
541541
)
542542
url="$deb_url"
543543
for i in "${!expected_hops[@]}"; do
@@ -751,7 +751,7 @@ jobs:
751751
expected_hops=(
752752
"https?://${WORKER_DOMAIN}/"
753753
"https://github\\.com/aaddrick/claude-desktop-debian/releases/download/v${repoVer}\\+claude${claudeVer}/"
754-
"https://objects\\.githubusercontent\\.com/"
754+
"https://(objects|release-assets)\\.githubusercontent\\.com/"
755755
)
756756
url="$rpm_url"
757757
for i in "${!expected_hops[@]}"; do

0 commit comments

Comments
 (0)