[GH-ISSUE #203] Dynamic auth #166

New issue

Closed

opened 2026-02-25 21:34:20 +03:00 by kerem · 4 comments

kerem commented

2026-02-25 21:34:20 +03:00

Owner

Originally created by @hartois on GitHub (Jun 14, 2017).
Original GitHub issue: https://github.com/cypht-org/cypht/issues/203

Originally assigned to: @jasonmunro on GitHub.

In fresh code of function setup_auth($config):

    if ($auth_type == 'dynamic' && !in_array('dynamic_login', $config->get_modules(), true)) {
        Hm_Functions::cease('Invalid auth configuration');
    }
    if ($auth_type && in_array($auth_type, array('DB', 'LDAP', 'IMAP', 'POP3'), true)) {
        $auth_class = sprintf('Hm_Auth_%s', $auth_type);
    }
    elseif ($auth_type == 'custom') {
        $auth_class = 'Custom_Auth';
    }
   if (!$auth_class) {                  <<<<<<<<<<< with "dynamic" EXIT HERE
        Hm_Functions::cease('Invalid auth configuration');
    }
    else {
        return $auth_class;
    }

dynamic auth cannot be used

Originally created by @hartois on GitHub (Jun 14, 2017). Original GitHub issue: https://github.com/cypht-org/cypht/issues/203 Originally assigned to: @jasonmunro on GitHub. In fresh code of function setup_auth($config): ``` if ($auth_type == 'dynamic' && !in_array('dynamic_login', $config->get_modules(), true)) { Hm_Functions::cease('Invalid auth configuration'); } if ($auth_type && in_array($auth_type, array('DB', 'LDAP', 'IMAP', 'POP3'), true)) { $auth_class = sprintf('Hm_Auth_%s', $auth_type); } elseif ($auth_type == 'custom') { $auth_class = 'Custom_Auth'; } if (!$auth_class) { <<<<<<<<<<< with "dynamic" EXIT HERE Hm_Functions::cease('Invalid auth configuration'); } else { return $auth_class; } ``` dynamic auth cannot be used

kerem

2026-02-25 21:34:20 +03:00

closed this issue
added the
bug

framework
labels

kerem commented

2026-02-25 21:34:20 +03:00

Author

Owner

@jasonmunro commented on GitHub (Jun 14, 2017):

Oops, my bad. I will dig into this and get it fixed up ASAP. Thanks for the report

@jasonmunro commented on GitHub (Jun 14, 2017): Oops, my bad. I will dig into this and get it fixed up ASAP. Thanks for the report

kerem commented

2026-02-25 21:34:20 +03:00

Author

Owner

@jasonmunro commented on GitHub (Jun 15, 2017):

This should be fixed now. Sorry about that! I refactored setup_session and setup_auth into a class, and cleanup up the code while I was at it.

@jasonmunro commented on GitHub (Jun 15, 2017): This should be fixed now. Sorry about that! I refactored setup_session and setup_auth into a class, and cleanup up the code while I was at it.

kerem commented

2026-02-25 21:34:20 +03:00

Author

Owner

@jasonmunro commented on GitHub (Jun 16, 2017):

I believe this is fixed so I'm going to close this issue, if you still have problems feel free to re-open or create a new one. Thanks!

@jasonmunro commented on GitHub (Jun 16, 2017): I believe this is fixed so I'm going to close this issue, if you still have problems feel free to re-open or create a new one. Thanks!

kerem commented

2026-02-25 21:34:20 +03:00

Author

Owner

@hartois commented on GitHub (Jun 17, 2017):

Thanks

@hartois commented on GitHub (Jun 17, 2017): Thanks