chore(deps): bump actions/checkout from 3 to 4 (#315) #384
This run and associated checks have been archived and are scheduled for deletion.
Learn more about checks retention
build-and-analyze.yml
on: push
Build and Test
47s
Analyze
5m 2s
Annotations
20 warnings
Build and Test:
src/Authentication/Model/AuthenticationEvent.cs#L24
The annotation for nullable reference types should only be used in code within a '#nullable' annotations context.
|
Build and Test:
src/Authentication/Model/AuthenticationEvent.cs#L34
The annotation for nullable reference types should only be used in code within a '#nullable' annotations context.
|
Build and Test:
src/Authentication/Controllers/AuthenticationController.cs#L240
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
|
Build and Test:
src/Authentication/Controllers/AuthenticationController.cs#L1004
'SecurityTokenUnableToValidateException' is obsolete: 'This expception is no longer being thrown by Microsoft.IdentityModel and will be removed in the next major version see: https://aka.ms/SecurityTokenUnableToValidateException'
|
Build and Test:
src/Authentication/Program.cs#L315
Suggest using top level route registrations instead of UseEndpoints (https://aka.ms/aspnet/analyzers)
|
Build and Test:
test/Altinn.Platform.Authentication.Tests/Services/EventLogServiceTest.cs#L21
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
|
Build and Test:
test/Altinn.Platform.Authentication.Tests/Services/EventLogServiceTest.cs#L39
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
|
Build and Test:
test/Altinn.Platform.Authentication.Tests/Fakes/SigningKeyResolverStub.cs#L13
XML comment has cref attribute 'SigningKeysResolver' that could not be resolved
|
|
Build and Test:
src/Authentication/Model/AuthenticationEvent.cs#L24
The annotation for nullable reference types should only be used in code within a '#nullable' annotations context.
|
Analyze:
src/Authentication/Model/AuthenticationEvent.cs#L24
The annotation for nullable reference types should only be used in code within a '#nullable' annotations context.
|
Analyze:
src/Authentication/Model/AuthenticationEvent.cs#L34
The annotation for nullable reference types should only be used in code within a '#nullable' annotations context.
|
Analyze:
src/Authentication/Controllers/AuthenticationController.cs#L240
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
|
Analyze:
src/Authentication/Controllers/AuthenticationController.cs#L1004
'SecurityTokenUnableToValidateException' is obsolete: 'This expception is no longer being thrown by Microsoft.IdentityModel and will be removed in the next major version see: https://aka.ms/SecurityTokenUnableToValidateException'
|
Analyze:
src/Authentication/appsettings.json#L28
"pwd" detected here, make sure this is not a hard-coded credential. (https://rules.sonarsource.com/csharp/RSPEC-2068)
|
Analyze:
src/Authentication/Program.cs#L1
Refactor this top-level file to reduce its Cognitive Complexity from 27 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
|
Analyze:
src/Authentication/Extensions/ClaimsPrincipalExtension.cs#L17
Loops should be simplified with "LINQ" expressions (https://rules.sonarsource.com/csharp/RSPEC-3267)
|
Analyze:
src/Authentication/Controllers/AuthenticationController.cs#L101
Constructor has 14 parameters, which is greater than the 7 authorized. (https://rules.sonarsource.com/csharp/RSPEC-107)
|
Analyze:
src/Authentication/Controllers/AuthenticationController.cs#L145
Refactor this method to reduce its Cognitive Complexity from 30 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
|
Analyze:
src/Authentication/Controllers/AuthenticationController.cs#L578
Remove this useless assignment to local variable 'authLevelValue'. (https://rules.sonarsource.com/csharp/RSPEC-1854)
|