Merge pull request #374 from humanmade/disable-two-factor-forced-roles

Allow disabling 2FA in config
humanmade · Dec 4, 2024 · 6049e7f · 6049e7f
2 parents fb88f00 + b46030b
commit 6049e7f
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/inc/namespace.php b/inc/namespace.php
@@ -131,7 +131,12 @@ function override_two_factor_universally_forced( bool $is_forced ) : bool {
  */
 function override_two_factor_forced_user_roles( $roles ) {
 	$config = Altis\get_config()['modules']['security']['2-factor-authentication'];
-	if ( ! empty( $config['required'] ) && is_array( $config['required'] ) ) {
+
+	if ( isset( $config['required'] ) && $config['required'] === false ) {
+		return [];
+	}
+
+	if ( is_array( $config['required'] ) ) {
 		return $config['required'];
 	}