Open
Conversation
デプロイオプションにsamlAuthPasswordFallbackEnabledを追加し、SAML有効時にメール/パスワードのログインも利用可能にする。 ?auth=passwordクエリパラメータでパスワードログイン画面へ直接アクセス可能。 samlAuthEnabledがfalseかつsamlAuthPasswordFallbackEnabledがtrueの組み合わせはバリデーションエラーにする。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of Changes
SSOが利用者全員に行き渡っていないユースケースに対応するために、
samlAuthPasswordFallbackEnabledオプションを追加し、SAML SSOが有効な環境でもメールとパスワードによるログインを併用できるようにしました。samlAuthEnabled: trueかつsamlAuthPasswordFallbackEnabled: trueの両方が設定された場合にのみ機能が有効になります。新フラグのデフォルト値は
falseのため、既存の環境には一切影響しません。フラグの組み合わせによる動作は以下の通りです。
samlAuthEnabledsamlAuthPasswordFallbackEnabledfalsefalsetruefalsetruetruefalsetrue?auth=passwordクエリパラメータを付与したURLでパスワードログイン画面に直接アクセスできます。これにより、ブックマークやリンク経由でログインするユーザーは毎回ログイン方式の選択画面を経由する必要がなくなります。
パスワードログイン画面には直接アクセスできるデザインにしているため、SAMLログインとパスワードログインを選択する画面では、パスワードログインは従来のSAMLログインボタンと比べて目立たないように配置しています。
Checklist
npm run cdk:testand if there are snapshot differences, executenpm run cdk:test:update-snapshotto update snapshotsRelated Issues
#1498