Disallow setting 'enable_sso' without SOMETHING in 'sso_url'

We should not allow Discourse admins to wedge themselves into this corner (or at least give them a warning?) where:

SiteSetting.enable_sso == true
SiteSetting.sso_url == ""
8 Likes

Yes definitely can we add a validation here @techAPJ

5 Likes

Fixed via:

https://github.com/discourse/discourse/commit/0514ac4ee2f7f383ba8ce7a33453fe0232af601a

6 Likes

This topic was automatically closed after 28 hours. New replies are no longer allowed.