Skip to content

need to specify permissions#130

Merged
derekmisler merged 1 commit intodocker:mainfrom
derekmisler:need-to-specify-permissions
Apr 14, 2026
Merged

need to specify permissions#130
derekmisler merged 1 commit intodocker:mainfrom
derekmisler:need-to-specify-permissions

Conversation

@derekmisler
Copy link
Copy Markdown
Contributor

@derekmisler derekmisler commented Apr 14, 2026

Related Issues

Closes: https://github.com/docker/gordon/issues/387

Summary

GitHub excludes the workflows permission from App installation tokens by default, even when the App itself has that permission granted. This PR explicitly passes all required permissions when requesting the installation token in src/app-token.ts, ensuring the generated token includes workflows: write along with every other permission the App has.

Permissions requested

Scope Level Type
actions write Repository
checks write Repository
contents write Repository
issues write Repository
pull_requests write Repository
statuses write Repository
variables read Repository
workflows write Repository
members read Organization

Note

Passing permissions explicitly scopes the token down to only what's listed, so all App permissions must be included to avoid accidentally dropping any.


Tip

Comment /review to trigger the PR Reviewer agent for automated feedback.
Comment /describe to generate a PR description.

Signed-off-by: Derek Misler <derek.misler@docker.com>
@derekmisler derekmisler self-assigned this Apr 14, 2026
@derekmisler derekmisler requested a review from a team April 14, 2026 20:15
@derekmisler derekmisler marked this pull request as ready for review April 14, 2026 20:15
@derekmisler derekmisler merged commit 06b8403 into docker:main Apr 14, 2026
18 checks passed
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