Skip to content

Commit 17351e2

Browse files
authored
Merge pull request #6 from mitre/recommended_settings
Update settings and propose new environment variables
2 parents 31b525f + 5c16c74 commit 17351e2

File tree

2 files changed

+9
-3
lines changed

2 files changed

+9
-3
lines changed

README.md

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,12 @@ Additional optional variables can be set to further configure the function. The
2626
| **OUTPUT_BUCKET** | x | none | "bucket-name" |
2727
| OUTPUT_ENABLED | | true | false |
2828
| OUTPUT_PREFIX | | "results/" | "output/", "results/hdf/", "" |
29-
| OUTPUT_TIMEOUT | | 60 | lambda timeout value in seconds |
29+
| OUTPUT_TIMEOUT | | 900 | lambda timeout value in seconds |
3030
| SERVICE_NAME | | "saf-lambda-function" | "different-service-name" |
31+
| IAM_ROLE_PATH | | none | </role/path/> e.g., "/delegatedadmin/developer/" |
32+
| IAM_ROLE_BOUNDARY | | none | <arn:aws:iam::${aws:accountId}:policy/permissions-boundary-policy>, e.g., "arn:aws:iam::1234567890:policy/cms-cloud-admin/developer-boundary-policy" |
33+
| SCHED_INTERVAL_MINUTES | | 1440 | scheduled event trigger interval in minutes |
34+
3135

3236
6. Set the required variables: `OUTPUT_BUCKET` and `COMMAND_STRING`.
3337
- Example:

serverless.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@ provider:
66
runtime: nodejs14.x
77
iam:
88
role:
9+
path: ${env:IAM_ROLE_PATH, ""}
10+
permissionsBoundary: ${env:IAM_ROLE_BOUNDARY, ""}
911
statements:
1012
- Effect: "Allow"
1113
Action:
@@ -17,10 +19,10 @@ provider:
1719

1820
functions:
1921
saf:
20-
timeout: ${env:TIMEOUT, 60}
22+
timeout: ${env:TIMEOUT, 900}
2123
handler: handler.saf
2224
events:
23-
- schedule: rate(3 minutes)
25+
- schedule: rate(${env:SCHED_INTERVAL_MINUTES, 1440} minutes)
2426
environment:
2527
COMMAND_STRING: ${env:COMMAND_STRING}
2628
OUTPUT_PREFIX: ${env:OUTPUT_PREFIX, "results/"}

0 commit comments

Comments
 (0)