aws-apigateway-dynamodb: setting "allowReadOperation" to false does not disable the read operation

[JakePartusch]

Setting allowReadOperation to false does not disable the read operation.

Reproduction Steps

import { ApiGatewayToDynamoDBProps, ApiGatewayToDynamoDB } from "@aws-solutions-constructs/aws-apigateway-dynamodb";

new ApiGatewayToDynamoDB(this, 'test-api-gateway-dynamodb-default', {
  allowReadOperation: false
});

Error Log

There is no error in synthesizing the stack

Environment

• CDK CLI Version : 1.79.0
• CDK Framework Version: 1.79.0
• AWS Solutions Constructs Version : 1.79.0
• OS : Mac
• Language : TypeScript

Other

This issue seems to be with this conditional:

aws-solutions-constructs/source/patterns/@aws-solutions-constructs/aws-apigateway-dynamodb/lib/index.ts

Line 154 in 8130c54

if (!props.allowReadOperation || props.allowReadOperation === true) {

Given the three input options (true/false/undefined), the conditional always will evaluate to true

---

This is 🐛 Bug Report

[hnishar]

JakePartusch@ This bug is fixed in v1.80.0, for proof see this