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

JSON output #82

Open
jackd248 opened this issue Oct 4, 2022 · 5 comments · May be fixed by #104
Open

JSON output #82

jackd248 opened this issue Oct 4, 2022 · 5 comments · May be fixed by #104
Labels
good first issue Good for newcomers

Comments

@jackd248
Copy link

jackd248 commented Oct 4, 2022

Will there be an option to display the audit results in JSON format?

@jeemok
Copy link
Owner

jeemok commented Oct 5, 2022

We could add the option, but you could also get the JSON format audit result (with more details) without the help of this library, e.g. npm audit --json https://docs.npmjs.com/cli/v8/commands/npm-audit

@jackd248
Copy link
Author

jackd248 commented Oct 7, 2022

Thanks for the hint, so I will filter the necessary information by my own.

@jackd248 jackd248 closed this as completed Oct 7, 2022
@jfaissolle
Copy link

We could add the option, but you could also get the JSON format audit result (with more details) without the help of this library, e.g. npm audit --json https://docs.npmjs.com/cli/v8/commands/npm-audit

OK, but how do we integrate this tool in a CI pipeline ? The added value of better-npm-audit is exception management which we do not get with just npm audit --json

@jeemok
Copy link
Owner

jeemok commented Aug 29, 2024

OK, but how do we integrate this tool in a CI pipeline ? The added value of better-npm-audit is exception management which we do not get with just npm audit --json

hmm that's true. I guess we could add an output option for JSON

@jeemok jeemok reopened this Aug 29, 2024
@jeemok jeemok added the good first issue Good for newcomers label Aug 29, 2024
@eliasm307
Copy link

i'm looking into using this package and do some actions with found vulnerabilities, so a JSON output would be good

It doesnt seem like someone is working on this so I'll do it, let me know if there are any objections

@eliasm307 eliasm307 linked a pull request Dec 1, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants