SSOの不整合の調整

貢献 特徴
1

Most profile fields can already be specified during the SSO process which is super-useful. From username to name, to bio, and even website was added last year. But, there’s some still minor quirks:

  • location seems to be missing — from a cursory look, seems to be the only obvious field that isn’t possible to change using SSO.
  • website can be specified during SSO, but unlike most (all?) other SSO profile fields, it doesn’t have a corresponding site setting to allow overriding the local value and prevent user from changing it.

I’m guessing these were just overlooked as the code base grew over time, as I can’t think of a reason for these exceptions. Unfortunately my Ruby and Discourse knowledge isn’t enough to submit a PR, but I wanted to report it — perhaps someone has the technical chops to do it.

2

上記の 2 つの項目が機能するように変更を実装しました 実装。Discourse に貢献したいと考えていますが、アップストリームでこの機能に関心があるかどうかはわかりません。/cc @sam

PR を提出する前に、ご意見をお聞かせください。コードは動作しますが、Discourse 初心者であるため、いくつかの修正や変更が必要かもしれません。

3

PR を開始して CLA プロセスを進めてください。ただし：

これは取り組む価値があまりないと感じます。そのページのユーザーカードの背景画像などの他のフィールドは SSO プラットフォームでは利用できないため、website に対してロックダウンを追加しても一貫性の利点はありません。

「いいね！」 1
4

申し訳ありませんが、その部分が何を意味しているのか理解できませんでした。カードの背景は SSO 上では限定的な用途しかないと同意します（ただし、すでに実装されています）。website もすでに実装されている SSO フィールドです。ローカル変更を許可せず、SSO の値を優先できるようにするために、ユーザー名、アバター、自己紹介文などに対して既に存在する設定と同様に、sso_overrides_website という設定を追加しました。