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

ROLIE feed category missing #41

Closed
tschmidtb51 opened this issue Jan 28, 2022 · 2 comments
Closed

ROLIE feed category missing #41

tschmidtb51 opened this issue Jan 28, 2022 · 2 comments
Assignees
@Fadiabb
Labels
bug Something isn't working csaf_provider important

Comments

@tschmidtb51
Copy link
Collaborator

The ROLIE feeds generated by the csaf_provider are missing the category field

"category": [ 
        {
          "scheme": "urn:ietf:params:rolie:category:information-type",
          "term": "csaf"
        }
      ]

which is required as per RFC 8322 Section 6.1.
@tschmidtb51 tschmidtb51 added bug Something isn't working csaf_provider labels Jan 28, 2022
@tschmidtb51
Copy link
Collaborator Author

The existence of that element should also be checked in the Provider Checker.

@bernhardreiter bernhardreiter mentioned this issue Feb 1, 2022
Closed
s-l-teichmann added a commit that referenced this issue May 25, 2022
@s-l-teichmann
Add missing category list in ROLIE feeds. Resolves issue #41
de595f5
@s-l-teichmann s-l-teichmann mentioned this issue May 25, 2022
Merged
Fadiabb added a commit that referenced this issue May 25, 2022
@Fadiabb
Merge pull request #151 from csaf-poc/rolie-category
3a2c4f8 
Add missing category list in ROLIE feeds. Resolves issue #41
@Fadiabb
Copy link
Contributor

Fadiabb commented Jun 9, 2022

Successfully retested.

@Fadiabb Fadiabb closed this as completed Jun 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Fadiabb Fadiabb

Labels
bug Something isn't working csaf_provider important
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@s-l-teichmann @bernhardreiter @Fadiabb @tschmidtb51