porta
porta copied to clipboard
🦋⚛️ Settings SSO Integrations to Patternfly 4
I think we should used the word
enforced
rather thanenabled
for SSO. Because as long as you publish an SSO proivder, SSO is already enabled and can be used to login. The oldenforce
checkbox removed all other auth methos to ensure only SSO can be used to login, but it doesn't enable or disable SSO.
I agree with this. enable
/ disable
in this context does not seem accurate to me.
Maybe, to suggest an alternative, "enforce SSO" could be worded as "disable username-password login", and "unenforce SSO" could be "enable username-password login" :grimacing:
Hmm, something doesn't work right for me now... I have a published SSO, and I'm logged in through SSO, but the switch is disabled still...
I haven't found any (related) errors, did you make sure the integration was recently tested before trying to enforcing SSO?
Hmm, something doesn't work right for me now... I have a published SSO, and I'm logged in through SSO, but the switch is disabled still...
I tried everything locally and I don't see that problem. Maybe you tested the SSO integration more than one hour ago?
@jlledom @josemigallas I did 100% have a freshly tested integration. But since I tested there were some more commits, so I will retest of course.