Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

zarf registry login hangs indefinitely with --password-stdin flag #1444

Closed
mikevanhemert opened this issue Mar 16, 2023 · 1 comment
Closed
Labels
enhancement ✨ New feature or request upstream

Comments

@mikevanhemert
Copy link

Environment

Device and OS:

  • On-prem customer server with Ubuntu 22.04.2 LTS
  • System76 Darter Pro with Pop!_OS 22.04 LTS

App version: v0.25.0-rc1
Kubernetes distro being used: n/a
Other:

Steps to reproduce

  1. Attempt zarf registry login with valid credentials to registry1.dso.mil
    zarf  tools registry login -u [username] --password-stdin registry1.dso.mil -l debug
    # enter your password

Expected result

Zarf prompts for password or entering password immediately proceeds to registry login

Actual Result

Zarf hangs with no indication of success, failure, next steps, etc.

Visual Proof (screenshots, videos, text, etc)

n/a

Severity/Priority

Low

Additional Context

Zarf registry login with valid credentials and the -p flag is successful

zarf tools registry login -u [username] -p [password] registry1.dso.mil -l debug
2023/03/16 14:17:22 logged in via /home/[system username]/.docker/config.json
@Racer159
Copy link
Contributor

The zarf tools registry login command is from crane and pulls the password from a pipe like:

echo $MYPASSWORD | zarf  tools registry login -u [username] --password-stdin registry1.dso.mil

The closest issue I could find upstream is this: google/go-containerregistry#718

Racer159 added a commit that referenced this issue Feb 23, 2024
## Description

This is a test for fixes to intermittent hanging.

## Related Issue

Relates to #1444

## Type of change

- [X] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Other (security config, docs update, etc)

## Checklist before merging

- [X] Test, docs, adr added or updated as needed
- [X] [Contributor Guide
Steps](https://github.com/defenseunicorns/zarf/blob/main/CONTRIBUTING.md#developer-workflow)
followed
@Noxsios Noxsios closed this as completed Mar 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ✨ New feature or request upstream
Projects
None yet
Development

No branches or pull requests

3 participants