Skip to content

Commit

Permalink
Don't write output to stdout
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
jenshnielsen committed Apr 26, 2024
1 parent 6a05b27 commit b9f5c99
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/artifacts_keyring/plugin.py
Original file line number Diff line number Diff line change
Expand Up @@ -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()

Expand Down

0 comments on commit b9f5c99

Please sign in to comment.