Skip to content

Route --version output to stdout #1432

Open
@neo-anderson

Description

@neo-anderson

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.

Metadata

Metadata

Assignees

No one assigned

    Labels

    component:cliIssues related to the command-line interfacepriority:lowMinor issue, nice-to-havestatus:ready-for-devIssue is ready for development

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions