feat: Add support for hierarchical ARNs in Arn utility #34988
+229
−21
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Reason for this change
Some AWS services are beginning to provide hierarchical arns for finer grained resource permissions management. AWS PCS is one of those services.
Description of changes
The Arn utility has been updated to support properly resolving the
resource
andresourceName
for hierarchical ARNs. Previously, using the slash ARN support, it would return the parent resource type forresource
andresourceName
would include the entire rest of the ARN string.I attempted to make this as minimal a change as possible. I followed TS best practices adding an optional "options" argument to the
Split
method in order to support hierarchical ARN depth configuration.Describe any new or updated permissions being added
None
Description of how you validated changes
Added unit tests for the hierarchical ARN support, and existing unit tests continue to pass.
Checklist
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license