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

110 tutorial vulnerabilities #217

Merged
merged 4 commits into from
Dec 20, 2024
Merged

110 tutorial vulnerabilities #217

merged 4 commits into from
Dec 20, 2024

Conversation

tdruez
Copy link
Contributor

@tdruez tdruez commented Dec 19, 2024

No description provided.

@tdruez
Copy link
Contributor Author

tdruez commented Dec 19, 2024

@DennisClark the progress on the new tutorial is available at https://dejacode.readthedocs.io/en/110-tutorial-vulnerabilities/tutorial-4-vulnerabilities.html
Let me know about any additions and changes required.

Note that the link for the file used in the documentation is not working as the branch is not merged yet but it can be reached at https://github.com/aboutcode-org/dejacode/blob/110-tutorial-vulnerabilities/docs/sboms/starship_engine_2.0_scan_results.json

@DennisClark
Copy link
Member

@tdruez I think the section https://dejacode.readthedocs.io/en/110-tutorial-vulnerabilities/tutorial-4-vulnerabilities.html#review-vulnerabilities-affecting-your-product can be improved. The tutorial user is already on the Inventory tab, so instead:

  1. Navigate to the Vulnerabilities tab on the Product page, which presents a comprehensive view of all the Vulnerabilities for your Product filtered to those with a Risk greater than the Risk Threshold defined for your Dataspace.

(show the initial presentation of the Vulnerabilities tab here)

  1. You can sort and filter by Risk, Exploitability and Severity, as well as other fields, to focus on specific Vulnerabilities.

The next section is fine, but the first sentence needs correcting to:

  1. Review each vulnerability in the Vulnerabilities tab.

rather than "Vulnerability tab".

Those are my only suggestions; thanks for doing this.

Signed-off-by: tdruez <tdruez@nexb.com>
@tdruez tdruez merged commit b614630 into main Dec 20, 2024
3 checks passed
@tdruez tdruez deleted the 110-tutorial-vulnerabilities branch December 20, 2024 08:42
@DennisClark
Copy link
Member

@tdruez the refined tutorial looks good, thanks. No problems found.

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

Successfully merging this pull request may close these issues.

2 participants