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

Rolie examples #315

Merged
merged 15 commits into from Jul 28, 2021
Merged

Rolie examples #315

merged 15 commits into from Jul 28, 2021

Conversation

tschmidtb51
Copy link
Contributor

- addresses part of oasis-tcs#313
- add example for ROLIE service document
- addresses part of oasis-tcs#313
- update example for ROLIE feed from "security advisory" to "csaf" to avoid conflicts with profiles
- addresses part of oasis-tcs#313
- fix mistake in ROLIE service document example
- addresses part of oasis-tcs#313
- add extended example for ROLIE service document
- addresses part of oasis-tcs#312 and oasis-tcs#313
- update namespace in ROLIE feed example
- addresses part of oasis-tcs#313
- add example for ROLIE category document
- addresses part of oasis-tcs#313
- according to https://datatracker.ietf.org/doc/html/rfc8322#section-6.1.1 as feed can have more than one category
@tschmidtb51 tschmidtb51 added documentation csaf 2.0 csaf 2.0 work editorial mostly nits and consistency labels Jul 23, 2021
@tschmidtb51 tschmidtb51 self-assigned this Jul 23, 2021
@tschmidtb51 tschmidtb51 marked this pull request as draft July 23, 2021 13:27
@tschmidtb51
Copy link
Contributor Author

@mprpic Please review this PR with ROLIE examples.

@tschmidtb51
Copy link
Contributor Author

tschmidtb51 commented Jul 26, 2021

@santosomar, @sthagen We did not receive additional feedback to this. Therefore, I suggest to merge it (to provide a full CSD for review) and point it out in the TC meeting.

@tschmidtb51 tschmidtb51 marked this pull request as ready for review July 26, 2021 08:33
Copy link
Contributor

@mprpic mprpic left a comment

Choose a reason for hiding this comment

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

Looks great!

"term": "Example Company Product B"
}
]
}
Copy link
Contributor

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:

{
   "categories": {
      "category": [
         {
            "term": "Example Company Product A"
         },
         {
            "term": "Example Company Product B"
         }
      ],
      "scheme": "<reference to itself?>"
   }
}

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.

Copy link
Contributor Author

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.

I agree that there SHOULD be a category scheme. However, while reading RFC 5023 Section 7 I did not see that it is mandatory

    appInlineCategories =
       element app:categories {
           attribute fixed { "yes" | "no" }?,
           attribute scheme { atomURI }?,
           (atomCategory*,
           undefinedContent)
       }

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.

Copy link
Contributor Author

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.

Copy link
Contributor Author

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.

Copy link
Contributor

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!

Copy link
Contributor Author

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.

tschmidtb51 added a commit to tschmidtb51/csaf that referenced this pull request Jul 27, 2021
- addresses review comment of oasis-tcs#315
- add missing "category" keyword
- fix typo
- addresses review comment of oasis-tcs#315
- add missing "category" keyword
- fix typo
- addresses review comment of oasis-tcs#315
- fix missing "category" keyword in service document examples as [Section 8.3.6](https://datatracker.ietf.org/doc/html/rfc5023#section-8.3.6) links to category description
- addresses review comment of oasis-tcs#313
- fix filename mistake (be compliant to Section 5.1)
- addresses parts of oasis-tcs#313
- fix filename also in other places
@tschmidtb51 tschmidtb51 requested a review from sthagen July 28, 2021 00:35
@santosomar santosomar merged commit ab2b8d5 into oasis-tcs:master Jul 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
csaf 2.0 csaf 2.0 work documentation editorial mostly nits and consistency
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add ROLIE examples
4 participants