Skip to content

chore(mixins-preview): adding toDestination() to vended logs#36896

Open
ShadowCat567 wants to merge 5 commits intomainfrom
vended-logs/to-destination
Open

chore(mixins-preview): adding toDestination() to vended logs#36896
ShadowCat567 wants to merge 5 commits intomainfrom
vended-logs/to-destination

Conversation

@ShadowCat567
Copy link
Contributor

@ShadowCat567 ShadowCat567 commented Feb 5, 2026

Reason for this change

This is partially to support cross account vended logs, partially because being able to provide a deliveryDestination arn to set up a delivery connection is useful.

Description of changes

toDestination() allows VendedLogs to accept any CfnDeliveryDestination and will create a log delivery with it to the source resource that is provided.

Describe any new or updated permissions being added

N/A

Description of how you validated changes

Added unit and integration tests

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 Feb 5, 2026
@aws-cdk-automation aws-cdk-automation requested a review from a team February 5, 2026 22:26
@github-actions github-actions bot added the valued-contributor [Pilot] contributed between 6-12 PRs to the CDK label Feb 5, 2026
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Feb 5, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
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
Contributor

github-actions bot commented Feb 5, 2026

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
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

@ShadowCat567 ShadowCat567 marked this pull request as ready for review February 6, 2026 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution/core This is a PR that came from AWS. p2 valued-contributor [Pilot] contributed between 6-12 PRs to the CDK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants