SSO IDP パラメータの追加に異議はありますか？

サポート SSO
1

/session/sso パスに、コントローラーが IDP のリクエスト URL に追加するオプションパラメータを追加したいと考えています。その目的は、IDP にどの認証フロー（ソーシャルログインかローカルログインか）を開始するかを伝えることです。初期の Discourse SSO リダイレクトの一部として IDP に何らかのシグナルを与えない限り、これは実現できません。

例えば、idp_param というオプションパラメータを想定しています：
/session/sso?return_path=/&idp_param=fb

これとプルリクエストに対して異論はあるでしょうか？

この提案の理由は、Discourse が提供する単一の「ログイン」ボタンでは、あまり誘導的ではなく情報量も不足していると感じているため、アップストリームのソーシャルプロバイダー（当社の IDP で設定されているもの）のいずれかを使って匿名訪問者にサインインを促す大きなバナーを作成したいからです。

現在のモックアップ（2 つのソーシャルボタンが URL に idp_param を追加するもの）：

image
image1427×856 242 KB

2

別の方法として、SSO URL の設定をキーと値のコレクションとし、パラメータでキーを指定して異なる IDP URL をターゲットにできるようにする方法があります。例えば：

default : https://myidp.com/sso
fb : https://myidp.com/sso?provider=fb
google : https://myidp.com/sso?provider=google

/session/sso?return_path=/&sso_uri=fb

ただし、これは使用頻度が低いシナリオに対しては、より多くの作業を要します。