Skip to content

Commit b5d2141

Browse files
committed
CCM-8572: PR feedback
1 parent 413277d commit b5d2141

File tree

5 files changed

+12
-2
lines changed

5 files changed

+12
-2
lines changed

infrastructure/terraform/components/sandbox/README.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,8 @@ No requirements.
3030
| <a name="output_api_base_url"></a> [api\_base\_url](#output\_api\_base\_url) | n/a |
3131
| <a name="output_cognito_user_pool_client_id"></a> [cognito\_user\_pool\_client\_id](#output\_cognito\_user\_pool\_client\_id) | n/a |
3232
| <a name="output_cognito_user_pool_id"></a> [cognito\_user\_pool\_id](#output\_cognito\_user\_pool\_id) | n/a |
33+
| <a name="output_internal_bucket_name"></a> [internal\_bucket\_name](#output\_internal\_bucket\_name) | n/a |
34+
| <a name="output_quarantine_bucket_name"></a> [quarantine\_bucket\_name](#output\_quarantine\_bucket\_name) | n/a |
3335
| <a name="output_templates_table_name"></a> [templates\_table\_name](#output\_templates\_table\_name) | n/a |
3436
<!-- vale on -->
3537
<!-- markdownlint-enable -->

infrastructure/terraform/modules/backend-api/README.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,8 @@ No requirements.
5151
| Name | Description |
5252
|------|-------------|
5353
| <a name="output_api_base_url"></a> [api\_base\_url](#output\_api\_base\_url) | n/a |
54+
| <a name="output_internal_bucket_name"></a> [internal\_bucket\_name](#output\_internal\_bucket\_name) | n/a |
55+
| <a name="output_quarantine_bucket_name"></a> [quarantine\_bucket\_name](#output\_quarantine\_bucket\_name) | n/a |
5456
| <a name="output_templates_table_name"></a> [templates\_table\_name](#output\_templates\_table\_name) | n/a |
5557
<!-- vale on -->
5658
<!-- markdownlint-enable -->

infrastructure/terraform/modules/backend-api/cloudwatch_event_rule_quarantine_tags_added.tf

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,12 @@ resource "aws_cloudwatch_event_rule" "quarantine_tags_added" {
99
bucket = {
1010
name = [module.s3bucket_quarantine.id]
1111
}
12+
object = {
13+
key = [
14+
{ prefix = "pdf-template/" },
15+
{ prefix = "test-data/" },
16+
]
17+
}
1218
}
1319
})
1420
}

infrastructure/terraform/modules/backend-api/cloudwatch_event_rule_virus_scan_failed.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ resource "aws_cloudwatch_event_rule" "virus_scan_failed" {
1111
}
1212
object = {
1313
tags = {
14-
GuardDutyMalwareScanStatus = ["THREATS_FOUND", "UNSUPPORTED", "ACCESS_DENIED", "FAILED"]
14+
GuardDutyMalwareScanStatus = [{ anything-but = "NO_THREATS_FOUND" }]
1515
}
1616
}
1717
}

lambdas/backend-api/src/templates/infra/template-repository.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -260,7 +260,7 @@ export class TemplateRepository {
260260
updates.push('#templateStatus = :templateStatusFailed');
261261
ExpressionAttributeNames['#templateStatus'] = 'templateStatus';
262262
ExpressionAttributeValues[':templateStatusFailed'] =
263-
'VIRUS_SCAN_FAILED' as TemplateStatus;
263+
'VIRUS_SCAN_FAILED' satisfies TemplateStatus;
264264
}
265265

266266
try {

0 commit comments

Comments
 (0)