Update nuget non-major dependencies #803
pep-test-and-analyze.yml
on: pull_request
Static code analysis
3m 39s
Annotations
15 warnings
Static code analysis:
src/Altinn.Common.PEP/Altinn.Common.PEP/Clients/AuthorizationApiClient.cs#L48
The annotation for nullable reference types should only be used in code within a '#nullable' annotations context.
|
Static code analysis:
src/Altinn.Common.PEP/UnitTests/AppAccessHandlerTest.cs#L306
Use IHeaderDictionary.Append or the indexer to append or set headers. IDictionary.Add will throw an ArgumentException when attempting to add a duplicate key. (https://aka.ms/aspnet/analyzers)
|
Static code analysis:
src/Altinn.Common.PEP/UnitTests/ResourceAccessHandlerTest.cs#L211
Use IHeaderDictionary.Append or the indexer to append or set headers. IDictionary.Add will throw an ArgumentException when attempting to add a duplicate key. (https://aka.ms/aspnet/analyzers)
|
Static code analysis:
src/Altinn.Common.PEP/UnitTests/ResourceAccessHandlerTest.cs#L216
Use IHeaderDictionary.Append or the indexer to append or set headers. IDictionary.Add will throw an ArgumentException when attempting to add a duplicate key. (https://aka.ms/aspnet/analyzers)
|
Static code analysis:
src/Altinn.Common.PEP/UnitTests/ResourceAccessHandlerTest.cs#L221
Use IHeaderDictionary.Append or the indexer to append or set headers. IDictionary.Add will throw an ArgumentException when attempting to add a duplicate key. (https://aka.ms/aspnet/analyzers)
|
Static code analysis:
src/Altinn.Common.PEP/Altinn.Common.PEP/Clients/AuthorizationApiClient.cs#L48
The annotation for nullable reference types should only be used in code within a '#nullable' annotations context.
|
Static code analysis:
src/Altinn.Common.PEP/Altinn.Common.PEP/Authorization/AppAccessHandler.cs#L24
Remove this unread private field '_logger' or refactor the code to use its value. (https://rules.sonarsource.com/csharp/RSPEC-4487)
|
Static code analysis:
src/Altinn.Common.PEP/Altinn.Common.PEP/Implementation/PDPAppSI.cs#L17
Rename class 'PDPAppSI' to match pascal case naming rules, consider using 'PdpAppSi'. (https://rules.sonarsource.com/csharp/RSPEC-101)
|
Static code analysis:
src/Altinn.Common.PEP/Altinn.Common.PEP/Authorization/ResourceAccessHandler.cs#L24
Remove this unread private field '_logger' or refactor the code to use its value. (https://rules.sonarsource.com/csharp/RSPEC-4487)
|
Static code analysis:
src/Altinn.Common.PEP/Altinn.Common.PEP/Interfaces/IPDP.cs#L11
Rename interface 'IPDP' to match pascal case naming rules, consider using 'IPdp'. (https://rules.sonarsource.com/csharp/RSPEC-101)
|
Static code analysis:
src/Altinn.Common.PEP/Altinn.Common.PEP/Program.cs#L9
Add a 'protected' constructor or the 'static' keyword to the class declaration. (https://rules.sonarsource.com/csharp/RSPEC-1118)
|
Static code analysis:
src/Altinn.Common.PEP/Altinn.Common.PEP/Utils/IDFormatDeterminator.cs#L9
Add a 'protected' constructor or the 'static' keyword to the class declaration. (https://rules.sonarsource.com/csharp/RSPEC-1118)
|
Static code analysis:
src/Altinn.Common.PEP/Altinn.Common.PEP/Models/IDFormat.cs#L10
'int' should not be explicitly used as the underlying type. (https://rules.sonarsource.com/csharp/RSPEC-1939)
|
Static code analysis:
src/Altinn.Common.PEP/Altinn.Common.PEP/Clients/AuthorizationApiClient.cs#L27
Remove the field '_httpContextAccessor' and declare it as a local variable in the relevant methods. (https://rules.sonarsource.com/csharp/RSPEC-1450)
|
Static code analysis:
src/Altinn.Common.PEP/Altinn.Common.PEP/Authorization/AppAccessHandler.cs#L59
The parameter name 'response' is not declared in the argument list. (https://rules.sonarsource.com/csharp/RSPEC-3928)
|