Anpassung von SSO-Inkonsistenzen

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.

Ich habe die Änderungen umgesetzt, um die beiden oben genannten Punkte zum Laufen zu bringen. Ich würde mich freuen, sie wieder in Discourse einzubringen, aber ich weiß nicht, ob es Interesse daran gibt, diese Änderungen upstream aufzunehmen? /cc @sam

Lass mich bitte Bescheid geben, bevor ich mich durch die Hürden des PR-Einreichungsprozesses kämpfe. Der Code funktioniert, muss aber möglicherweise überarbeitet werden, da ich neu bei Discourse bin.

Starte den PR, damit du den CLA-Prozess durchlaufen kannst, aber:

Das scheint nicht wirklich den Aufwand wert zu sein – andere Felder auf dieser Seite, wie z. B. das Hintergrundbild der Benutzerkarte, sind in SSO-Plattformen ohnehin nicht verfügbar, sodass es keinen Konsistenzvorteil bringt, eine Sperre für die Website einzuführen.

1 „Gefällt mir“

Entschuldigung, ich bin mir nicht sicher, ob ich das richtig verstanden habe. Ich stimme zu, dass das Hintergrundbild der Karte auf SSO nur begrenzt nützlich ist (obwohl dies bereits implementiert ist). website ist ebenfalls ein bereits implementiertes SSO-Feld – ich habe lediglich eine Einstellung namens sso_overrides_website hinzugefügt, ähnlich wie es sie bereits für Benutzername, Avatar, Biografie usw. gibt, um lokale Änderungen zu unterbinden und den SSO-Wert vorrangig zu behandeln.