-
Notifications
You must be signed in to change notification settings - Fork 432
Cognitoユーザープールにメール認証によるMFAを導入 #1401
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
Changes from all commits
f5ff69a
17b0122
d952576
22499b6
df029e6
fc59846
e2bcea3
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -13,6 +13,9 @@ const baseStackInputSchema = z.object({ | |
| samlAuthEnabled: z.boolean().default(false), | ||
| samlCognitoDomainName: z.string().nullish(), | ||
| samlCognitoFederatedIdentityProviderName: z.string().nullish(), | ||
| mfaEnabled: z.boolean().default(false), | ||
| mfaFromEmail: z.string().email().default('no-reply@example.com'), | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. すみません、こちら default 値指定したほうがいいかもしれないとコメントさせていただいたのですが、検証されたメールを設定しないとデプロイ時エラーになるため 「mfaEnabled の際は検証されたメールを明示的に入力する必要がある」を徹底するために、ここでは nullish にして refine でデプロイ前にエラーメッセージを表示する形にしたほうが良いかもしれません。 // 既存の refine に追加 |
||
| mfaReplyToEmail: z.string().nullish(), | ||
| // Frontend | ||
| hiddenUseCases: z | ||
| .object({ | ||
|
|
||
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.
Cognito の Feature Plan は Essentials 以上でないと Email MFA が使えません。
という挙動の差があり、今の実装ですと古い環境ではデプロイに失敗するようです。
以下のように明示的に MFA 有効化する際は Essential 指定することで古い環境でもデプロイできました。
featurePlan: props.mfaEnabled ? FeaturePlan.ESSENTIALS : undefined,