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 aws.iam#iamAction trait #2034

Merged
merged 1 commit into from
Nov 8, 2023
Merged

Add aws.iam#iamAction trait #2034

merged 1 commit into from
Nov 8, 2023

Conversation

kstich
Copy link
Contributor

@kstich kstich commented Nov 8, 2023

This commit introduces the iamAction trait to both consolidate and enhance the customization for IAM actions in derived from Smithy operations. It deprecates the actionName, actionPermissionDescription, and requiredAction traits - placing their behavior in the name, documentation, and requiredAction members respectively.

A relativeDocumentation member is new, allowing for linking to docs within IAM documentation.

A createsResources member is new, allowing for the explicit listing of resources that performing an IAM action will create.

A resources member is new, allowing for the override configuration of required and optional resources that an IAM action can be authorized against (including the condition keys that can be used). This enables the detachment of an IAM action from being required to authorize against its resource hierarchy for special cases.

The specification is also reorganized into sections for traits affecting resources, actions, and condition keys for easier navigation.

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

@kstich kstich requested a review from a team as a code owner November 8, 2023 07:27
@kstich kstich changed the title Add aws.iam#iamAction trait Add aws.iam#iamAction trait Nov 8, 2023
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@kstich kstich force-pushed the add_iamAction_trait branch from 6cdae7f to 11c00cb Compare November 8, 2023 16:46
This commit introduces the iamAction trait to both consolidate and
enhance the customization for IAM actions in derived from Smithy
operations. It deprecates the actionName, actionPermissionDescription,
and requiredAction traits - placing their behavior in the name,
documentation, and requiredAction members respectively.

A relativeDocumentation member is new, allowing for linking to docs
within IAM documentation.

A createsResources member is new, allowing for the explicit listing
of resources that performing an IAM action will create.

A resources member is new, allowing for the override configuration
of required and optional resources that an IAM action can be authorized
against (including the condition keys that can be used). This enables
the detachment of an IAM action from being required to authorize against
its resource hierarchy for special cases.

The specification is also reorganized into sections for traits affecting
resources, actions, and condition keys for easier navigation.
@kstich kstich force-pushed the add_iamAction_trait branch from 11c00cb to 55c81b8 Compare November 8, 2023 17:08
haydenbaker added a commit that referenced this pull request Nov 8, 2023
@kstich kstich merged commit fdc5f1b into main Nov 8, 2023
11 checks passed
@kstich kstich deleted the add_iamAction_trait branch November 8, 2023 18:10
haydenbaker added a commit that referenced this pull request Nov 8, 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 <124718352+hpmellema@users.noreply.github.com>

* Add 'plugin'

Co-authored-by: Hunter Mellema <124718352+hpmellema@users.noreply.github.com>

* Add entry for #2036

---------

Co-authored-by: Hunter Mellema <124718352+hpmellema@users.noreply.github.com>
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