-
Notifications
You must be signed in to change notification settings - Fork 302
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
Feat: Added flexibility to super-easy deployment mode #318
Feat: Added flexibility to super-easy deployment mode #318
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your kind coding including some validations!
cdk/test/cdk.test.ts
Outdated
@@ -29,6 +29,7 @@ describe("Fine-grained Assertions Test", () => { | |||
start: {}, | |||
}, | |||
enableMistral: false, | |||
selfSignUpEnabled: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix the indent
cdk/test/cdk.test.ts
Outdated
@@ -84,6 +85,7 @@ describe("Fine-grained Assertions Test", () => { | |||
start: {}, | |||
}, | |||
enableMistral: false, | |||
selfSignUpEnabled: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix the indent
@@ -34,8 +35,8 @@ export class Auth extends Construct { | |||
requireDigits: true, | |||
minLength: 8, | |||
}, | |||
// Disable if identity providers are configured | |||
selfSignUpEnabled: !props.idp.isExist(), | |||
// Disable id selfSignUpEnabled is given as false or if selfSignUpEnabled is true and idp is provided |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
今更で申し訳ないのですが、この変更にはどのような意味があるのでしょうか?
外部idpを使用している場合でも、自己登録をONにしたい需要はあると思うのと、これで制限されているのがどこかに明文化されているのでしょうか?
よく分からない仕様になってしまわないかな?と思っていたりするのですが。
普通にREADMEに記載されていました。すみません。
外部 ID プロバイダーを構成すると、値は無視され、自動的に無効になります。
Description of changes:
New Features
Benefits
By leveraging these new parameters, you can tailor the deployment to better meet your security requirements and operational needs.