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

Resolves #1180 Schema version auto populated when omitted in secretariat endpoints #1191

Merged
merged 2 commits into from
Mar 5, 2024

Conversation

jdaigneau5
Copy link
Collaborator

Closes Issue #1180

Summary

Per secretariat request, the dataVersion field is no longer required for secretariat only CVE endpoints. The field will. be auto populated when omitted.

Important Changes

middleware.js

  • removed check for specific dataVersion number in validateCveJsonSchema.
  • adds version number when field is omitted.

Testing

Steps to manually test updated functionality, if possible

  • 1) Example test step.

…, auto populated if missing. Removed no longer relevent test
@jdaigneau5 jdaigneau5 requested a review from david-rocca March 5, 2024 15:18
@david-rocca
Copy link
Collaborator

Local tests confirm that when no dataVersion is listed 5.1 is added. But it respects dataVersion when set. LGTM

@david-rocca david-rocca merged commit bf564d2 into dev Mar 5, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants