On Pull Request

[APIPUB-76] - Adds Sonar Analyzer #118

Sign in to view logs

Triggered via pull request September 27, 2024 17:15

DavidJGapCR

synchronize #80

APIPUB-76

Status Failure

Total duration 49s

Artifacts 1

on-pullrequest.yml

on: pull_request

Scan Actions, scan all files for BIDI Trojan Attacks / action-allowedlist

Scan Actions, scan all files for BIDI Trojan Attacks / bidi-scanner

Annotations

10 errors and 1 warning

Build and Test: src/EdFi.Tools.ApiPublisher.Connections.Api/Configuration/ApiConnectionDetails.cs#L17

Do not forget to remove this deprecated code someday. (https://rules.sonarsource.com/csharp/RSPEC-1133)

Build and Test: src/EdFi.Tools.ApiPublisher.Connections.Api/Configuration/ApiConnectionDetails.cs#L47

Do not forget to remove this deprecated code someday. (https://rules.sonarsource.com/csharp/RSPEC-1133)

Build and Test: src/EdFi.Tools.ApiPublisher.Connections.Api/Configuration/ApiConnectionDetails.cs#L32

Do not forget to remove this deprecated code someday. (https://rules.sonarsource.com/csharp/RSPEC-1133)

Build and Test: src/EdFi.Tools.ApiPublisher.Connections.Api/Processing/Source/MessageProducers/EdFiApiChangeVersionReversePagingStreamResourcePageMessageProducer.cs#L130

Change this condition so that it does not always evaluate to 'True'. Some code paths are unreachable. (https://rules.sonarsource.com/csharp/RSPEC-2583)

Build and Test: src/EdFi.Tools.ApiPublisher.Connections.Api/Processing/Target/Blocks/DeleteResourceProcessingBlocksFactory.cs#L188

Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)

Build and Test: src/EdFi.Tools.ApiPublisher.Connections.Api/Processing/Target/Blocks/DeleteResourceProcessingBlocksFactory.cs#L193

Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)

Build and Test: src/EdFi.Tools.ApiPublisher.Connections.Api/Processing/Target/Blocks/PostResourceProcessingBlocksFactory.cs#L415

Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)

Build and Test: src/EdFi.Tools.ApiPublisher.Connections.Api/Processing/Target/Blocks/DeleteResourceProcessingBlocksFactory.cs#L315

Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)

Build and Test: src/EdFi.Tools.ApiPublisher.Connections.Api/Processing/Target/Blocks/DeleteResourceProcessingBlocksFactory.cs#L320

Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)

Build and Test: src/EdFi.Tools.ApiPublisher.Connections.Api/Processing/Target/Blocks/ChangeResourceKeyProcessingBlocksFactory.cs#L238

Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)

Build and Test

The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-dotnet@607fce577a46308457984d59e4954e075820f10a. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/

Artifacts

Produced during runtime

Name	Size
Event File	3.22 KB