Add support for -u to roundup-admin

Reimplement -u <login>[:<password>]. This opens the database as the
login and applies expected permissions. It also creates history
entries for the user. Note that the password is unused. The CLI has
full access to the files so a password check is not useful. Left an
edge case is when the login has a : in it. In this case it may not
work as expected. So don't do that.

Author: rouilj

CHANGES.txt

@@ -25,7 +25,14 @@ Features:
 - issue2550919 - Anti-bot signup using 4 second delay. New config.ini
   param [web] registration_delay must be set to 0 if template
   user.register.html is not modified.  See upgrading.txt for details.
-
+- Reimplement -u <login>[:<password>]. This opens the database as the
+  user and applies expected permissions. It also creates history
+  entries for the user. Note that the password is unused, no mention
+  of it is in the spec that I can find, so not sure what it was
+  supposed to be used for as the CLI has full access to the files so a
+  password check is not useful. An edge case is when the login has a :
+  in it. In this case it may not work as expected. So don't do that.
+  
 Fixed:
 
 - issue2550996 - Give better error message when running with -c

roundup/admin.py

@@ -121,7 +121,7 @@ def usage(self, message=''):
 
 Options:
  -i instance home  -- specify the issue tracker "home directory" to administer
- -u                -- the user[:password] to use for commands
+ -u                -- the user[:password] to use for commands (default admin)
  -d                -- print full designators not just class id numbers
  -c                -- when outputting lists of data, comma-separate them.
                       Same as '-S ","'.
@@ -1560,7 +1560,7 @@ def run_command(self, args):
 
         # only open the database once!
         if not self.db:
-            self.db = tracker.open('admin')
+            self.db = tracker.open(self.name)
 
         self.db.tx_Source = 'cli'
 
@@ -1620,13 +1620,13 @@ def main(self):
 
         # handle command-line args
         self.tracker_home = os.environ.get('TRACKER_HOME', '')
-        # TODO: reinstate the user/password stuff (-u arg too)
-        name = password = ''
+        self.name = 'admin'
+        self.password = ''  # unused
         if 'ROUNDUP_LOGIN' in os.environ:
             l = os.environ['ROUNDUP_LOGIN'].split(':')
-            name = l[0]
+            self.name = l[0]
             if len(l) > 1:
-                password = l[1]
+                self.password = l[1]
         self.separator = None
         self.print_designator = 0
         self.verbose = 0
@@ -1658,6 +1658,11 @@ def main(self):
                 self.separator = ' '
             elif opt == '-d':
                 self.print_designator = 1
+            elif opt == '-u':
+                l = arg.split(':')
+                self.name = l[0]
+                if len(l) > 1:
+                    self.password = l[1]
 
         # if no command - go interactive
         # wrap in a try/finally so we always close off the db