InTheForest/chromium_depot_tools
1
0
Fork 0
mirror of https://chromium.googlesource.com/chromium/tools/depot_tools.git synced 2026-01-11 18:51:29 +00:00
Code Issues Packages Projects Releases Wiki Activity

[git_cl] Remove the auto cleanup from disabled new auth

Browse Source 
This is in preparation for removing the old auto config code.  New
auth has been rolled out stably for a long time now, and in any case
the auto config/cleanup code isn't super reliable (hence why we're
removing it).

We remove this final callsite before deleting all of the relevant
code, and leave a message of assistance in case anyone needs it.

Bug: 446999231
Change-Id: Ia13e5d75a89a4ddfb178ee9e4fe2090596493218
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/6976601
Commit-Queue: Allen Li <ayatane@chromium.org>
Reviewed-by: Gavin Mak <gavinmak@google.com>
This commit is contained in:
Allen Li
2025-09-23 19:52:53 -07:00
committed by LUCI CQ
parent 813b6f7588
commit 838e4c6aa7
2 changed files with 12 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
git_auth.py
View File

@@ -292,10 +292,8 @@ def ConfigureGlobal(cwd: str, remote_url: str) -> None:
def ClearRepoConfig(cwd: str, cl: git_cl.Changelist) -> None:
"""Clear the current Git repo authentication."""
logging.debug('Clearing current Git repo authentication...')
c = ConfigChanger.new_from_env(cwd, cl)
c.mode = ConfigMode.NO_AUTH
c.apply(cwd)
# TODO(ayatane): Disable prior to removal
return
class _ConfigError(Exception):