add tools get-creds command #1220
Conversation
|
Initial changes, will refactor get-git-password to support
printing only the string secret, supporting users who will pipe the secret. |
|
Can you run (if a pre-commit linter complains when you try to commit please ignore the pre-commit and just commit with |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| fmt.Println(state.LoggingSecret) | ||
| case "git": | ||
| message.Note("Git Server push password (username: " + state.GitServer.PushUsername + "):") | ||
| fmt.Println(state.GitServer.PushPassword) |
Check failure
Code scanning / CodeQL
Clear-text logging of sensitive information
| fmt.Println(state.GitServer.PushPassword) | ||
| case "git-readonly": | ||
| message.Note("Git Server (read-only) password (username: " + state.GitServer.PullUsername + "):") | ||
| fmt.Println(state.GitServer.PullPassword) |
Check failure
Code scanning / CodeQL
Clear-text logging of sensitive information
| fmt.Println(state.GitServer.PullPassword) | ||
| case "registry": | ||
| message.Note("Image Registry password (username: " + state.RegistryInfo.PushUsername + "):") | ||
| fmt.Println(state.RegistryInfo.PushPassword) |
Check failure
Code scanning / CodeQL
Clear-text logging of sensitive information
andrewg-xyz
requested review from
jeff-mccoy,
YrrepNoj,
Racer159,
Madeline-UX,
JasonvanBrackel and
RothAndrew
as code owners
…istry-credentials
…istry-credentials
…istry-credentials
…istry-credentials
…istry-credentials
Co-authored-by: Wayne Starr <[email protected]>
…istry-credentials
…istry-credentials
…istry-credentials
jeff-mccoy
deleted the
1098-zarf-cli-should-easily-show-git-and-registry-credentials
branch
## Description Add command to easily get credentials printed during zarf init (i.e. git, logging, registry) ## Related Issue Fixes #1098 ## Type of change - [ ] Bug fix (non-breaking change which fixes an issue) - [x] New feature (non-breaking change which adds functionality) - [ ] Other (security config, docs update, etc) ## Checklist before merging - [ ] Test, docs, adr added or updated as needed - [ ] [Contributor Guide Steps](https://github.com/defenseunicorns/zarf/blob/main/CONTRIBUTING.md#developer-workflow) followed
Description
Add command to easily get credentials printed during zarf init (i.e. git, logging, registry)
Related Issue
Fixes #1098
Type of change
Checklist before merging