Minor refactoring focused on self-contained systems (#624) #573
account-management.yml
on: push
Build and Test
3m 59s
Code Style and Linting
0s
Deploy API
/
Production
0s
Deploy Workers
/
Production
0s
Annotations
10 warnings
Build and Test:
application/shared-kernel/SharedKernel/Authentication/UserInfo.cs#L87
"Array.Find" static method should be used instead of the "FirstOrDefault" extension method. (https://rules.sonarsource.com/csharp/RSPEC-6602)
|
Build and Test:
application/AppGateway/Program.cs#L47
Refactor your code not to use hardcoded absolute paths or URIs. (https://rules.sonarsource.com/csharp/RSPEC-1075)
|
Build and Test:
application/account-management/Core/Features/Authentication/Commands/RefreshAuthenticationTokens.cs#L78
Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)
|
Build and Test:
application/account-management/Core/Features/Authentication/Commands/StartLogin.cs#L56
Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)
|
Build and Test:
application/account-management/Core/Database/DatabaseMigrations.cs#L17
Define a constant instead of using this literal 'datetimeoffset' 10 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
Build and Test:
application/account-management/Core/Database/DatabaseMigrations.cs#L19
Define a constant instead of using this literal 'varchar(30)' 4 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
Build and Test:
application/account-management/Core/Features/Authentication/Commands/RefreshAuthenticationTokens.cs#L34
Define a constant instead of using this literal 'Invalid refresh token' 6 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
Build and Test:
application/account-management/Core/Features/Users/Shared/AvatarUpdater.cs#L34
Make sure this weak hash algorithm is not used in a sensitive context here. (https://rules.sonarsource.com/csharp/RSPEC-4790)
|
Build and Test:
application/account-management/Core/Features/Authentication/Commands/CompleteLogin.cs#L21
Constructor has 8 parameters, which is greater than the 7 authorized. (https://rules.sonarsource.com/csharp/RSPEC-107)
|
Build and Test:
application/account-management/Core/Features/Signups/Commands/StartSignup.cs#L53
Collection-specific "Exists" method should be used instead of the "Any" extension. (https://rules.sonarsource.com/csharp/RSPEC-6605)
|
Artifacts
Produced during runtime
Name | Size | |
---|---|---|
account-management-api
|
13.4 MB |
|
account-management-workers
|
11.2 MB |
|