Works around rate limiting by trying to apply CL unauthenticated first.

apply_issue.py

 #!/usr/bin/env python
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Applies an issue from Rietveld.
 """
 
 import getpass
 import logging
@@ skipping 13 matching lines @@
 
 BASE_DIR = os.path.dirname(os.path.abspath(__file__))
 
 
 def main():
   parser = optparse.OptionParser(description=sys.modules[__name__].__doc__)
   parser.add_option(
       '-v', '--verbose', action='count', default=0,
       help='Prints debugging infos')
   parser.add_option(
-      '-e', '--email', default='',
+      '-e', '--email',
       help='Email address to access rietveld.  If not specified, anonymous '
            'access will be used.')
   parser.add_option(
       '-w', '--password', default=None,
       help='Password for email addressed.  Use - to read password from stdin.')
   parser.add_option(
       '-i', '--issue', type='int', help='Rietveld issue number')
   parser.add_option(
       '-p', '--patchset', type='int', help='Rietveld issue\'s patchset number')
   parser.add_option(
@@ skipping 15 matching lines @@
     parser.error('Extra argument(s) "%s" not understood' % ' '.join(args))
   if not options.issue:
     parser.error('Require --issue')
   options.server = options.server.rstrip('/')
   if not options.server:
     parser.error('Require a valid server')
 
   if options.password == '-':
     options.password = sys.stdin.readline().strip()
 
-  obj = rietveld.Rietveld(options.server, options.email, options.password)
+  # Always try un-authenticated first.
+  # TODO(maruel): Use OAuth2 properly so we don't hit rate-limiting on login
+  # attempts.
+  # Bad except clauses order (HTTPError is an ancestor class of
+  # ClientLoginError)
+  # pylint: disable=E0701
+  obj = rietveld.Rietveld(options.server, '', None)
+  properties = None
   try:
     properties = obj.get_issue_properties(options.issue, False)
+  except urllib2.HTTPError, e:
+    if e.getcode() != 302:
+      raise
+    # TODO(maruel): A few 'Invalid username or password.' are printed first, we
+    # should get rid of those.
   except rietveld.upload.ClientLoginError, e:
-    if sys.stdout.closed:
-      print >> sys.stderr, 'Accessing the issue requires login.'
-      return 1
+    # Fine, we'll do proper authentication.
+    pass
+  if properties is None:
+    if options.email is not None:
+      try:
+        obj = rietveld.Rietveld(options.server, options.email, options.password)
+      except rietveld.upload.ClientLoginError, e:
+        if sys.stdout.closed:
+          print >> sys.stderr, 'Accessing the issue requires login.'
+          return 1
     print('Accessing the issue requires login.')
     obj = rietveld.Rietveld(options.server, None, None)
     properties = obj.get_issue_properties(options.issue, False)
 
   if not options.patchset:
     options.patchset = properties['patchsets'][-1]
     print('No patchset specified. Using patchset %d' % options.patchset)
 
   print('Downloading the patch.')
   try:
@@ skipping 44 matching lines @@
       if sys.platform == 'win32':
         gclient_path += '.bat'
       return subprocess.call(
           [gclient_path, 'sync', '--revision', base_rev], cwd=gclient_root)
   return 0
 
 
 if __name__ == "__main__":
   fix_encoding.fix_encoding()
   sys.exit(main())