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

SSVC #871

Draft
wants to merge 21 commits into
base: editor-revision-2025-02-26
Choose a base branch
from
Draft

Conversation

tschmidtb51
Copy link
Contributor

  • addresses parts of Include support for SSVC #803
  • add SSVC decision point value selection 1.0.1 to schema
  • add SSVC decision point value selection 1.0.1 file into referenced schemas
  • adapt test scripts
  • add SSVC link in informative references
  • mention SSVC in design consideration principles
  • add SSVC to metrics section
  • add SSVC to guidance on size
  • add conversion rule
  • add mandatory test 6.1.43 to detect inconsistent SSVC IDs
  • add SSVC schema to testscript for test data
  • add mandatory test 6.1.44 for SSVC
  • add mandatory test 6.1.45 for SSVC Decision Point Namespace
  • add informative test 6.3.13 for Non-Latest SSVC decision point version
  • add optional test 6.2.33 for unknown SSVC namespaces
  • add optional test 6.2.34 for unknown SSVC roles
  • add invalid examples
  • add valid examples

- addresses parts of oasis-tcs#803
- add SSVC decision point value selection 1.0.1 to schema
- add SSVC decision point value selection 1.0.1 file into referenced schemas
- adapt test scripts
- addresses parts of oasis-tcs#803
- add SSVC link in informative references
- mention SSVC in design consideration principles
- add SSVC to metrics section
- addresses parts of oasis-tcs#803
- update SSVC key in schema to align with CVSS
- addresses parts of oasis-tcs#803
- update referenced SSVC schema to reflect change from CERTCC/SSVC#654
- reformat JSON schema
- addresses parts of oasis-tcs#803
- update referenced SSVC schema
- addresses parts of oasis-tcs#803
- add SSVC to guidance on size
…to ssvc

- resolve conflict in guidance-on-size.md by correct sorting
- addresses parts of oasis-tcs#803
- update link as indicated by CERT/CC
- addresses parts of oasis-tcs#803
- add conversion rule
- addresses parts of oasis-tcs#803
- add mandatory test 6.1.43 to detect inconsistent SSVC IDs
- add invalid examples
- add valid examples
- addresses parts of oasis-tcs#803
- add SSVC schema to testscript for test data
- addresses parts of oasis-tcs#803
- add mandatory test 6.1.44 for SSVC
- add invalid examples
- add valid examples
- addresses parts of oasis-tcs#803
- correct that 6.1.43 and 6.1.44 just have a single relevant path, not multiple
- addresses parts of oasis-tcs#803
- add mandatory test 6.1.45 for SSVC Decision Point Namespace
- add invalid examples
- add valid examples
- addresses parts of oasis-tcs#803
- add informative test 6.3.13 for Non-Latest SSVC decision point version
- add invalid example
- add valid example
- addresses parts of oasis-tcs#803
- add optional test 6.2.33 for unknown SSVC namespaces
- add invalid example
- add valid example
- addresses parts of oasis-tcs#803
- improve wording by using registered namespaces instead of reserved
- addresses parts of oasis-tcs#803
- add optional test 6.2.34 for unknown SSVC roles
- add invalid example
- add valid example
- addresses parts of oasis-tcs#803
- update with latest developments from SSVC
@tschmidtb51 tschmidtb51 requested a review from sthagen February 20, 2025 22:48
@tschmidtb51 tschmidtb51 self-assigned this Feb 20, 2025
@tschmidtb51 tschmidtb51 added editor-revision already worked on in the editor revision csaf 2.1 csaf 2.1 work labels Feb 20, 2025
@tschmidtb51
Copy link
Contributor Author

@sei-vsarvepalli Please review (especially the examples)

@tschmidtb51
Copy link
Contributor Author

This is in draft mode as we need to wait for the changes from CERTCC/SSVC#704

Copy link

@sei-vsarvepalli sei-vsarvepalli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly reviewed and run locally to verify CSAF test with what we hope to produce as well. Will need to look at updating the schema JSON file ./csaf_2.1/referenced_schema/certcc/Decision_Point_Value_Selection-1-0-1.schema.json according to what will be published as non-breaking changes to the schema in https://certcc.github.io/SSVC/data/schema/v1/ location. location.

* Classification for Document Distribution
* Traffic Light Protocol (TLP)
* Default Definition: https://www.first.org/tlp/
* Vulnerability Classification
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we move this down, so that we have some loose ordering and grouping like:

- Classification for Document Distribution
- Platform Data
- Vulnerability Categorization
- Vulnerability Classification
- Vulnerability Scoring

... and drop colons where only some same level entries have them or complete (my take is always to not have trailing punctuation when using these 2 dimensional lists engineers love soooo much :-)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done in 674e64d

- addresses review comment from oasis-tcs#871
- unify formatting
- sort list lexiographically
@tschmidtb51
Copy link
Contributor Author

Open ToDos:

  • Update referenced schemas once they are provided by upstream SSVC
  • Check tests (6.1.45 might be obsolete or need rephrasing)
  • Check test numbering as other PRs might be merge before this

Copy link
Contributor

@sthagen sthagen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM already.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
csaf 2.1 csaf 2.1 work editor-revision already worked on in the editor revision
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants