Merge pull request 'Give the unclaim step a CA bundle' (#110) from fix/unclaim-ca-certs into main
Release / release (push) Successful in 31s
CI / e2e (push) Successful in 6m3s
CI / check (push) Successful in 2m22s
Build & publish the Android APK / apk (push) Successful in 1m24s
Build & publish Arch package / arch-package (push) Successful in 2m29s
Attach the desktop build to the release / linux (push) Successful in 52s
Sync Homebrew formula / sync-formula (push) Successful in 6s
Release / release (push) Successful in 31s
CI / e2e (push) Successful in 6m3s
CI / check (push) Successful in 2m22s
Build & publish the Android APK / apk (push) Successful in 1m24s
Build & publish Arch package / arch-package (push) Successful in 2m29s
Attach the desktop build to the release / linux (push) Successful in 52s
Sync Homebrew formula / sync-formula (push) Successful in 6s
Reviewed-on: #110
This commit was merged in pull request #110.
This commit is contained in:
@@ -58,8 +58,15 @@ jobs:
|
|||||||
run: |
|
run: |
|
||||||
set -euo pipefail
|
set -euo pipefail
|
||||||
|
|
||||||
|
# `ca-certificates` is named because `--no-install-recommends`
|
||||||
|
# skips it, and `ubuntu:24.04` ships no CA bundle of its own —
|
||||||
|
# so curl comes up unable to verify TLS against our own Gitea
|
||||||
|
# and fails with "error setting certificate file" (exit 77).
|
||||||
|
# Every other containerised workflow here spells it out for the
|
||||||
|
# same reason; this one did not, and cost a release cycle.
|
||||||
apt-get update -qq
|
apt-get update -qq
|
||||||
apt-get install -y -qq --no-install-recommends curl jq >/dev/null
|
apt-get install -y -qq --no-install-recommends \
|
||||||
|
ca-certificates curl jq >/dev/null
|
||||||
|
|
||||||
label_id=$(
|
label_id=$(
|
||||||
curl -sSf -H "Authorization: token $TOKEN" "$API/labels?limit=100" |
|
curl -sSf -H "Authorization: token $TOKEN" "$API/labels?limit=100" |
|
||||||
@@ -78,8 +85,14 @@ jobs:
|
|||||||
# label answers the same as one that did, which is what makes
|
# label answers the same as one that did, which is what makes
|
||||||
# this safe to run on *every* close rather than only the ones
|
# this safe to run on *every* close rather than only the ones
|
||||||
# that were claimed.
|
# that were claimed.
|
||||||
|
# The body is captured, not discarded, so a refusal is
|
||||||
|
# diagnosable from this log alone. Whether the automatic
|
||||||
|
# token carries issue-write scope is still unproven, and
|
||||||
|
# "DELETE returned 403" without Gitea's own sentence costs
|
||||||
|
# another merge to find out which of the two it is.
|
||||||
|
body=$(mktemp)
|
||||||
code=$(
|
code=$(
|
||||||
curl -sS -o /dev/null -w '%{http_code}' -X DELETE \
|
curl -sS -o "$body" -w '%{http_code}' -X DELETE \
|
||||||
-H "Authorization: token $TOKEN" \
|
-H "Authorization: token $TOKEN" \
|
||||||
"$API/issues/$ISSUE/labels/$label_id"
|
"$API/issues/$ISSUE/labels/$label_id"
|
||||||
)
|
)
|
||||||
@@ -88,6 +101,7 @@ jobs:
|
|||||||
204) echo "unclaim: #$ISSUE is closed and unclaimed" ;;
|
204) echo "unclaim: #$ISSUE is closed and unclaimed" ;;
|
||||||
*)
|
*)
|
||||||
echo "unclaim: DELETE returned $code for #$ISSUE" >&2
|
echo "unclaim: DELETE returned $code for #$ISSUE" >&2
|
||||||
|
cat "$body" >&2
|
||||||
exit 1
|
exit 1
|
||||||
;;
|
;;
|
||||||
esac
|
esac
|
||||||
|
|||||||
Reference in New Issue
Block a user