Skip to content

fix: authenticate GitHub API curl call in update-android-sdk-version workflow#246

Merged
jonathannorris merged 3 commits into
mainfrom
fix/authenticate-github-api-curl-in-android-sdk-update
May 19, 2026
Merged

fix: authenticate GitHub API curl call in update-android-sdk-version workflow#246
jonathannorris merged 3 commits into
mainfrom
fix/authenticate-github-api-curl-in-android-sdk-update

Conversation

@jonathannorris
Copy link
Copy Markdown
Member

Summary

  • Adds an Authorization header to the curl call that fetches the latest Android SDK version from the GitHub API.

Unauthenticated API calls are subject to a 60 req/hour rate limit per IP. The shared GitHub Actions runner pool exhausts this quickly, causing the request to return null and the workflow to silently skip the update (or fail in the iOS equivalent). Authenticating with GITHUB_TOKEN raises the limit to 5,000 req/hour.

Same fix applied to iOS in #245.

Copilot AI review requested due to automatic review settings May 15, 2026 21:03
@jonathannorris jonathannorris requested a review from a team as a code owner May 15, 2026 21:03
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Authenticates the GitHub API curl call in the Android SDK version update workflow to avoid hitting the 60 req/hour unauthenticated rate limit on shared runners, which was causing silent skips when fetching the latest release tag.

Changes:

  • Adds Authorization: token ${{ secrets.GITHUB_TOKEN }} header to the curl call fetching the latest Android SDK release.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings May 19, 2026 15:39
@jonathannorris jonathannorris enabled auto-merge (squash) May 19, 2026 15:40
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@jonathannorris jonathannorris merged commit 14b7d4a into main May 19, 2026
8 checks passed
@jonathannorris jonathannorris deleted the fix/authenticate-github-api-curl-in-android-sdk-update branch May 19, 2026 15:41
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