Skip to content

Commit

Permalink
fixed type issue
Browse files Browse the repository at this point in the history
  • Loading branch information
jnayak-moj committed Dec 3, 2024
1 parent 4b2ce0a commit 67c91bf
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion iam_builder/iam_builder.py
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ def build_iam_policy(config: dict) -> dict: # noqa: C901
# Deal with read only access
if "read_only" in config["secretsmanager"]:
secretsmanager_read_only = get_secretsmanager_read_only_policy(
config["secretsmanager"]
config["secretsmanager"]["read_only"]
)
iam["Statement"].append(secretsmanager_read_only)
else:
Expand Down
4 changes: 3 additions & 1 deletion tests/test_config/secretsmanager_read_only.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
iam_role_name: an_iam_role_name

secretsmanager: read_only
secretsmanager:

Check failure on line 3 in tests/test_config/secretsmanager_read_only.yaml

View workflow job for this annotation

GitHub Actions / build

3:16 [trailing-spaces] trailing spaces
read_only:
- test/test_secret

0 comments on commit 67c91bf

Please sign in to comment.