Fix parsing of boolean attribute in SSO (PR #3496)

The require_activation field was added and intended to be a boolean, but wasn’t being parsed as one (i.e. if anything was present, including “false”, it evaluated to true).

GitHub

You’ve signed the CLA, paulkaplan. Thank you! This pull request is ready for review.

Thanks, mind squashing the commits into 1?

Squashed!

cool good catch, is this working out for you?

@SamSaffron we are about to enable it! I’ll let you know.