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

Don't fail if -u, -o or -a are not supplied for cve show command #92

Closed
MrSeccubus opened this issue Sep 17, 2024 · 1 comment · Fixed by #93
Closed

Don't fail if -u, -o or -a are not supplied for cve show command #92

MrSeccubus opened this issue Sep 17, 2024 · 1 comment · Fixed by #93

Comments

@MrSeccubus
Copy link
Contributor

Since the cve show command does not require a username it should not fail or prompt for an API key if none are supplied.

expected

$ cve show CVE-2024-27120
CVE-2024-27120
├─ State:	PUBLISHED
├─ Owning CNA:	DIVD
└─ Updated on:	Wed Aug 14 19:56:50 2024 +0000

reality

$ cve show CVE-2024-27120
Usage: cve [OPTIONS] COMMAND [ARGS]...
Try 'cve -h' for help.

Error: Missing option '-u' / '--username'.
$ cve -u unknown show CVE-2024-27120
Usage: cve [OPTIONS] COMMAND [ARGS]...
Try 'cve -h' for help.

Error: Missing option '-o' / '--org'
$  cve -u unknown -o unknonw show CVE-2024-27120
API key: ^C
Aborted!
$  cve -u unknown -o unknonw -a unknonw show CVE-2024-27120
CVE-2024-27120
├─ State:	PUBLISHED
├─ Owning CNA:	DIVD
└─ Updated on:	Wed Aug 14 19:56:50 2024 +0000
@mprpic
Copy link
Contributor

mprpic commented Oct 3, 2024

Fixed! ping and show don't require auth credentials any more. Please test if you can; I'll release a new version to PyPI next week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants