Skip to content

[Sync] Update project files from source repository (6bf8447) - #159

Merged
mrz1836 merged 1 commit into
masterfrom
chore/sync-files-third-party-sdks-20260304-154258-6bf8447
Mar 4, 2026
Merged

mrz1836 merged 1 commit into
masterfrom
chore/sync-files-third-party-sdks-20260304-154258-6bf8447

Conversation

@mrz1836

@mrz1836 mrz1836 commented Mar 4, 2026

Copy link
Copy Markdown
Owner

What Changed

  • Added a new github-token input parameter to .github/actions/setup-go-with-cache/action.yml for private module authentication
  • Added a new github-token input parameter to .github/actions/warm-cache/action.yml for private module authentication
  • Implemented a new step "🔐 Configure private module authentication" in the setup-go-with-cache action that conditionally runs when both GOPRIVATE is set and github-token is provided
  • The authentication step configures git to use the token via git config --global url."https://x-access-token:${PRIVATE_MODULE_TOKEN}@github.com/" and sets GONOSUMDB to match GOPRIVATE if not already set

Why It Was Necessary

  • Enable GitHub Actions workflows to authenticate and download private Go modules during builds and tests
  • Support repositories that depend on internal or private Go modules that require authentication to access
  • Provide a secure way to configure git credentials using GitHub tokens without hardcoding authentication details

Testing Performed

  • Verified the action accepts the new github-token input parameter with appropriate default values
  • Confirmed the authentication step only executes when both conditions are met (GOPRIVATE set and token provided)
  • Validated that the git configuration correctly formats the token for HTTPS URL authentication

Impact / Risk

  • Breaking Change: None - the new input parameter is optional with an empty string default
  • Security: Low risk - token is passed as an environment variable and only used when explicitly provided
  • Compatibility: Existing workflows continue to function unchanged; new functionality is opt-in only

@mrz1836 mrz1836 self-assigned this Mar 4, 2026
@mrz1836 mrz1836 added automated-sync Automated sync PR, e.g. from a fork or external repo automerge Label to automatically merge pull requests that meet all required conditions chore Simple dependency updates or version bumps labels Mar 4, 2026
@github-actions github-actions Bot added size/M Medium change (51-200 lines) update General updates labels Mar 4, 2026
@mrz1836
mrz1836 merged commit 7bc0591 into master Mar 4, 2026
66 of 72 checks passed
@github-actions
github-actions Bot deleted the chore/sync-files-third-party-sdks-20260304-154258-6bf8447 branch March 4, 2026 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated-sync Automated sync PR, e.g. from a fork or external repo automerge Label to automatically merge pull requests that meet all required conditions chore Simple dependency updates or version bumps size/M Medium change (51-200 lines) update General updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

⌂ 换网址