Skip to content

ECC-8262: adopt OAuth mirror workflow pattern#36

Open
pisto wants to merge 1 commit into
masterfrom
ECC-8262
Open

ECC-8262: adopt OAuth mirror workflow pattern#36
pisto wants to merge 1 commit into
masterfrom
ECC-8262

Conversation

@pisto
Copy link
Copy Markdown

@pisto pisto commented May 26, 2026

Align mirror-repository.yaml with the canonical pattern from terraform-cloud-infra:

  • OAuth token via BITBUCKET_CLIENT_ID/BITBUCKET_CLIENT_SECRET instead of static BITBUCKET_ACCESS_TOKEN
  • Daily schedule cron
  • Concurrency group to cancel redundant runs
  • permissions: contents: read

Copilot AI review requested due to automatic review settings May 26, 2026 17:32
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the repository mirroring GitHub Actions workflow to match the canonical “OAuth mirror” pattern by switching from a static Bitbucket access token to an OAuth client-credentials flow and improving run control.

Changes:

  • Add a daily scheduled trigger alongside push events.
  • Add concurrency settings to cancel redundant in-progress runs.
  • Replace BITBUCKET_ACCESS_TOKEN usage with an OAuth token fetched at runtime via BITBUCKET_CLIENT_ID/BITBUCKET_CLIENT_SECRET, and scope workflow permissions to contents: read.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +26 to +39
run: |
token="$(
curl \
--silent \
--fail \
--request POST \
--user "${BITBUCKET_CLIENT_ID}:${BITBUCKET_CLIENT_SECRET}" \
--header "content-type: application/x-www-form-urlencoded" \
--data "grant_type=client_credentials" \
"https://bitbucket.org/site/oauth2/access_token" \
| jq --exit-status --raw-output '.access_token'
)"
echo "::add-mask::${token}"
printf 'token=%s\n' "${token}" >> "${GITHUB_OUTPUT}"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants