Allow to skip EnsureAuthenticated for Gerrit.

git_cl.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.
 
 # Copyright (C) 2008 Evan Martin <martine@danga.com>
 
 """A git-command for integrating reviews on Rietveld and Gerrit."""
 
 from distutils.version import LooseVersion
@@ skipping 571 matching lines @@
     self.default_server = None
     self.cc = None
     self.root = None
     self.is_git_svn = None
     self.svn_branch = None
     self.tree_status_url = None
     self.viewvc_url = None
     self.updated = False
     self.is_gerrit = None
     self.squash_gerrit_uploads = None
+    self.gerrit_skip_ensure_authenticated = None
     self.git_editor = None
     self.project = None
     self.force_https_commit_url = None
     self.pending_ref_prefix = None
 
   def LazyUpdateIfNeeded(self):
     """Updates the settings from a codereview.settings file, if available."""
     if not self.updated:
       # The only value that actually changes the behavior is
       # autoupdate = "false". Everything else means "true".
@@ skipping 162 matching lines @@
     return self.is_gerrit
 
   def GetSquashGerritUploads(self):
     """Return true if uploads to Gerrit should be squashed by default."""
     if self.squash_gerrit_uploads is None:
       self.squash_gerrit_uploads = (
           RunGit(['config', '--bool', 'gerrit.squash-uploads'],
                  error_ok=True).strip() == 'true')
     return self.squash_gerrit_uploads
 
+  def GetGerritSkipEnsureAuthenticated(self):
+    """Return True if EnsureAuthenticated should not be done for Gerrit
+    uploads."""
+    if self.gerrit_skip_ensure_authenticated is None:
+      self.gerrit_skip_ensure_authenticated = (
+          RunGit(['config', '--bool', 'gerrit.skip_ensure_authenticated'],
+                 error_ok=True).strip() == 'true')
+    return self.gerrit_skip_ensure_authenticated
+
   def GetGitEditor(self):
     """Return the editor specified in the git config, or None if none is."""
     if self.git_editor is None:
       self.git_editor = self._GetConfig('core.editor', error_ok=True)
     return self.git_editor or None
 
   def GetLintRegex(self):
     return (self._GetRietveldConfig('cpplint-regex', error_ok=True) or
             DEFAULT_LINT_REGEX)
 
@@ skipping 1231 matching lines @@
         self._gerrit_host = '.'.join(parts)
         self._gerrit_server = 'https://%s' % self._gerrit_host
     return self._gerrit_server
 
   @classmethod
   def IssueSettingSuffix(cls):
     return 'gerritissue'
 
   def EnsureAuthenticated(self, force):
     """Best effort check that user is authenticated with Gerrit server."""
+    if settings.GetGerritSkipEnsureAuthenticated():
+      # For projects with unusual authentication schemes.
+      # See http://crbug.com/603378.
+      return
     # Lazy-loader to identify Gerrit and Git hosts.
     if gerrit_util.GceAuthenticator.is_gce():
       return
     self.GetCodereviewServer()
     git_host = self._GetGitHost()
     assert self._gerrit_server and self._gerrit_host
     cookie_auth = gerrit_util.CookiesAuthenticator()
 
     gerrit_auth = cookie_auth.get_auth_header(self._gerrit_host)
     git_auth = cookie_auth.get_auth_header(git_host)
@@ skipping 675 matching lines @@
   SetProperty('run-post-upload-hook', 'RUN_POST_UPLOAD_HOOK',
               unset_error_ok=True)
 
   if 'GERRIT_HOST' in keyvals:
     RunGit(['config', 'gerrit.host', keyvals['GERRIT_HOST']])
 
   if 'GERRIT_SQUASH_UPLOADS' in keyvals:
     RunGit(['config', 'gerrit.squash-uploads',
             keyvals['GERRIT_SQUASH_UPLOADS']])
 
+  if 'GERRIT_SKIP_ENSURE_AUTHENTICATED' in keyvals:
+    RunGit(['config', 'gerrit.skip_ensure_authenticated',
+            keyvals['GERRIT_SKIP_ENSURE_AUTHENTICATED']])
+
   if 'PUSH_URL_CONFIG' in keyvals and 'ORIGIN_URL_CONFIG' in keyvals:
     #should be of the form
     #PUSH_URL_CONFIG: url.ssh://gitrw.chromium.org.pushinsteadof
     #ORIGIN_URL_CONFIG: http://src.chromium.org/git
     RunGit(['config', keyvals['PUSH_URL_CONFIG'],
             keyvals['ORIGIN_URL_CONFIG']])
 
 
 def urlretrieve(source, destination):
   """urllib is broken for SSL connections via a proxy therefore we
@@ skipping 2077 matching lines @@
 if __name__ == '__main__':
   # These affect sys.stdout so do it outside of main() to simplify mocks in
   # unit testing.
   fix_encoding.fix_encoding()
   setup_color.init()
   try:
     sys.exit(main(sys.argv[1:]))
   except KeyboardInterrupt:
     sys.stderr.write('interrupted\n')
     sys.exit(1)