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 #1075 - Addition of cna_modified boolean #1086

Merged
merged 5 commits into from
Jun 22, 2023
Merged

Resolves #1075 - Addition of cna_modified boolean #1086

merged 5 commits into from
Jun 22, 2023

Conversation

david-rocca
Copy link
Collaborator

@david-rocca david-rocca commented Jun 20, 2023

Closes Issue #1075

Summary

This PR adds a boolean to the get cve request located at /api/cve which, when set to true, will only return results who's cna containers have been modified within the selected timeframe.

Important Changes

  • src/controller/cve.controller/cve_controller.js
  • src/controller/cve.controller/cve.middleware.js
  • src/controller/cve.controller/index.js.js

Testing

Steps to manually test updated functionality, if possible

  • 1) run the following command npm run test:integration

@david-rocca david-rocca requested a review from jdaigneau5 June 20, 2023 18:02
@david-rocca david-rocca self-assigned this Jun 20, 2023
@jdaigneau5 jdaigneau5 merged commit 596c0ce into dev Jun 22, 2023
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