Skip to content
Snippets Groups Projects

Mfa enforce settings

Merged Pavel Břoušek requested to merge mfaEnforceSettings into main

Created by: melanger

Merge request reports

Checking pipeline status.

Merged by avatar (Jul 18, 2025 8:58am UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Pavel Vyskočil
    Pavel Vyskočil @445753 started a thread on commit 99623d15
  • 51 53 throw new Exception('authswitcher:SwitchMfaMethods: ' . $variable . ' missing in state attributes');
    52 54 }
    53 55 }
    56
    57 public static function isMFAEnforced($state, $entityID = null)
    58 {
    59 if (!empty($state['Attributes'][AuthSwitcher::MFA_ENFORCE_SETTINGS])) {
    60 $settings = $state['Attributes'][AuthSwitcher::MFA_ENFORCE_SETTINGS];
    61 if (is_string($settings)) {
    62 $settings = json_decode($settings, true, 3, JSON_THROW_ON_ERROR);
    63 }
    64
    65 if (!empty($settings['all'])) {
    66 Logger::info(self::DEBUG_PREFIX . 'MFA was forced for all services by settings');
    • Maybe use the method name after the DEBUG_PREFIX to specify, where the message was logged. (The same for all log messages in this commit)

  • Review: Commented

    Only some comments for log messages

  • Author Contributor

    Merged by: melanger at 2022-07-27 18:22:37 UTC

  • Author Contributor

    Created by: github-actions[bot]

    :tada: This PR is included in version 10.7.0 :tada:

    The release is available on GitHub release

    Your semantic-release bot :package::rocket:

  • Please register or sign in to reply
    Loading