10up / wpcli-vulnerability-scanner

WP-CLI command for checking installed plugins and themes for vulnerabilities reported on wpvulndb.com
MIT License
278 stars 40 forks source link

Fix/ids output format #51

Closed pabamato closed 2 years ago

pabamato commented 3 years ago

Description of the Change

Fix output format for ids : --format=ids Output now is a list of vulnerabilities ids from wpscan.com API

Benefits

Correct output format

Possible Drawbacks

Verification Process

run $ wp vuln status --format=ids

Checklist:

Applicable Issues

Issue: https://github.com/10up/wpcli-vulnerability-scanner/issues/24 Dependency: https://github.com/10up/wpcli-vulnerability-scanner/pull/50

Changelog Entry

Fixed:

image

pabamato commented 3 years ago

@jeffpaul @TheLastCicada : This is taking advantage of methods introduced in https://github.com/10up/wpcli-vulnerability-scanner/pull/50, as you can see has taken that as a base branch. Let me know if you consider is best for history preserve reasons to start from the develop branch and then manually merge these methods. Also, I'll need help with unit test. Thanks!

jeffpaul commented 2 years ago

Confirmed the following with @kojraai:

looks good to me as well, I can see the expected output.