hohn/codeql
1
0
Fork 0
mirror of https://github.com/github/codeql.git synced 2026-04-26 17:25:19 +02:00
Code Issues Packages Projects Releases Wiki Activity

C#: Fix logging in assets file parsing

Browse Source
This commit is contained in:
Tamas Vajk
2024-08-12 14:06:28 +02:00
parent c514d36d90
commit 089999dd3c
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
csharp/extractor/Semmle.Extraction.CSharp.DependencyFetching/Assets.cs
View File

@@ -189,13 +189,13 @@ namespace Semmle.Extraction.CSharp.DependencyFetching
/// (together with used package information) required for compilation.
/// </summary>
/// <returns>True if parsing succeeds, otherwise false.</returns>
public bool TryParse(string json)
public bool TryParse(string json, string? jsonPath = null)
{
try
{
var obj = JObject.Parse(json);
AddPackageDependencies(obj, json);
AddFrameworkDependencies(obj, json);
AddPackageDependencies(obj, jsonPath ?? json);
AddFrameworkDependencies(obj, jsonPath ?? json);
return true;
}
catch (Exception e)
@@ -228,7 +228,7 @@ namespace Semmle.Extraction.CSharp.DependencyFetching
{
if (TryReadAllText(asset, logger, out var json))
{
TryParse(json);
TryParse(json, asset);
}
}