Skip to content

ECC-8262: adopt OAuth mirror workflow pattern#7

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

ECC-8262: adopt OAuth mirror workflow pattern#7
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:26
Copy link
Copy Markdown

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

This PR updates the repository-mirroring GitHub Actions workflow to follow the OAuth-based “mirror workflow” pattern (as described in the PR metadata), adding scheduled runs, run concurrency controls, and restricting default token permissions.

Changes:

  • Replace static Bitbucket access token usage with an OAuth client-credentials flow to fetch a short-lived access token at runtime.
  • Add a daily scheduled run (cron) alongside push-triggered runs.
  • Add a concurrency group to cancel redundant runs and set permissions: 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