Fix basic authentication.

Instatiating the login action would fail if the user is not set.
We now first set the user to anonymous and then try basic authentication
if enabled.

Author: schlatterbeck

CHANGES.txt

@@ -42,6 +42,9 @@ Fixed:
   thanks Nathan Russell. (John Kristensen)
 - issue2550756: Fix `oder' typo in mailer.Mailer.bounce_message docstring,
   thanks W. Trevor King (John Kristensen)
+- Fix basic authentication: instatiating the login action would fail if
+  the user is not set. We now first set the user to anonymous and then
+  try basic authentication if enabled.
 
 
 2012-05-15: 1.4.20

roundup/cgi/client.py

@@ -706,6 +706,10 @@ def determine_user(self):
                         pass
                     username, password = decoded.split(':', 1)
                     try:
+                        # Current user may not be None, otherwise
+                        # instatiation of the login action will fail.
+                        # So we set the user to anonymous first.
+                        self.make_user_anonymous()
                         login = self.get_action_class('login')(self)
                         login.verifyLogin(username, password)
                     except LoginError, err: