Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add disableConditionKeyInheritance to iamResource #2036

Merged
merged 1 commit into from
Nov 8, 2023

Conversation

kstich
Copy link
Contributor

@kstich kstich commented Nov 8, 2023

This commit adds a disableConditionKeyInheritance property to the iamResource trait, which can be set to true to decouple an IAM resource's condition keys from those of its parent resource(s). This can be combined with the conditionKeys trait to explicitly configure an IAM resource's condition keys.

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

This commit adds a disableConditionKeyInheritance property to the
iamResource trait, which can be set to true to decouple an IAM
resource's condition keys from those of its parent resource(s). This
can be combined with the conditionKeys trait to explicitly configure
an IAM resource's condition keys.
@kstich kstich requested a review from a team as a code owner November 8, 2023 23:20
@kstich kstich merged commit 60f0f80 into main Nov 8, 2023
11 checks passed
@kstich kstich deleted the add_iamResource_disableConditionKeyInheritance branch November 8, 2023 23:56
@kstich kstich mentioned this pull request Nov 8, 2023
haydenbaker added a commit that referenced this pull request Nov 9, 2023
haydenbaker added a commit that referenced this pull request Nov 9, 2023
* Bump version to 1.41.0

* Add entry for #2034

* Remove 'therefore'

Co-authored-by: Hunter Mellema <[email protected]>

* Add 'plugin'

Co-authored-by: Hunter Mellema <[email protected]>

* Add entry for #2036

---------

Co-authored-by: Hunter Mellema <[email protected]>
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.

2 participants