Commit e0f886e
authored
fix(ci): surface refresh-baselines HTTP error body and status code (#161)
* fix(ci): surface refresh-baselines HTTP error body and status code
Replace `curl -s -f` (silent + fail-no-body) with explicit HTTP code
capture and response body dump. Workflow has been failing every 6h
with exit 22 (HTTP >=400) but root cause was hidden. Next failure
will print HTTP status + Supabase error message in logs.
* fix(ci): harden refresh-baselines step against curl/HTTP edge cases
Address CodeRabbit review feedback on PR #161:
- Add EXIT trap to clean up mktemp response file
- Capture curl exit code; surface clear error when curl fails before
producing an HTTP code (DNS, TLS, connection reset)
- Validate http_code is numeric before integer comparison to avoid
obscure "integer expression expected" failure
- Only dump response body on HTTP >=400 to avoid noisy logs and
reduce risk of leaking verbose payloads on success1 parent c159293 commit e0f886e
1 file changed
Lines changed: 22 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
14 | 14 | | |
15 | 15 | | |
16 | 16 | | |
17 | | - | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
18 | 23 | | |
19 | 24 | | |
20 | 25 | | |
21 | | - | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
0 commit comments