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
Rolie examples #315
Merged
Merged
Rolie examples #315
Changes from 8 commits
Commits
Show all changes
15 commits
Select commit
Hold shift + click to select a range
ca58bb1
ROLIE examples
tschmidtb51 f2743ae
ROLIE examples
tschmidtb51 25444c3
ROLIE examples
tschmidtb51 68e3371
ROLIE examples
tschmidtb51 6fa1eeb
Merge remote-tracking branch 'upstream/master' into rolie-examples
tschmidtb51 930246d
Namespace upgrade
tschmidtb51 7275e29
ROLIE examples
tschmidtb51 fa67b69
ROLIE examples
tschmidtb51 665fbc5
Merge remote-tracking branch 'upstream/master' into rolie-examples
tschmidtb51 10eae59
ROLIE examples
tschmidtb51 6346760
Added final newline
sthagen 84593f7
ROLIE examples
tschmidtb51 f3103c3
Merge branch 'rolie-examples' of github:tschmidtb51/csaf into rolie-e…
tschmidtb51 b1f1159
ROLIE examples
tschmidtb51 ca2d9a8
ROLIE examples
tschmidtb51 File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
{ | ||
"categories": [ | ||
{ | ||
"term": "Example Company Product A" | ||
}, | ||
{ | ||
"term": "Example Company Product B" | ||
} | ||
] | ||
} | ||
This file contains 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
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
{ | ||
"service": { | ||
"workspace": [ | ||
{ | ||
"title": "Public CSAF feed", | ||
"collection": [ | ||
{ | ||
"title": "Example CSAF feed (TLP:WHITE)", | ||
"href": "https://psirt.domain.tld/advisories/csaf/feed-tlp-white.json", | ||
"categories": [ | ||
{ | ||
"scheme": "urn:ietf:params:rolie:category:information-type", | ||
"term": "csaf" | ||
} | ||
] | ||
} | ||
] | ||
} | ||
] | ||
} | ||
} |
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,54 @@ | ||
{ | ||
"service": { | ||
"workspace": [ | ||
{ | ||
"title": "Public CSAF feed", | ||
"collection": [ | ||
{ | ||
"title": "Example CSAF feed (TLP:WHITE)", | ||
"href": "https://psirt.domain.tld/advisories/csaf/feed-tlp-white.json", | ||
"categories": [ | ||
{ | ||
"scheme": "urn:ietf:params:rolie:category:information-type", | ||
"term": "csaf" | ||
} | ||
] | ||
} | ||
] | ||
}, | ||
{ | ||
"title": "Private CSAF feeds", | ||
"collection": [ | ||
{ | ||
"title": "Example CSAF feed (TLP:AMBER)", | ||
"href": "https://psirt.domain.tld/advisories/tlp-amber/csaf/feed-tlp-amber.json", | ||
"categories": [ | ||
{ | ||
"scheme": "urn:ietf:params:rolie:category:information-type", | ||
"term": "csaf" | ||
}, | ||
{ | ||
"scheme": "urn:ietf:params:rolie:category:information-type", | ||
"term": "tlp-amber" | ||
} | ||
] | ||
}, | ||
{ | ||
"title": "Example CSAF feed (TLP:RED)", | ||
"href": "https://psirt.domain.tld/advisories/tlp-red/csaf/feed-tlp-red.json", | ||
"categories": [ | ||
{ | ||
"scheme": "urn:ietf:params:rolie:category:information-type", | ||
"term": "csaf" | ||
}, | ||
{ | ||
"scheme": "urn:ietf:params:rolie:category:information-type", | ||
"term": "tlp-red" | ||
} | ||
] | ||
} | ||
] | ||
} | ||
] | ||
} | ||
} |
This file contains 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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this also need to specify a scheme to which these categories relate. Though, I'm not sure what the value of that field should be, so:
Perhaps it can just be an arbitrary URN that gets then used in the individual documents. I think only terms specific to the
urn:ietf:params:rolie:category:information-type
category don't need to be explicitly noted in the category document.For now, I'd recommend using a self-reference to the category document itself as the scheme.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree that there SHOULD be a category scheme. However, while reading RFC 5023 Section 7 I did not see that it is mandatory
Therefore, I skipped it. But I might have overlooked something. Can you please double-check that?
If it is necessary, I second your approach to app the reference to itself in the document.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nevertheless, you are right about mentioning of
category
.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I fixed that in 10eae59.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right, it is optional. Let's leave it out. Thanks for the
category
fix!There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No worries. I also fixed that in the service document examples and fixed a filename error.