Resolved Issues
Not all vulnerabilities and issues you discover with Sandworm are addressable by code alone. You may want to mark an issue as resolved when:
A fix has already been started
There is no bandwidth to work on a fix right now
The risk is tolerable for the project
The issue is inaccurate or incorrect
The vulnerable code is not actually used
When running an audit, Sandworm looks for a resolved-issues.json file in the root directory of the app being audited. The resolved issues file should contain a json array of objects, each object describing a resolved issue.
A resolved issue is described by the following attributes:
id- the GitHub advisory id or the Sandworm issue idpaths- the current dependency tree paths to affected package versionsnotes- resolution notes
Here's an example resolved issues file:
Commit the resolved-issues.json file to your repository for full historical visibility into who resolved what & when.
For longer resolution notes, we suggest creating a markdown file in a dedicated directory in your repository, and pointing to that file in the resolution note.
Using paths
Paths help segment the packages affected by an issue into multiple categories. Paths are used to provide granular resolutions, and also help identify potentially new uses of vulnerable package versions.
Let's say your app depends on packageA and packageB, who both depend on packageC. A vulnerability is identified in packageC. After analysis, you conclude that:
packageAdoesn't use thepackageCfunctions exposing the vulnerability;packageBdoes use a vulnerable function inpackageC, but properly escapes user input so the vulnerability is avoided.
You'll want to create two resolutions: one for path packageA>packageC and one for path packageB>packageC:
Now let's say you install packageD, and that too turns out to depend on a vulnerable version of packageC. Sandworm's audit would label this as a new issue, since there's no resolved path that matches the new packageD>packageC. To mark this as fixed, either:
Add the
packageD>packageCpath to one of the existing resolutions, if the same notes apply to the new situation;Or, create a new resolution, specific for the
packageD>packageCpath.
Using the command-line tool
Instead of manually managing the resolved-issues.json file, you can use sandworm resolve [issueId]. The command-line tool makes a number of validations, and walks you through creating a new resolution or attaching a new path to an existing resolution.
The output of sandworm audit includes issue ids that you can resolve:
After running sandworm resolve [issueID] on one of the detected issues, you specify one or more paths to address:
If any existing resolutions share the same issue id, resolve prompts you to either create a new resolution, or attach the new paths to an existing one:
Using version wildcards in issue ids
This can, however, become tedious when working with internal or otherwise fully trusted packages. For such scenarios, when resolving issues via the JSON file or the CLI, you can use a wildcard for the version in the Sandworm issue id, like:
The wildcard in the issue id matches any current or future instance of the Sandworm error with code 102 (license is not OSI approved) for the package named azure-common.
If you don't fully trust a dependency package, don't use wildcards for its resolved issue ids. Doing so is dangerous, and can leave the door open for certain types of security issues.
Imagine, for example, that you've resolved the SWRM-102-package-name-0.9.10 issue, triggered because package-name@0.9.10 uses Apache 2.0 as a license, which isn't an OSI approved string. But then you update, and get a new SWRM-102-package-name-0.9.11 issue, that has the same underlying cause as before, but needs to be marked as resolved again.
So you use SWRM-102-package-name-* to future-proof your audits against this issue. However, along comes package-name@0.10.0 that changes the license string from Apache 2.0 to AGPL, which is still not an OSI approved string. The core of the issue has changed, but the issue id is the same and matches your resolution wildcard. A significant change that requires attention could slip by unnoticed this way.
Last updated
Was this helpful?
