Skip to content

fix: Add null checking when parsing the license json in AbstractNpmAnalyzer.#7784

Merged
jeremylong merged 1 commit intodependency-check:mainfrom
floverfelt:bugfix/7738
Jul 6, 2025
Merged

fix: Add null checking when parsing the license json in AbstractNpmAnalyzer.#7784
jeremylong merged 1 commit intodependency-check:mainfrom
floverfelt:bugfix/7738

Conversation

@floverfelt
Copy link
Copy Markdown
Contributor

Description of Change

Adds null checking when the license block is defined as a json object but the type is still missing

Related issues

fixes: #7783

Have test cases been added to cover the new functionality?

no

@boring-cyborg boring-cyborg bot added the core changes to core label Jul 3, 2025
@floverfelt floverfelt changed the title Add null checking when parsing the license json in AbstractNpmAnalyzer. fix: Add null checking when parsing the license json in AbstractNpmAnalyzer. Jul 3, 2025
Copy link
Copy Markdown
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 added this to the 12.1.4 milestone Jul 6, 2025
@jeremylong jeremylong merged commit ebe6f0d into dependency-check:main Jul 6, 2025
5 of 6 checks passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 6, 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.

NullPointerException on NodePackageAnalyzer license handling

2 participants