Skip to content

Commit 4083a86

Browse files
committed
Use simple connectivity check for startup health probe
Older spiceio binaries may not set the Server header at all. The startup health check only needs to confirm the process we just started is responding — drop the Server header requirement and use a plain GET.
1 parent 7ba1269 commit 4083a86

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.github/actions/setup/action.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ runs:
160160
exit 1
161161
fi
162162
ENDPOINT=$(grep 'listening on' "$SPICEIO_LOG" 2>/dev/null | grep -o 'http://[^ ]*' | tail -1 || true)
163-
if [[ -n "$ENDPOINT" ]] && curl -s -I "$ENDPOINT/" 2>/dev/null | grep -qi "server: spiceio"; then
163+
if [[ -n "$ENDPOINT" ]] && curl -s -o /dev/null "$ENDPOINT/" 2>/dev/null; then
164164
echo "spiceio ready at $ENDPOINT (PID $PID)"
165165
echo "endpoint=$ENDPOINT" >> "$GITHUB_OUTPUT"
166166
exit 0

0 commit comments

Comments
 (0)