Skip to content

CON-382: Allow using GITHUB_TOKEN environment variable for Github API.#9

Merged
Alxe merged 2 commits intorticommunity:masterfrom
Alxe:feature/CON-382-gh-token
Feb 19, 2026
Merged

CON-382: Allow using GITHUB_TOKEN environment variable for Github API.#9
Alxe merged 2 commits intorticommunity:masterfrom
Alxe:feature/CON-382-gh-token

Conversation

@Alxe
Copy link
Member

@Alxe Alxe commented Feb 17, 2026

Add GITHUB_TOKEN support to build.rs. See #8 for details.

@Alxe Alxe marked this pull request as ready for review February 18, 2026 16:42
@Alxe Alxe requested a review from Copilot February 18, 2026 16:42
Copy link

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

Adds support for authenticated GitHub API requests in build.rs by optionally using a GITHUB_TOKEN environment variable, reducing the likelihood of hitting unauthenticated rate limits when resolving/downloading Connector native libraries.

Changes:

  • Introduces a helper to attach an Authorization: Bearer … header when GITHUB_TOKEN is set.
  • Applies the helper to GitHub release metadata fetch and asset download requests.

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

Use Github API for asset download
Dirty on GITHUB_TOKEN changes
Sanitize GITHUB_TOKEN to some extent
@Alxe Alxe merged commit a126f58 into rticommunity:master Feb 19, 2026
0 of 2 checks passed
@Alxe Alxe deleted the feature/CON-382-gh-token branch February 19, 2026 12:23
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.

1 participant

Comments