Extend authorized parties with instance delegations #2187
andreasisnesbuild-and-analyze.yml
on: pull_request
Build and Test
1m 12s
Analyze
5m 23s
Annotations
10 warnings
|
Analyze:
src/Altinn.AccessManagement.Core/Services/AppsInstanceDelegationService.cs#L188
Define a constant instead of using this literal 'appInstanceDelegationRequest.Resource' 6 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
Analyze:
src/Altinn.AccessManagement.Core/Repositories/Interfaces/IDelegationMetadataRepository.cs#L67
Method has 8 parameters, which is greater than the 7 authorized. (https://rules.sonarsource.com/csharp/RSPEC-107)
|
|
Analyze:
src/Altinn.AccessManagement.Core/Services/AppsInstanceDelegationService.cs#L220
'delegableRights' is null on at least one execution path. (https://rules.sonarsource.com/csharp/RSPEC-2259)
|
|
Analyze:
src/Altinn.AccessManagement.Core/Services/AuthorizedPartiesService.cs#L331
Correct one of the identical expressions on both sides of operator '=='. (https://rules.sonarsource.com/csharp/RSPEC-1764)
|
|
Analyze:
src/Altinn.AccessManagement.Core/Services/AuthorizedPartiesService.cs#L331
Correct one of the identical expressions on both sides of operator '=='. (https://rules.sonarsource.com/csharp/RSPEC-1764)
|
|
Analyze:
src/Altinn.AccessManagement.Core/Services/AuthorizedPartiesService.cs#L192
Refactor this method to reduce its Cognitive Complexity from 73 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
|
|
Analyze:
src/Altinn.AccessManagement.Core/Services/AuthorizedPartiesService.cs#L331
Collection-specific "Exists" method should be used instead of the "Any" extension. (https://rules.sonarsource.com/csharp/RSPEC-6605)
|
|
Analyze:
src/Altinn.AccessManagement.Integration/Clients/AltinnRolesClient.cs#L54
Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)
|
|
Analyze:
src/Altinn.AccessManagement.Integration/Clients/AltinnRolesClient.cs#L79
Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)
|
|
Analyze:
src/Altinn.AccessManagement.Integration/Clients/AltinnRolesClient.cs#L105
Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)
|