DataDog / dd-trace-dotnet

.NET Client Library for Datadog APM
https://docs.datadoghq.com/tracing/
Apache License 2.0
456 stars 142 forks source link

[CONTSEC-1501] Comment the action that uploads SARIF to Datadog #5976

Closed ganeshkumarsv closed 2 months ago

ganeshkumarsv commented 2 months ago

Summary of changes

Commenting out the action that uploads SARIF to Datadog until the issue (Datadog does not like the SARIF report that is being uploaded for csharp.) gets resolved.

Reason for change

The action advanced-security/filter-sarif@v1 is altering the SARIF fields when filtering the results which is not getting supported by Datadog when uploading.

Implementation details

Test coverage

Other details

andrewlock commented 2 months ago

Done in https://github.com/DataDog/dd-trace-dotnet/pull/5977, thanks!