-
Notifications
You must be signed in to change notification settings - Fork 3.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[core] Unable to set CfnIdentityPoolProps allowUnauthenticatedIdentities to false #10455
Comments
Also of note, the same behaviour occurs when trying to use the
|
Currently working around the issue with:
|
Definitely something weird going on here. It seems that when there all properties on a Simply typescript app that reproduces this problem - import { App, Duration, Stack } from '@aws-cdk/core';
import { CfnIdentityPool } from '@aws-cdk/aws-cognito';
const app = new App();
const stack = new Stack(app, 'mystack');
new CfnIdentityPool(stack, 'pool', {
allowUnauthenticatedIdentities: false,
}); |
Any CloudFormation resource that defines a single boolean property set to false is not rendered to the CloudFormation template. The bug is in implementation of `_toCloudFormation()` API in `CfnResource`. It treated `false` and `undefined` the same way. fixes #10455
The bug is in the |
…10539) Any CloudFormation resource that defines a single boolean property set to false is not rendered to the CloudFormation template. The bug is in implementation of `_toCloudFormation()` API in `CfnResource`. It treated `false` and `undefined` the same way. fixes #10455 ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
When creating a Cognito Identity pool using the
CfnIdentityPool
class, setting theallowUnauthenticatedIdentities
field to false on theCfnIdentityPoolProps
builder results in no properties being included in the cloudformation template produced by runningcdk synth
. This property is required to be set, and so any attempt at acdk deploy
will fail to create the resource. When the value is set totrue
, the property is correctly included in the synthesised cloudformation.Reproduction Steps
Main app:
ReproductionStack:
Output of
cdk synth
:Output of
cdk synth
when.allowUnauthenticatedIdentities(true)
is used:What did you expect to happen?
A Cloudformation template including the correct properties for the defined identity pool would be created.
What actually happened?
The incorrect Cloudformation template is generated.
Environment
This is 🐛 Bug Report
The text was updated successfully, but these errors were encountered: