From b9f5c99734a4934b8a6dadb72cb5e77249554ee8 Mon Sep 17 00:00:00 2001 From: "Jens H. Nielsen" Date: Fri, 26 Apr 2024 09:39:53 +0200 Subject: [PATCH] Don't write output to stdout Writing output to stdout breaks the usage of artifacts-keyring with the keyring CLI since it is impossible to discriminate between the output of the CLI and other text writen to stdout. --- src/artifacts_keyring/plugin.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/artifacts_keyring/plugin.py b/src/artifacts_keyring/plugin.py index 457c678..048509c 100644 --- a/src/artifacts_keyring/plugin.py +++ b/src/artifacts_keyring/plugin.py @@ -115,8 +115,8 @@ def _get_credentials_from_credential_provider(self, url, is_retry): # from it for Device Flow authentication. for stderr_line in iter(proc.stderr.readline, b''): line = stderr_line.decode("utf-8", "ignore") - sys.stdout.write(line) - sys.stdout.flush() + sys.stderr.write(line) + sys.stderr.flush() proc.wait()