Skip to content

fix: Add eternal configuration for NODEAUDIT cache results. - #8758

Open
ejouvin wants to merge 1 commit into
dependency-check:mainfrom
ejouvin:issue-8757-noideaudit-cache-refresh
Open

fix: Add eternal configuration for NODEAUDIT cache results.#8758
ejouvin wants to merge 1 commit into
dependency-check:mainfrom
ejouvin:issue-8757-noideaudit-cache-refresh

Conversation

@ejouvin

@ejouvin ejouvin commented Aug 20, 2026

Copy link
Copy Markdown

Description of Change

Update configuration file to set isEternal flag value to false, only on NODEAUDIT cache.
Without this, the cache is never refresh.

In documentation for configuration nodeAuditAnalyzerUseCache
Sets whether the Node Audit Analyzer will cache results. Cached results expire after 24 hours.

But it is never the case, results in the cache never expire as they are flagged as eternal.

Related issues

fixes #8757

Have test cases been added to cover the new functionality?

yes

See issue dependency-check#8757. No configuration set on CENTRAL and POM as it was
mentionned it is expected behaviour.
@boring-cyborg boring-cyborg Bot added the core changes to core label Aug 20, 2026
@ejouvin ejouvin changed the title Add eternal configuration for NODEAUDIT cache results. fix:Add eternal configuration for NODEAUDIT cache results. Aug 20, 2026
@ejouvin ejouvin changed the title fix:Add eternal configuration for NODEAUDIT cache results. fix: Add eternal configuration for NODEAUDIT cache results. Aug 20, 2026

@chadlwilson chadlwilson left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This may be valid, but as noted at #8757 (comment) NodeAuditAnalyzer is not functional right now due to #8422, so there's no way to validate whether this is actually working or whether it's the correct approach or even the fix to the cited problem.

And still no reproducer supplied with logs or anything else to make it easier to validate when NodeAudit is reinstated?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core changes to core

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CENTRAL - NODEAUDIT - POM Cached results never expire

2 participants