Skip to content

Feature/843 revoke instance delegation2 #2198

Feature/843 revoke instance delegation2

Feature/843 revoke instance delegation2 #2198

Triggered via pull request October 29, 2024 13:12
Status Success
Total duration 6m 3s
Artifacts

build-and-analyze.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
Analyze: src/Altinn.AccessManagement.Core/Services/AppsInstanceDelegationService.cs#L175
Define a constant instead of using this literal 'appInstanceDelegationRequest.Resource' 6 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
Analyze: src/Altinn.AccessManagement.Core/Services/PolicyAdministrationPoint.cs#L123
Define a constant instead of using this literal 'Could not acquire blob lease lock on delegation policy at path: {policyPath}' 4 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
Analyze: src/Altinn.AccessManagement.Core/Services/PolicyAdministrationPoint.cs#L36
Update this logger to use its enclosing type. (https://rules.sonarsource.com/csharp/RSPEC-6672)
Analyze: src/Altinn.AccessManagement.Core/Helpers/DelegationHelper.cs#L350
Method has 15 parameters, which is greater than the 7 authorized. (https://rules.sonarsource.com/csharp/RSPEC-107)
Analyze: src/Altinn.AccessManagement.Core/Helpers/DelegationHelper.cs#L609
Refactor this method to reduce its Cognitive Complexity from 27 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
Analyze: src/Altinn.AccessManagement.Core/Helpers/DelegationHelper.cs#L648
Collection-specific "Exists" method should be used instead of the "Any" extension. (https://rules.sonarsource.com/csharp/RSPEC-6605)
Analyze: src/Altinn.AccessManagement.Core/Helpers/DelegationHelper.cs#L959
Collection-specific "Contains" method should be used instead of the "Any" extension. (https://rules.sonarsource.com/csharp/RSPEC-6617)
Analyze: src/Altinn.AccessManagement.Core/Helpers/DelegationHelper.cs#L968
Collection-specific "Contains" method should be used instead of the "Any" extension. (https://rules.sonarsource.com/csharp/RSPEC-6617)
Analyze: src/Altinn.AccessManagement.Core/Services/PolicyAdministrationPoint.cs#L96
Use PascalCase for named placeholders. (https://rules.sonarsource.com/csharp/RSPEC-6678)
Analyze: src/Altinn.AccessManagement.Core/Services/PolicyAdministrationPoint.cs#L123
Use PascalCase for named placeholders. (https://rules.sonarsource.com/csharp/RSPEC-6678)