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

Providing invalid --keyring-backend value not validated and defaults to file without notifying user #2759

Closed
AndrewSisley opened this issue Jun 24, 2024 · 0 comments · Fixed by #2784
Assignees
Labels
area/cli Related to the CLI binary bug Something isn't working

Comments

@AndrewSisley
Copy link
Contributor

For example, executing defradb keyring import bar efcc38b57ccb9ddba3cf85a856acaeb28bbe7360fc0fe37630b3e91ff0bca86c --keyring-backend fdggsfa prompts for password and then completes without outputting anything (#2758).

The imported key is then accessible via defradb keyring export bar (using the file backend).

@AndrewSisley AndrewSisley added bug Something isn't working area/cli Related to the CLI binary labels Jun 24, 2024
@nasdf nasdf mentioned this issue Jun 26, 2024
4 tasks
@nasdf nasdf self-assigned this Jun 26, 2024
nasdf added a commit that referenced this issue Jun 27, 2024
## Relevant issue(s)

Resolves #2759
Resolves #2758
Resolves #2757

## Description

This PR adds more logs to the keyring to improve the user experience.

## Tasks

- [x] I made sure the code is well commented, particularly
hard-to-understand areas.
- [x] I made sure the repository-held documentation is changed
accordingly.
- [x] I made sure the pull request title adheres to the conventional
commit style (the subset used in the project can be found in
[tools/configs/chglog/config.yml](tools/configs/chglog/config.yml)).
- [x] I made sure to discuss its limitations such as threats to
validity, vulnerability to mistake and misuse, robustness to
invalidation of assumptions, resource requirements, ...

## How has this been tested?

- unit test

Specify the platform(s) on which this was tested:
- MacOS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/cli Related to the CLI binary bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants