Skip to content
Code test and analysis

SonarClound creates issues, so so we should get local warnig #2662

Sign in to view logs

SonarClound creates issues, so so we should get local warnig

SonarClound creates issues, so so we should get local warnig #2662

Triggered via pull request November 7, 2024 09:57
@ivarneivarne
synchronize #893
chore/activate-sonar-warnings
Status Success
Total duration 7m 52s
Artifacts –

test-and-analyze.yml

on: pull_request
Static code analysis
7m 39s
Static code analysis
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
Static code analysis: src/Altinn.App.Api/Controllers/InstancesController.cs#L1150
Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)
Static code analysis: src/Altinn.App.Api/Controllers/ActionsController.cs#L78
ModelState.IsValid should be checked in controller actions. (https://rules.sonarsource.com/csharp/RSPEC-6967)
Static code analysis: src/Altinn.App.Api/Controllers/AuthorizationController.cs#L108
ModelState.IsValid should be checked in controller actions. (https://rules.sonarsource.com/csharp/RSPEC-6967)
Static code analysis: src/Altinn.App.Api/Controllers/AuthorizationController.cs#L45
ModelState.IsValid should be checked in controller actions. (https://rules.sonarsource.com/csharp/RSPEC-6967)
Static code analysis: src/Altinn.App.Api/Controllers/InstancesController.cs#L191
Refactor this method to reduce its Cognitive Complexity from 31 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
Static code analysis: src/Altinn.App.Api/Controllers/InstancesController.cs#L286
Merge this if statement with the enclosing one. (https://rules.sonarsource.com/csharp/RSPEC-1066)
Static code analysis: src/Altinn.App.Api/Controllers/InstancesController.cs#L80
Constructor has 18 parameters, which is greater than the 7 authorized. (https://rules.sonarsource.com/csharp/RSPEC-107)
Static code analysis: src/Altinn.App.Api/Controllers/InstancesController.cs#L279
'System.Exception' should not be thrown by user code. (https://rules.sonarsource.com/csharp/RSPEC-112)
Static code analysis: src/Altinn.App.Api/Controllers/InstancesController.cs#L253
Change this condition so that it does not always evaluate to 'False'. Some code paths are unreachable. (https://rules.sonarsource.com/csharp/RSPEC-2583)
Static code analysis: src/Altinn.App.Api/Controllers/DataTagsController.cs#L50
ModelState.IsValid should be checked in controller actions. (https://rules.sonarsource.com/csharp/RSPEC-6967)