Fixing customer_condition to ignore trailing semicolon (#239)

ExpediaGroup · Aug 25, 2023 · a168cd3 · a168cd3
1 parent 03fca3f
commit a168cd3
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -3,6 +3,10 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/) and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## [6.19.1] - 2023-08-25
+### Fixed
+- `customer_condition` to ignore trailing semicolon.
+
 ## [6.19.0] - 2023-08-16
 ### Added
 - Added variable `enable_hms_housekeeper ` to support hive metastore lock house keeper.

diff --git a/templates/apiary-bucket-policy.json b/templates/apiary-bucket-policy.json
@@ -43,6 +43,7 @@
         },
 %{else}
 %{for condition in split(";",customer_condition)}
+%{if length(trimspace(condition)) > 0 }
         {
             "Sid": "Apiary customer account object permissions",
             "Effect": "Allow",
@@ -64,6 +65,7 @@
                 "arn:aws:s3:::${bucket_name}/*"
             ]
         },
+%{endif}
 %{endfor}
 %{endif}
 %{for prefix, role_list in consumer_prefix_roles ~}