[git_cl] Fix cookie setting

Bug: b/351071334
Change-Id: I0245e38b71b8c65e77dcc2aeee040121c725b440
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/5718644
Commit-Queue: Allen Li <ayatane@chromium.org>
Reviewed-by: Yiwei Zhang <yiwzhang@google.com>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
changes/44/5718644/3
Allen Li 1 year ago committed by LUCI CQ
parent fa224cdb07
commit 2fe4794943

@ -3698,7 +3698,7 @@ class GitAuthConfigChanger(object):
# #
# Increment this when making changes to the config, so that reliant # Increment this when making changes to the config, so that reliant
# code can determine whether the config needs to be re-applied. # code can determine whether the config needs to be re-applied.
VERSION: int = 1 VERSION: int = 2
def __init__( def __init__(
self, self,
@ -3797,14 +3797,16 @@ class GitAuthConfigChanger(object):
def _apply_gitcookies(self, cwd: str) -> None: def _apply_gitcookies(self, cwd: str) -> None:
"""Apply config changes relating to gitcookies.""" """Apply config changes relating to gitcookies."""
# TODO(ayatane): Clear invalid setting. Remove line after a few weeks
self._set_config(cwd, 'http.gitcookies', None, modify_all=True)
if self.mode == GitConfigMode.NEW_AUTH: if self.mode == GitConfigMode.NEW_AUTH:
# Override potential global gitcookie config # Override potential global setting
self._set_config(cwd, 'http.gitcookies', '', modify_all=True) self._set_config(cwd, 'http.cookieFile', '', modify_all=True)
elif self.mode == GitConfigMode.NEW_AUTH_SSO: elif self.mode == GitConfigMode.NEW_AUTH_SSO:
# Override potential global gitcookie config # Override potential global setting
self._set_config(cwd, 'http.gitcookies', '', modify_all=True) self._set_config(cwd, 'http.cookieFile', '', modify_all=True)
elif self.mode == GitConfigMode.OLD_AUTH: elif self.mode == GitConfigMode.OLD_AUTH:
self._set_config(cwd, 'http.gitcookies', None, modify_all=True) self._set_config(cwd, 'http.cookieFile', None, modify_all=True)
else: else:
raise TypeError(f'Invalid mode {self.mode!r}') raise TypeError(f'Invalid mode {self.mode!r}')

Loading…
Cancel
Save