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

2 files changed, 2 insertions, 2 deletions

diff --git a/Bugzilla/Auth/Login/Stack.pm b/Bugzilla/Auth/Login/Stack.pm
index ab9a93bce..a5752f22b 100644
--- 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;
diff --git a/Bugzilla/Auth/Verify/Stack.pm b/Bugzilla/Auth/Verify/Stack.pm
index 0ddb9a441..c23b532fd 100644
--- 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;