Skip to content

Upgrade to .net 8

Upgrade to .net 8 #8

Triggered via pull request April 22, 2024 08:43
Status Success
Total duration 2m 56s
Artifacts

pull-request.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

11 warnings
ui
Could not determine node version from /home/runner/work/IdOps/IdOps/src/UI/package.json. Falling back
tests: src/IdentityServer/src/Hashing/Pbkdf2Algorithm.cs#L70
'Rfc2898DeriveBytes.Rfc2898DeriveBytes(string, byte[])' is obsolete: 'The default hash algorithm and iteration counts in Rfc2898DeriveBytes constructors are outdated and insecure. Use a constructor that accepts the hash algorithm and the number of iterations.' (https://aka.ms/dotnet-warnings/SYSLIB0041)
tests: src/IdentityServer/src/Hashing/Pbkdf2Algorithm.cs#L6
Add a 'protected' constructor or the 'static' keyword to the class declaration. (https://rules.sonarsource.com/csharp/RSPEC-1118)
tests: src/IdentityServer/src/Hashing/PasswordProvider.cs#L6
Fix this implementation of 'IDisposable' to conform to the dispose pattern. (https://rules.sonarsource.com/csharp/RSPEC-3881)
tests: src/Server/src/Abstractions/ISecretValueEncryptor.cs#L20
Non-nullable property 'KeyProvider' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
tests: src/Server/src/Abstractions/ISecretValueEncryptor.cs#L22
Non-nullable property 'Key' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
tests: src/Server/src/Abstractions/ISecretValueEncryptor.cs#L24
Non-nullable property 'Algorithm' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
tests: src/Server/src/Abstractions/Messages/ResourcePublishedSuccessMessage.cs#L31
Non-nullable property 'ResourceType' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
tests: src/Server/src/Abstractions/Messages/ResourcePublishedSuccessMessage.cs#L35
Non-nullable property 'EnvironmentName' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
tests: src/Server/src/Abstractions/Messages/ResourcePublishedSuccessMessage.cs#L37
Non-nullable property 'RequestedBy' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
tests: src/Server/src/Abstractions/Messages/ResourcePublishedSuccessMessage.cs#L39
Non-nullable property 'ErrorMessage' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.