Skip to content

feat(codebuild): add CodeConnections auth support for GitHub source#37522

Open
lvthillo wants to merge 4 commits intoaws:mainfrom
lvthillo:feat/codebuild-codeconnections-auth
Open

feat(codebuild): add CodeConnections auth support for GitHub source#37522
lvthillo wants to merge 4 commits intoaws:mainfrom
lvthillo:feat/codebuild-codeconnections-auth

Conversation

@lvthillo
Copy link
Copy Markdown
Contributor

@lvthillo lvthillo commented Apr 4, 2026

Issue # (if applicable)

Closes #31236
Closes #31726

Reason for this change

Add CodeConnections auth support for GitHub source

Description of changes

Adds a connectionArn property to GitHubSourceProps that allows configuring CodeConnections (GitHub App) authentication for CodeBuild GitHub sources.

Describe any new or updated permissions being added

When connectionArn is provided, codeconnections:UseConnection is granted to the project role, scoped to the specific connection ARN.

Description of how you validated changes

  • GitHub source with CodeConnections auth (repo-level)
  • Organizational webhook with CodeConnections auth
  • IAM permission grant verification
  • Absence of auth when connectionArn is not provided

Checklist


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@github-actions github-actions bot added the p2 label Apr 4, 2026
@aws-cdk-automation aws-cdk-automation requested a review from a team April 4, 2026 19:10
@github-actions github-actions bot added the beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK label Apr 4, 2026
Copy link
Copy Markdown
Collaborator

@aws-cdk-automation aws-cdk-automation left a comment

Choose a reason for hiding this comment

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

(This review is outdated)

@aws-cdk-automation aws-cdk-automation dismissed their stale review April 4, 2026 19:22

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 4, 2026

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
This security report is NOT a review blocker. Please try merge from main to avoid findings unrelated to the PR.


TestsPassed ✅SkippedFailed
Security Guardian Results24 ran24 passed
TestResult
No test annotations available

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 4, 2026

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
This security report is NOT a review blocker. Please try merge from main to avoid findings unrelated to the PR.


TestsPassed ✅SkippedFailed
Security Guardian Results with resolved templates24 ran24 passed
TestResult
No test annotations available

@aws-cdk-automation aws-cdk-automation added the pr/needs-further-review PR requires additional review from our team specialists due to the scope or complexity of changes. label Apr 4, 2026
@lvthillo
Copy link
Copy Markdown
Contributor Author

lvthillo commented Apr 4, 2026

I'm unable to run the integration test with a real deployment as it requires an authorized CodeConnections connection.
The integ test file is included but has no snapshot. Could a maintainer please run the integration test to generate the snapshot?

Copy link
Copy Markdown
Collaborator

@aws-cdk-automation aws-cdk-automation left a comment

Choose a reason for hiding this comment

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

The pull request linter fails with the following errors:

❌ Features must contain a change to an integration test file and the resulting snapshot.

If you believe this pull request should receive an exemption, please comment and provide a justification. A comment requesting an exemption should contain the text Exemption Request. Additionally, if clarification is needed, add Clarification Request to a comment.

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

Labels

beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK p2 pr/needs-further-review PR requires additional review from our team specialists due to the scope or complexity of changes.

Projects

None yet

2 participants