lirantal / lockfile-lint

Lint an npm or yarn lockfile to analyze and detect security issues
Apache License 2.0
781 stars 35 forks source link

fix(validators): only validating when URL is present #59

Closed lirantal closed 4 years ago

lirantal commented 4 years ago

Description

a regression introduced via PR #53 due to flat package list now being all-inclusive of packages, some of which have no protocol or URL to resolve to (the nested deps) and so the validators failed.

Types of changes

Related Issue

Fixes #58

How Has This Been Tested?

Missing from this PR is a set of unit tests to cover the error and will need to follow-up on.