Skip to content

Read password from stdin for podman-hpc login - #444

Open
EZoni wants to merge 3 commits into
BLAST-AI-ML:mainfrom
EZoni:podmanhpc_password_stdin
Open

Read password from stdin for podman-hpc login#444
EZoni wants to merge 3 commits into
BLAST-AI-ML:mainfrom
EZoni:podmanhpc_password_stdin

Conversation

@EZoni

@EZoni EZoni commented May 8, 2026

Copy link
Copy Markdown
Member

Overview

Small change extracted from #432, which in turn was extracted from #428, now replaced by #439.

Avoid exposing REGISTRY_PASSWORD in the process list (ps output) by piping it via stdin instead of passing it as a --password argument.

Note that all documented examples at https://docs.podman.io/en/stable/markdown/podman-login.1.html (podman instead of podman-hpc) use --password-stdin.

Notes

The original suggestion in #432 and #428 had printf '%s\n' instead of echo to guard against passwords starting with -n, -e, or containing backslash sequences, but it should be robust enough to use echo in this case.

@EZoni EZoni added the ml Changes related to the ML models label May 8, 2026
@EZoni
EZoni requested a review from ax3l June 4, 2026 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ml Changes related to the ML models

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants