fix(util-endpoints): revert handling case when resourcePath contains …

…both delimiters This reverts commit 855f1fa. It'll be added, if needed, in a separate PR.
aws · Aug 13, 2024 · 3937a6a · 3937a6a
1 parent 855f1fa
commit 3937a6a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 11 deletions.
diff --git a/packages/util-endpoints/src/lib/aws/parseArn.spec.ts b/packages/util-endpoints/src/lib/aws/parseArn.spec.ts
@@ -74,16 +74,6 @@ describe(parseArn.name, () => {
         resourceId: ["myTopic"],
       },
     ],
-    [
-      "arn:aws:s3:us-west-2:123456789012:my:folder/my:file",
-      {
-        partition: "aws",
-        service: "s3",
-        region: "us-west-2",
-        accountId: "123456789012",
-        resourceId: ["my", "folder", "my", "file"],
-      },
-    ],
   ];
 
   it.each(VALID_TEST_CASES)("returns for valid arn %s", (input: string, outout: EndpointARN) => {

diff --git a/packages/util-endpoints/src/lib/aws/parseArn.ts b/packages/util-endpoints/src/lib/aws/parseArn.ts
@@ -17,7 +17,9 @@ export const parseArn = (value: string): EndpointARN | null => {
 
   if (arn !== "arn" || partition === "" || service === "" || resourcePath.join(ARN_DELIMITER) === "") return null;
 
-  const resourceId = resourcePath.map((resource) => resource.split(RESOURCE_DELIMITER)).flat();
+  const resourceId = resourcePath[0].includes(RESOURCE_DELIMITER)
+    ? resourcePath[0].split(RESOURCE_DELIMITER)
+    : resourcePath;
 
   return {
     partition,