Bug 488467: Verify and Login auth methods were being called in a random order,…

Bug 488467: Verify and Login auth methods were being called in a random order, causing sudo sessions to frequently not need the user to re-enter their password. Patch by Max Kanat-Alexander <mkanat@bugzilla.org> r=LpSolit, a=LpSolit

Bug 488467: Verify and Login auth methods were being called in a random order,…
27d919b3 · mkanat%bugzilla.org · 3b1bc81a · 27d919b3 · 27d919b3
Commit 27d919b3 authored Apr 17, 2009 by mkanat%bugzilla.org
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

Stack.pm Bugzilla/Auth/Login/Stack.pm +1 -1

Stack.pm Bugzilla/Auth/Verify/Stack.pm +1 -1

No files found.
--- a/Bugzilla/Auth/Login/Stack.pm
+++ b/Bugzilla/Auth/Login/Stack.pm
@@ -38,7 +38,7 @@ sub new {
    Bugzilla::Hook::process('auth-login_methods', { modules => \%methods });

    $self->{_stack} = [];
-    foreach my $login_method (keys %methods) {
+    foreach my $login_method (split(',', $list)) {
        my $module = $methods{$login_method};
        require $module;
        $module =~ s|/|::|g;

--- a/Bugzilla/Auth/Verify/Stack.pm
+++ b/Bugzilla/Auth/Verify/Stack.pm
@@ -33,7 +33,7 @@ sub new {
    Bugzilla::Hook::process('auth-verify_methods', { modules => \%methods });

    $self->{_stack} = [];
-    foreach my $verify_method (keys %methods) {
+    foreach my $verify_method (split(',', $list)) {
        my $module = $methods{$verify_method};
        require $module;
        $module =~ s|/|::|g;