Skip to content

Commit 73a1de1

Browse files
daiwei920mergify[bot]
authored andcommitted
fix(apigateway): json schema additionalProperties should be boolean (#3997)
* Add support for json schema additionalProperties * Change it to boolean according to code review * add breaking change exclusions
1 parent 21eed2d commit 73a1de1

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

allowed-breaking-changes.txt

+3
Original file line numberDiff line numberDiff line change
@@ -16,3 +16,6 @@ incompatible-argument:@aws-cdk/aws-lambda.Function.<initializer>
1616
incompatible-argument:@aws-cdk/aws-lambda.SingletonFunction.<initializer>
1717
incompatible-argument:@aws-cdk/aws-lambda.Function.addEnvironment
1818
changed-type:@aws-cdk/aws-dynamodb.Table.tableStreamArn
19+
incompatible-argument:@aws-cdk/aws-apigateway.LambdaRestApi.addModel
20+
incompatible-argument:@aws-cdk/aws-apigateway.Model.<initializer>
21+
incompatible-argument:@aws-cdk/aws-apigateway.RestApi.addModel

packages/@aws-cdk/aws-apigateway/lib/json-schema.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ export interface JsonSchema {
6363
readonly minProperties?: number;
6464
readonly required?: string[];
6565
readonly properties?: { [name: string]: JsonSchema };
66-
readonly additionalProperties?: JsonSchema;
66+
readonly additionalProperties?: boolean;
6767
readonly patternProperties?: { [name: string]: JsonSchema };
6868
readonly dependencies?: { [name: string]: JsonSchema | string[] };
6969
readonly propertyNames?: JsonSchema;

0 commit comments

Comments
 (0)