-
Notifications
You must be signed in to change notification settings - Fork 4.5k
aws-lambda: incorrect securityGroups check #27147
Description
Describe the bug
This commit in aws-lambda adds a check for the presence of a security groups parameter if the VPC parameter is not present. Since TypeScript treats empty arrays as true, the check for the securityGroups parameter evaluates to true when an empty array is passed in for FunctionProps.securityGroups.
Expected Behavior
This condition checking for the presence of FunctionProps.securityGroups should evaluate to false if FunctionProps.securityGroups is an empty array.
Current Behavior
This condition checking for the presence of FunctionProps.securityGroups should evaluate to true if FunctionProps.securityGroups is an empty array.
Reproduction Steps
Pass in an empty array for the securityGroups field and no value for the vpc field in FunctionProps when calling Function::configureVpc method.
Possible Solution
Explicitly check for empty arrays in addition to the current truthiness check.
Additional Information/Context
No response
CDK CLI Version
2.93.0
Framework Version
No response
Node.js Version
14.19.1
OS
Linux
Language
Typescript
Language Version
No response
Other information
No response