search

snyk / snyk-nuget-plugin

Basic Snyk CLI plugin for .NET support.
Other
7 stars 14 forks source link

fix: stringify-range-error #131

Closed dagrest closed 1 year ago

dagrest commented 1 year ago

What does this PR do?

Fixes RangeError - 'Invalid string length' while using JSON.stringify() for huge dependency tree.

Where should the reviewer start?

.../lib/nuget-parser/dotnet-core-parser.ts

How should this be manually tested?

It can be tested by test 'parse-large-dependency-tree', located in .../test/generate-huge-project-assets-json.spec.ts

Any background context you want to provide?

To resolve the problem - JSON.stringify() was replaced by lodash.cloneDeep()

What are the relevant tickets?

CLAassistant commented 1 year ago

CLA assistant check
All committers have signed the CLA.

snyksec commented 1 year ago

:tada: This PR is included in version 1.24.1 :tada:

The release is available on:

Your semantic-release bot :package::rocket: