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

Include GitHub authentication token while setting up protoc. #503

Merged
merged 2 commits into from
Feb 14, 2025

Conversation

chungyc
Copy link
Member

@chungyc chungyc commented Feb 1, 2025

arduino/setup-protoc uses the GitHub API to fetch release data. Unauthenticated requests have a much lower rate limit, so pass on the GitHub token so that it can issue authenticated requests. Unauthenticated requests with the lower rate limit may cause builds to fail.

Explicitly set the permissions to be read-only, since the workflow does not need write access to the repository, so that accidental or malicious changes cannot be made despite having the authentication token.

See also

@agrue agrue merged commit 0d85d56 into google:master Feb 14, 2025
11 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