Skip to content

Commit 794483c

Browse files
committed
Update nag suppressions
1 parent 1b8ca73 commit 794483c

File tree

1 file changed

+1
-56
lines changed

1 file changed

+1
-56
lines changed

packages/cdk/nagSuppressions.ts

Lines changed: 1 addition & 56 deletions
Original file line numberDiff line numberDiff line change
@@ -73,26 +73,6 @@ export const nagSuppressions = (stack: Stack) => {
7373
]
7474
)
7575

76-
// Suppress S3 warnings on EpsAssistDocsBucket
77-
safeAddNagSuppression(
78-
stack,
79-
"/EpsAssistMeStack/Storage/DocsBucket/EpsAssistDocsBucket/Resource",
80-
[
81-
{
82-
id: "AwsSolutions-S1",
83-
reason: "No access logs needed for internal development usage."
84-
},
85-
{
86-
id: "AwsSolutions-S10",
87-
reason: "SSL enforcement via bucket policy is deferred."
88-
},
89-
{
90-
id: "S3_BUCKET_REPLICATION_ENABLED",
91-
reason: "Replication not required for internal bucket."
92-
}
93-
]
94-
)
95-
9676
// Suppress missing WAF on API stage for Apis construct
9777
safeAddNagSuppression(
9878
stack,
@@ -105,42 +85,6 @@ export const nagSuppressions = (stack: Stack) => {
10585
]
10686
)
10787

108-
// Suppress warnings on access logs bucket
109-
safeAddNagSuppression(
110-
stack,
111-
"/EpsAssistMeStack/Storage/AccessLogsBucket/EpsAssistAccessLogsBucket/Resource",
112-
[
113-
{
114-
id: "AwsSolutions-S10",
115-
reason: "SSL policy is pending; logged for follow-up."
116-
},
117-
{
118-
id: "S3_BUCKET_REPLICATION_ENABLED",
119-
reason: "Replication not needed."
120-
},
121-
{
122-
id: "S3_BUCKET_VERSIONING_ENABLED",
123-
reason: "Short-lived logs don't need versioning."
124-
},
125-
{
126-
id: "S3_BUCKET_LOGGING_ENABLED",
127-
reason: "No logging needed on logging bucket."
128-
}
129-
]
130-
)
131-
132-
// Suppress SSL warning on actual access log bucket policy resource
133-
safeAddNagSuppression(
134-
stack,
135-
"/EpsAssistMeStack/Storage/AccessLogsBucket/EpsAssistAccessLogsBucket/Policy/Resource",
136-
[
137-
{
138-
id: "AwsSolutions-S10",
139-
reason: "SSL enforcement on access logs bucket policy is deferred and documented."
140-
}
141-
]
142-
)
143-
14488
// Suppress IAM wildcard permissions for Bedrock execution role
14589
safeAddNagSuppression(
14690
stack,
@@ -152,6 +96,7 @@ export const nagSuppressions = (stack: Stack) => {
15296
appliesTo: [
15397
"Resource::<StorageDocsBucketEpsAssistDocsBucket7DBA5CCC.Arn>/*",
15498
"Resource::<StorageEpsAssistDocsBucket3189F25F.Arn>/*",
99+
"Resource::<StorageDocsBucketDocs0C9A9D9E.Arn>/*",
155100
"Action::aoss:*",
156101
"Resource::*",
157102
"Resource::<OsCollection.Arn>/*",

0 commit comments

Comments
 (0)