feat(CSAF2.1): #447 add recommended test test 6.2.47#458
Open
rainer-exxcellent wants to merge 2 commits intomainfrom
Open
feat(CSAF2.1): #447 add recommended test test 6.2.47#458rainer-exxcellent wants to merge 2 commits intomainfrom
rainer-exxcellent wants to merge 2 commits intomainfrom
Conversation
rainer-exxcellent
changed the title
rainer-exxcellent
force-pushed
the
feat/447-csaf-2.1_recommended_test_6.2.47
branch
from
8b28797 to
95b448e
Compare
|
Coverage after merging feat/447-csaf-2.1_recommended_test_6.2.47 into main
Coverage Report
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
tschmidtb51
requested changes
Oct 23, 2025
Contributor
tschmidtb51
left a comment
tschmidtb51
left a comment
There was a problem hiding this comment.
Please see my comments
…Url, improve naming and messages
rainer-exxcellent
force-pushed
the
feat/447-csaf-2.1_recommended_test_6.2.47
branch
from
95b448e to
0fb89f9
Compare
|
Coverage after merging feat/447-csaf-2.1_recommended_test_6.2.47 into main
Coverage Report
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
bendo-eXX
approved these changes
Oct 30, 2025
tschmidtb51
approved these changes
Feb 13, 2026
Contributor
tschmidtb51
left a comment
tschmidtb51
left a comment
There was a problem hiding this comment.
In general Ok. Minor comments, no need for a second review from my side unless significant changes occur.
| reference.url !== undefined && | ||
| reference.url.startsWith('https://') && | ||
| reference.url.endsWith( | ||
| trackingId.toLowerCase().replace(/[^+\-a-z0-9]+/g, '_') + '.json' |
Contributor
There was a problem hiding this comment.
Didn't we have a function to compute the filename?
| * - The url starts with https:// | ||
| * - The url ends with the valid filename for the CSAF document | ||
| * A filename must apply the following rules | ||
| * - The value /trackingId is converted into lower case |
Contributor
There was a problem hiding this comment.
Suggested change
| * - The value /trackingId is converted into lower case | |
| * - The value trackingId is converted into lower case |
Comment on lines
+147
to
+148
| 'a qualitative severity rating is used by the issuing party (as no "source" is given' + | ||
| ' or the source property equals to the canonical URL)', |
Contributor
There was a problem hiding this comment.
If you think, the consumers would benefit from the differentiation between the options "source does not exists" and "source is canonical URL" please change the code.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.