Move login command inside try so we clear env correctly

pull/1421/head
YuviPanda 2025-02-28 15:07:43 -08:00
rodzic d3facba5fb
commit 342eea96a8
1 zmienionych plików z 12 dodań i 12 usunięć

Wyświetl plik

@ -190,19 +190,19 @@ class DockerEngine(ContainerEngine):
shutil.copy2(dc_path, new_dc_path)
os.environ["DOCKER_CONFIG"] = d
proc = subprocess.run(
[
"docker",
"login",
"--username",
username,
"--password-stdin",
registry,
],
input=password.encode(),
check=True,
)
try:
subprocess.run(
[
"docker",
"login",
"--username",
username,
"--password-stdin",
registry,
],
input=password.encode(),
check=True,
)
yield
finally:
if old_dc_path: