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.

Sign up for GitHub

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

Jump to bottom

Clarify CPE version #796

Open
sonnyvanlingen opened this issue Oct 16, 2024 · 4 comments · May be fixed by #819
Open

Clarify CPE version #796

sonnyvanlingen opened this issue Oct 16, 2024 · 4 comments · May be fixed by #819
Assignees
@tschmidtb51
Labels
csaf 2.1 csaf 2.1 work editor-revision already worked on in the editor revision editorial mostly nits and consistency

Comments

@sonnyvanlingen
Copy link

As can be read in 3.1.3.3.1 Full Product Name Type - Product Identification Helper - CPE, the text includes a reference to the CPE Naming Specification Version 2.3 for more details.

It is not documented explicitly that CPE version 2.3 is required, although one could deduce this from:

  1. the regular expression provided
  2. the reference to the CPE 2.3 Naming specification.

For usability reasons. I suggest to clarify in the text which version of CPE is used
@tschmidtb51
Copy link
Contributor

Even though we mention only CPE 2.3 as reference both, CPE 2.2 and CPE 2.3, are allowed.

@sonnyvanlingen
Copy link
Author

Right, indeed the regular expression seems to allow both.
My suggestion still stands, let's clarify textually :)

tschmidtb51 added a commit to tschmidtb51/csaf that referenced this issue Oct 17, 2024
@tschmidtb51
CPE
2ba8975 
- resolves oasis-tcs#796
- add note that CPE 2.2 and 2.3 is supported
@tschmidtb51 tschmidtb51 mentioned this issue Oct 17, 2024
Merged
@tschmidtb51 tschmidtb51 self-assigned this Oct 17, 2024
@tschmidtb51 tschmidtb51 added editorial mostly nits and consistency editor-revision already worked on in the editor revision csaf 2.1 csaf 2.1 work labels Oct 17, 2024
@tschmidtb51
Copy link
Contributor

I added a suggestion to resolve it with #800

@sonnyvanlingen
Copy link
Author

LGTM

@tschmidtb51 tschmidtb51 linked a pull request Oct 30, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tschmidtb51 tschmidtb51

Labels
csaf 2.1 csaf 2.1 work editor-revision already worked on in the editor revision editorial mostly nits and consistency
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Editor revision for TC meeting 2024-10-30 oasis-tcs/csaf
2 participants
@sonnyvanlingen @tschmidtb51