Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

repo_token is not needed, GH provides a token #162

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

abates
Copy link
Contributor

@abates abates commented Sep 5, 2024

In the publish_gh job the svenstaro/upload-release-action@v2 action does not require a repo_token and will default to github.token (see the action README). The github.token is autogenerated during workflow runs (see the documentation) and will work just fine for publishing to releases.

I have updated this in the AWX Runner repo and it worked fine.

Copy link
Contributor

@cmsirbu cmsirbu left a comment

Choose a reason for hiding this comment

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

Thanks! Normally this would've been a quick merge, but we're planning on replacing this action completely - see #176.

In the meantime, I don't see any harm in this going in though... thoughts @gsnider2195 @smk4664 ?

@gsnider2195
Copy link
Contributor

This will work as long as nobody has changed the default github token permissions in the repo or org.

https://docs.github.com/en/actions/security-for-github-actions/security-guides/automatic-token-authentication#permissions-for-the-github_token

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.

3 participants