Skip to content

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

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

jeremylong merged 1 commit into
dependency-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

@jeremylong jeremylong 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.

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
@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