Updated Tests - WIP #2
FrostyApeOnebuild-test-security.yml
on: push
build-and-test
/
build-and-test
1m 19s
Annotations
8 warnings
|
build-and-test / build-and-test:
src/DfE.CoreLibs.Security/Authorization/Events/RejectSessionCookieWhenAccountNotInCacheEvents.cs#L54
Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)
|
|
build-and-test / build-and-test:
src/DfE.CoreLibs.Security/Authorization/AuthorizationExtensions.cs#L26
Refactor this method to reduce its Cognitive Complexity from 41 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
|
|
build-and-test / build-and-test:
src/DfE.CoreLibs.Security/Authorization/AuthorizationExtensions.cs#L61
The collection-specific "TrueForAll" method should be used instead of the "All" extension (https://rules.sonarsource.com/csharp/RSPEC-6603)
|
|
build-and-test / build-and-test:
src/DfE.CoreLibs.Security/Authorization/AuthorizationExtensions.cs#L69
The collection-specific "TrueForAll" method should be used instead of the "All" extension (https://rules.sonarsource.com/csharp/RSPEC-6603)
|
|
build-and-test / build-and-test:
src/DfE.CoreLibs.Security/Authorization/Events/RejectSessionCookieWhenAccountNotInCacheEvents.cs#L54
Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)
|
|
build-and-test / build-and-test:
src/DfE.CoreLibs.Security/Authorization/AuthorizationExtensions.cs#L26
Refactor this method to reduce its Cognitive Complexity from 41 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
|
|
build-and-test / build-and-test:
src/DfE.CoreLibs.Security/Authorization/AuthorizationExtensions.cs#L61
The collection-specific "TrueForAll" method should be used instead of the "All" extension (https://rules.sonarsource.com/csharp/RSPEC-6603)
|
|
build-and-test / build-and-test:
src/DfE.CoreLibs.Security/Authorization/AuthorizationExtensions.cs#L69
The collection-specific "TrueForAll" method should be used instead of the "All" extension (https://rules.sonarsource.com/csharp/RSPEC-6603)
|