Skip to content

Fix drop-release auth to use git config extraheader — Closes #129#130

Merged
conradbzura merged 1 commit intowool-labs:masterfrom
conradbzura:129-fix-drop-release-auth
Mar 27, 2026
Merged

Fix drop-release auth to use git config extraheader — Closes #129#130
conradbzura merged 1 commit intowool-labs:masterfrom
conradbzura:129-fix-drop-release-auth

Conversation

@conradbzura
Copy link
Copy Markdown
Contributor

@conradbzura conradbzura commented Mar 27, 2026

Summary

Replace the git remote set-url authentication in the drop-release job with the git config --local http.extraheader approach that cut-release.yaml already uses successfully. The drop-release step was failing to authenticate when deleting the release branch, and this may fix it.

Closes #129

Proposed changes

Switch drop-release auth method (publish-release.yaml)

The drop-release job authenticated by embedding the app token in the remote URL via git remote set-url, which was failing. Replace it with git config --local http.extraheader, which sets an AUTHORIZATION header with a base64-encoded token — the same approach used in cut-release.yaml:71.

Before:

git remote set-url origin "https://x-access-token:${GH_TOKEN}@github.com/${REPO}.git"

After:

git config --local http.https://github.com/.extraheader "AUTHORIZATION: basic $(echo -n "x-access-token:${GH_TOKEN}" | base64)"

The drop-release step was authenticating via git remote set-url with
an embedded token, which fails for GitHub App tokens. Switch to the
git config http.extraheader approach that cut-release already uses
successfully.
@conradbzura conradbzura marked this pull request as ready for review March 27, 2026 01:54
@conradbzura conradbzura self-assigned this Mar 27, 2026
@conradbzura conradbzura added bug Something isn't working cicd CI/CD pipeline changes labels Mar 27, 2026
@conradbzura conradbzura merged commit 9255fcd into wool-labs:master Mar 27, 2026
4 checks passed
@conradbzura conradbzura linked an issue Mar 27, 2026 that may be closed by this pull request
@conradbzura conradbzura removed the cicd CI/CD pipeline changes label Mar 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix drop-release auth to use git config extraheader

1 participant