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.
I have submitted a PR here: #1431
Currently,
saml2aws --versionprints the application version tostderr. This change modifies the behavior to print the version string tostdoutinstead while other logs and errors remain onstderr.Reasoning:
Printing the version to
stdoutaligns better with common scripting practices and standard CLI tool behavior for version reporting. This makes it easier to capture the version string programmatically without needingstderrredirection (2>&1).For example, users with bootstrapping or environment setup scripts that check dependency versions currently need to add special handling for
saml2awslikeor like
Routing
--versionoutput tostdoutremoves this inconsistency.