Skip to content

Commit 9bef291

Browse files
fix: pin strfry image and fix health check in CI
Pin strfry Docker image to a specific digest to satisfy zizmor's unpinned-images check. Replace broken health check (curl -sf fails on HTTP 426 Upgrade Required) with status code check that accepts any non-zero HTTP response as "ready". Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 5c29315 commit 9bef291

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

.github/workflows/ci.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ jobs:
105105
contents: read
106106
services:
107107
strfry:
108-
image: ghcr.io/hoytech/strfry:latest
108+
image: ghcr.io/hoytech/strfry@sha256:7cf3b97c66a8c3087e16e5d31a3bb8eb92312f08dd969ff3dced9625242ac55f # latest
109109
ports:
110110
- 7777:7777
111111
steps:
@@ -125,9 +125,9 @@ jobs:
125125
- name: Wait for strfry
126126
run: |
127127
for i in $(seq 1 30); do
128-
if curl -sf -o /dev/null http://localhost:7777 2>/dev/null || \
129-
curl -sf -o /dev/null -w '%{http_code}' http://localhost:7777 2>/dev/null | grep -qE '4[0-9]{2}'; then
130-
echo "strfry is ready"
128+
STATUS=$(curl -s -o /dev/null -w '%{http_code}' http://localhost:7777 2>/dev/null || true)
129+
if [ "$STATUS" != "000" ] && [ -n "$STATUS" ]; then
130+
echo "strfry is ready (HTTP $STATUS)"
131131
exit 0
132132
fi
133133
echo "Waiting for strfry... ($i/30)"

0 commit comments

Comments
 (0)