[6.x] Add config for disabling Elevated Sessions#14464
[6.x] Add config for disabling Elevated Sessions#144641stevengrant wants to merge 5 commits intostatamic:6.xfrom
Conversation
jasonvarga
left a comment
There was a problem hiding this comment.
Opting out of a security feature like this feels awkward.
How are you implementing OAuth on your site? It might make sense if the Elevated Session page allowed you to re-authenticate however you authenticated in the first place.
That might be a big rabbit hole though. It'll probably end up better to just allow a way to disable it in the end, but I'd still like to know.
In any case, could you change it to elevated_sessions_enabled (with true as the default) as having disabled: true always feels backwards.
|
We use Auth0 to authenticate via OTP. There's a customised login screen hosted with them and then we have a success callback that handles Laravel auth. We never login via the control panel. Works fine currently with v5. |
|
Fair enough. Yeah let's just flip the config to be |
Summary
elevated_session_disabledconfig option toconfig/users.phpthat allows disabling elevated sessions entirelyRequireElevatedSessionmiddleware is bypassed, so users are never prompted to reauthorizeContext
Our Statamic implementation uses Auth0 as the primary auth driver, which means elevated sessions (password re-confirmation) are not applicable since authentication is handled externally. This config option allows us (and others using OAuth providers) to cleanly disable the feature.
References: