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

Set permissions: write-all in hosting.md #1819

Merged
merged 2 commits into from
May 15, 2022
Merged

Conversation

rikhuijzer
Copy link
Contributor

@rikhuijzer rikhuijzer commented May 15, 2022

GitHub seems to have changed the permissions for the secrets.GITHUB_TOKEN. It does not allow writing by default anymore. I've now had two new repositories where I copy-pasted some workflow and got the following back

[docs cdb3219] build based on 2be61e3
 12 files changed, 16162 insertions(+)
 create mode 100644 dev/assets/documenter.js
 create mode 100644 dev/assets/search.js
 create mode 100644 dev/assets/themes/documenter-dark.css
 create mode 100644 dev/assets/themes/documenter-light.css
 create mode 100644 dev/assets/themeswap.js
 create mode 100644 dev/assets/warner.js
 create mode 100644 dev/index.html
 create mode 100644 dev/search/index.html
 create mode 100644 dev/search_index.js
 create mode 100644 dev/siteinfo.js
 create mode 100644 index.html
 create mode 100644 versions.js
remote: Write access to repository not granted.
fatal: unable to access 'https://github.com/rikhuijzer/MyProject.jl.git/': The requested URL returned error: 403

Explicitly setting

permissions: write-all

fixed it.

More information about the permissions for the GITHUB_TOKEN can be found at https://docs.github.com/en/actions/security-guides/automatic-token-authentication. I haven't yet found related release notes.

@mortenpi
Copy link
Member

Yep, can confirm that it looks like new repositories are created with read-only permission on GITHUB_TOKEN by default now:

Screenshot from 2022-05-16 10-06-42

So this LGTM. Thanks @rikhuijzer!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants