Open
Description
I have submitted a PR here: #1431
Currently, saml2aws --version
prints the application version to stderr
. This change modifies the behavior to print the version string to stdout
instead while other logs and errors remain on stderr
.
Reasoning:
Printing the version to stdout
aligns better with common scripting practices and standard CLI tool behavior for version reporting. This makes it easier to capture the version string programmatically without needing stderr
redirection (2>&1
).
For example, users with bootstrapping or environment setup scripts that check dependency versions currently need to add special handling for saml2aws
like
version = result.stdout.strip() if tool != "saml2aws" else result.stderr.strip()
or like
VERSION=$(saml2aws --version 2>&1)
Routing --version
output to stdout
removes this inconsistency.