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

Persist impact, recommendation and source fields in FindingType objects in XTDB #1126

Merged
merged 7 commits into from
Jun 7, 2023

Conversation

Lisser
Copy link
Contributor

@Lisser Lisser commented Jun 7, 2023

Changes

Add impact, recommendation and source fields to kat_finding_types.json
Add impact, recommendation and source fields to FindingType model
Yield impact, recommendation and source from KATFindingType normalizer
Yield source from CVEFindingType normalizer
Yield source from SnykFindingType normalizer

Issue link

Closes N/A

Proof

Screenshot 2023-06-07 at 12 12 11


Code Checklist

  • All the commits in this PR are properly PGP-signed and verified;
  • This PR only contains functionality relevant to the issue; tickets have been created for newly discovered issues.
  • I have written unit tests for the changes or fixes I made.
  • For any non-trivial functionality, I have added integration and/or end-to-end tests.
  • I have performed a self-review of my code and refactored it to the best of my abilities.

Communication

  • I have informed others of any required .env changes files if required and changed the .env-dist accordingly.
  • I have made corresponding changes to the documentation, if necessary.

Checklist for code reviewers:

Copy-paste the checklist from the docs/source/templates folder into your comment.


Checklist for QA:

Copy-paste the checklist from the docs/source/templates folder into your comment.

@Lisser Lisser force-pushed the add-fields-to-kat-finding-types-json branch from ad99f59 to 910ac8f Compare June 7, 2023 09:02
@Lisser Lisser force-pushed the add-fields-to-kat-finding-types-json branch from 910ac8f to 7b9b0f5 Compare June 7, 2023 09:08
@Lisser Lisser marked this pull request as ready for review June 7, 2023 09:37
@Lisser Lisser requested a review from a team as a code owner June 7, 2023 09:37
@Lisser Lisser changed the title impact and recommendation fields to kat_finding_types.json impact and recommendation fields Jun 7, 2023
@Lisser Lisser changed the title impact and recommendation fields Persist impact and recommendation fields in FindingType objects in XTDB Jun 7, 2023
@Lisser Lisser changed the title Persist impact and recommendation fields in FindingType objects in XTDB Persist impact, recommendation and source fields in FindingType objects in XTDB Jun 7, 2023
@praseodym
Copy link
Contributor

What else needs to be done before the data that's now in data/kat_finding_types.json can be removed from rocky/OOI_database_seed.json?

@Lisser
Copy link
Contributor Author

Lisser commented Jun 7, 2023

What else needs to be done before the data that's now in data/kat_finding_types.json can be removed from rocky/OOI_database_seed.json?

Basically, stop accessing it in Rocky. This is being worked on in #1118

@dekkers
Copy link
Contributor

dekkers commented Jun 7, 2023

What else needs to be done before the data that's now in data/kat_finding_types.json can be removed from rocky/OOI_database_seed.json?

There are also a few descriptions of models in the seed. I think it would be best to move those descriptions to the models itself.

@dekkers dekkers merged commit 88cb93f into main Jun 7, 2023
@dekkers dekkers deleted the add-fields-to-kat-finding-types-json branch June 7, 2023 13:08
jpbruinsslot added a commit that referenced this pull request Jun 12, 2023
* main:
  Add bit to set default values for FindingType risk levels in Octopoes (#1075)
  Fix thread termination in Mula (#1003)
  test(boefjes): snyk (#1116)
  Add endpoints in Octopoes for bulk operations in the object list page (#1067)
  Remove tasks and items on mutation delete (#1090)
  Persist impact, recommendation and source fields in FindingType objects in XTDB (#1126)
  Handle an empty plugin.consumes field for the plugin detail page (#1104)
  Add script to automatically backport PR to release branch (#1097)
  Fix typos in 'no organizations found' message (#1123)
  Finding Types Boefjes (#1056)
  add findingtype files (#1117)
  Remove containers after `docker-compose run` (#1112)
  Bump cryptography from 39.0.1 to 41.0.0 in /boefjes/boefjes/plugins/kat_ssl_certificates (#1099)
  Bump cryptography from 40.0.2 to 41.0.1 (#1108)
  Bump cryptography from 40.0.2 to 41.0.0 in /bytes (#1100)
  Fix failing test-debian-install in CI (#1111)
  Remove unused boefje fields when creating a BoefjeTask object to send to the scheduler (#1103)
  add 'ideas' as a category in project guidelines (#1105)
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.

3 participants