Custom Metrics for Vault #1873
Annotations
1 error and 12 warnings
Run Analysis
Process completed with exit code 2.
|
Build:
test/ProjectOrigin.Vault.Tests/REST/TransfersControllerTests.cs#L33
Non-nullable field '_transferMetrics' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the field as nullable.
|
Build:
test/ProjectOrigin.Vault.Tests/REST/TransfersControllerTests.cs#L33
Non-nullable field '_transferMetrics' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the field as nullable.
|
Run Analysis:
src/ProjectOrigin.Vault/Activities/SendInformationToReceiverWalletActivity.cs#L50
Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)
|
Run Analysis:
src/ProjectOrigin.Vault/CommandHandlers/ClaimCertificateCommandHandler.cs#L65
Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)
|
Run Analysis:
src/ProjectOrigin.Vault/CommandHandlers/TransferCertificateCommandHandler.cs#L102
Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)
|
Run Analysis:
src/ProjectOrigin.Vault/Activities/SendInformationToReceiverWalletActivity.cs#L109
Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)
|
Run Analysis:
src/ProjectOrigin.Vault/Activities/SendInformationToReceiverWalletActivity.cs#L72
Reduce the number of Information logging calls within this code block from 4 to the 2 allowed. (https://rules.sonarsource.com/csharp/RSPEC-6664)
|
Run Analysis:
src/ProjectOrigin.Vault/Activities/SendInformationToReceiverWalletActivity.cs#L118
Reduce the number of Information logging calls within this code block from 3 to the 2 allowed. (https://rules.sonarsource.com/csharp/RSPEC-6664)
|
Run Analysis:
src/ProjectOrigin.Vault/Startup.cs#L177
Make 'PrintNetworkOptions' a static method. (https://rules.sonarsource.com/csharp/RSPEC-2325)
|
Run Analysis:
src/ProjectOrigin.Vault/Services/REST/v1/CertificatesController.cs#L174
Value type property used as input in a controller action should be nullable, required or annotated with the JsonRequiredAttribute to avoid under-posting. (https://rules.sonarsource.com/csharp/RSPEC-6964)
|
Run Analysis:
src/ProjectOrigin.Vault/Services/REST/v1/CertificatesController.cs#L179
Value type property used as input in a controller action should be nullable, required or annotated with the JsonRequiredAttribute to avoid under-posting. (https://rules.sonarsource.com/csharp/RSPEC-6964)
|
Run Analysis:
src/ProjectOrigin.Vault/Services/REST/v1/ClaimsController.cs#L106
Remove the unused local variable 'timeZoneInfo'. (https://rules.sonarsource.com/csharp/RSPEC-1481)
|
Loading