Skip to content

Conversation

@ftiercelin
Copy link
Contributor

@ftiercelin ftiercelin commented Dec 4, 2024

Fixes Issue

Description of Change

  • turn into UTC the last modified date retrieved from database
  • change minusDays(-120) into plusDays(120) to enhance code legibility

Have test cases been added to cover the new functionality?

n/a not a new functionality

…red by NvdCveClientBuilder#withLastModifiedFilter

replace minusDays(-120) with plusDays(120) when calculating end date for increased legibility
@boring-cyborg boring-cyborg bot added the core changes to core label Dec 4, 2024
@ftiercelin ftiercelin changed the title Tranform into UTC the last modified date from database fix: Tranform into UTC the last modified date from database Dec 4, 2024
@jeremylong jeremylong added this to the 12.0.0 milestone Dec 4, 2024
@jeremylong
Copy link
Collaborator

Thank you for figuring this out! I really appreciate the PR!

jeremylong
jeremylong previously approved these changes Dec 4, 2024
Copy link
Collaborator

@jeremylong jeremylong left a comment

Choose a reason for hiding this comment

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

LGTM

@jeremylong jeremylong merged commit 97c3cee into dependency-check:main Dec 4, 2024
5 of 6 checks passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 10, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

core changes to core

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants