4213 show failed transfers #1997
tnickelsensonarcloud.yaml
on: pull_request
analyse
/
sonar-analysis
7m 45s
Annotations
10 warnings
|
analyse / sonar-analysis:
src/ProjectOrigin.Vault/Activities/WaitCommittedRegistryTransactionActivity.cs#L45
Refactor this method to reduce its Cognitive Complexity from 21 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
|
|
analyse / sonar-analysis:
src/ProjectOrigin.Vault/Activities/WaitCommittedRegistryTransactionActivity.cs#L110
Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)
|
|
analyse / sonar-analysis:
src/ProjectOrigin.Vault/Activities/UpdateSliceStateActivity.cs#L53
Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)
|
|
analyse / sonar-analysis:
src/ProjectOrigin.Vault/Activities/WaitCommittedRegistryTransactionActivity.cs#L80
Use PascalCase for named placeholders. (https://rules.sonarsource.com/csharp/RSPEC-6678)
|
|
analyse / sonar-analysis:
src/ProjectOrigin.Vault/CommandHandlers/TransferCertificateCommandHandler.cs#L117
Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)
|
|
analyse / sonar-analysis:
src/ProjectOrigin.Vault/Activities/TransferPartialSliceActivity.cs#L117
Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)
|
|
analyse / sonar-analysis:
src/ProjectOrigin.Vault/Activities/SendInformationToReceiverWalletActivity.cs#L75
Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)
|
|
analyse / sonar-analysis:
src/ProjectOrigin.Vault/Activities/SendInformationToReceiverWalletActivity.cs#L91
Reduce the number of Information logging calls within this code block from 4 to the 2 allowed. (https://rules.sonarsource.com/csharp/RSPEC-6664)
|
|
analyse / sonar-analysis:
src/ProjectOrigin.Vault/Activities/SendInformationToReceiverWalletActivity.cs#L132
Reduce the number of Information logging calls within this code block from 3 to the 2 allowed. (https://rules.sonarsource.com/csharp/RSPEC-6664)
|
|
analyse / sonar-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)
|