Extend authorized parties with instance delegations #2189
andreasisnesbuild-and-analyze.yml
on: pull_request
Build and Test
1m 11s
Analyze
4m 39s
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#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.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#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#L105
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.Persistence/DelegationMetadataRepository.cs#L93
Define a constant instead of using this literal '_offeredByPartyId' 6 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
Analyze:
src/Altinn.AccessManagement.Persistence/DelegationMetadataRepository.cs#L94
Define a constant instead of using this literal '_coveredByUserId' 6 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
Analyze:
src/Altinn.AccessManagement.Persistence/DelegationMetadataRepository.cs#L95
Define a constant instead of using this literal '_coveredByPartyId' 5 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|