Skip to content

Commit

Permalink
fix(Remove debug statement): Remove extra debug statement in kinesisf…
Browse files Browse the repository at this point in the history
…irehose-s3 (#649)

* Remove debug statement

The JSON.stringify() call in this statement caused problems

* Update index.ts
  • Loading branch information
biffgaut authored Apr 7, 2022
1 parent b620ce6 commit 26e9ec0
Showing 1 changed file with 2 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ import { Construct } from "@aws-cdk/core";
import * as s3 from "@aws-cdk/aws-s3";
import * as defaults from "@aws-solutions-constructs/core";
import * as iam from "@aws-cdk/aws-iam";
import { overrideProps, printWarning, consolidateProps } from "@aws-solutions-constructs/core";
import { overrideProps, consolidateProps } from "@aws-solutions-constructs/core";
import * as logs from "@aws-cdk/aws-logs";
import * as cdk from "@aws-cdk/core";
import * as kms from "@aws-cdk/aws-kms";
Expand Down Expand Up @@ -170,7 +170,6 @@ export class KinesisFirehoseToS3 extends Construct {
awsManagedKey
);

printWarning(`kinesisFirehoseProps: ${JSON.stringify(props.kinesisFirehoseProps, null, 2)}`);
// if the client didn't explicity say it was a Kinesis client, then turn on encryption
if (!props.kinesisFirehoseProps ||
!props.kinesisFirehoseProps.deliveryStreamType ||
Expand All @@ -194,4 +193,4 @@ export class KinesisFirehoseToS3 extends Construct {
kinesisFirehoseProps
);
}
}
}

0 comments on commit 26e9ec0

Please sign in to comment.