Skip to content

Conversation

@picatz
Copy link
Contributor

@picatz picatz commented Nov 4, 2025

Summary

  • .github/workflows/create-tag.yml: Set both checkout steps to disable credential persistence while continuing to use the generated app token.
  • .github/workflows/flaky-tests-report.yml: Checkout now uses the generated app token and turns off persistent credentials.
  • .github/workflows/trigger-publish.yml: Added explicit Temporal organization owner and repository scope so the app token can dispatch to docker-builds.
  • .github/workflows/trigger-version-info-service.yml: Formatted the repository scope as a newline-delimited list to match the updated token requirements.

Previous summary:

Summary

  • .github/workflows/trigger-publish.yml: Swapped tibdex/github-app-token for actions/create-github-app-token@v2 and translated inputs to new kebab-case fields to keep workflow functional and remove deprecated action usage.

This change was made by an automated process to ensure all GitHub Actions workflows use the official GitHub application token action.

Assisted-by: GPT-5 Codex via Camper
@picatz picatz requested review from a team as code owners November 4, 2025 16:50
@semgrep-managed-scans
Copy link

Semgrep found 1 missing-explicit-permissions finding:

No explicit GITHUB_TOKEN permissions found at the workflow or job level. Add a permissions: block at the workflow root (applies to all jobs) or per job with least privilege (e.g., contents: read and only specific writes like pull-requests: write if needed).

@picatz
Copy link
Contributor Author

picatz commented Nov 4, 2025

☝️ Should be resolved with #8565

This refresh aligns owner/repositories inputs with actions/create-github-app-token@v2 defaults while preserving necessary repository access.

Assisted-by: GPT-5 Codex via Camper
@semgrep-managed-scans

This comment was marked as duplicate.

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