Skip to content

Setup more stuff - #3

Merged
admclamb merged 12 commits into
mainfrom
admclamb/setup-templates-and-application
Oct 4, 2026
Merged

admclamb merged 12 commits into
mainfrom
admclamb/setup-templates-and-application

Fix SonarCloud security vulnerabilities flagged on PR #3

1ba18fa
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Oct 4, 2026 in 1m 9s

Quality Gate failed

Annotations

Check warning on line 133 in StackDuel.Application.Tests/Queries/Games/GetAdminGamesPageable/GetAdminGamesPageableHandlerTests.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Call independent Assert statements from inside an Assert.Multiple

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaSeRIX282Dw4WMTL&open=AaEIaSeRIX282Dw4WMTL&pullRequest=3

Check warning on line 80 in StackDuel.Domain.Tests/Games/Entities/GameTests.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Call independent Assert statements from inside an Assert.Multiple

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaSidIX282Dw4WMTz&open=AaEIaSidIX282Dw4WMTz&pullRequest=3

Check warning on line 90 in StackDuel.IntegrationTests/Users/AdminUsersApiTests.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer 'static readonly' fields over constant array arguments if the called method is called repeatedly and is not mutating the passed array

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaSjiIX282Dw4WMT2&open=AaEIaSjiIX282Dw4WMT2&pullRequest=3

Check warning on line 5 in StackDuel.Api/Requests/Game/GetAdminGamesPageableRequest.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Value type property used as input in a controller action should be nullable, required or annotated with the JsonRequiredAttribute to avoid under-posting.

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaSKdIX282Dw4WMRV&open=AaEIaSKdIX282Dw4WMRV&pullRequest=3

Check warning on line 130 in StackDuel.Api/RateLimiting/RateLimitExtensions.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Change return type of method 'DiscoverAttributes' from 'System.Collections.Generic.IEnumerable<StackDuel.Api.RateLimiting.RateLimitAttribute>' to 'System.Collections.Generic.List<StackDuel.Api.RateLimiting.RateLimitAttribute>' for improved performance

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaSP-IX282Dw4WMSN&open=AaEIaSP-IX282Dw4WMSN&pullRequest=3

Check warning on line 3 in StackDuel.Api/Requests/Problem/GetAdminProblemsPageableRequest.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Value type property used as input in a controller action should be nullable, required or annotated with the JsonRequiredAttribute to avoid under-posting.

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaSLMIX282Dw4WMRg&open=AaEIaSLMIX282Dw4WMRg&pullRequest=3

Check warning on line 56 in StackDuel.Api/Middleware/UserContextMiddleware.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Evaluation of this argument may be expensive and unnecessary if logging is disabled

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaSRuIX282Dw4WMSP&open=AaEIaSRuIX282Dw4WMSP&pullRequest=3

Check warning on line 3 in StackDuel.Api/Requests/Audit/GetAuditLogPageableRequest.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Value type property used as input in a controller action should be nullable, required or annotated with the JsonRequiredAttribute to avoid under-posting.

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaSOMIX282Dw4WMSG&open=AaEIaSOMIX282Dw4WMSG&pullRequest=3

Check warning on line 45 in StackDuel.Application.Tests/LanguageServer/LspFramingTests.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Call independent Assert statements from inside an Assert.Multiple

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaShRIX282Dw4WMTp&open=AaEIaShRIX282Dw4WMTp&pullRequest=3

Check warning on line 116 in StackDuel.Infrastructure/Messaging/Consumers/RabbitMqConsumerService.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Evaluation of this argument may be expensive and unnecessary if logging is disabled

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaRzxIX282Dw4WMQ8&open=AaEIaRzxIX282Dw4WMQ8&pullRequest=3

Check warning on line 3 in StackDuel.Api/Requests/Submission/GetAdminSubmissionsPageableRequest.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Value type property used as input in a controller action should be nullable, required or annotated with the JsonRequiredAttribute to avoid under-posting.

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaSOCIX282Dw4WMSC&open=AaEIaSOCIX282Dw4WMSC&pullRequest=3

Check warning on line 156 in StackDuel.Domain.Tests/User/Entities/UserTests.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use Is.Not.EqualTo constraint instead of direct comparison for better assertion messages in case of failure

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaSiGIX282Dw4WMTw&open=AaEIaSiGIX282Dw4WMTw&pullRequest=3

Check warning on line 3 in StackDuel.Api/Requests/Audit/GetAuditLogPageableRequest.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Value type property used as input in a controller action should be nullable, required or annotated with the JsonRequiredAttribute to avoid under-posting.

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaSOMIX282Dw4WMSE&open=AaEIaSOMIX282Dw4WMSE&pullRequest=3

Check warning on line 102 in StackDuel.Application.Tests/Queries/Problems/GetAdminProblemDetail/GetAdminProblemDetailHandlerTests.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer 'static readonly' fields over constant array arguments if the called method is called repeatedly and is not mutating the passed array

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaSfgIX282Dw4WMTY&open=AaEIaSfgIX282Dw4WMTY&pullRequest=3

Check warning on line 3 in StackDuel.Api/Requests/Problem/GetProblemsPageableRequest.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Value type property used as input in a controller action should be nullable, required or annotated with the JsonRequiredAttribute to avoid under-posting.

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaSL8IX282Dw4WMRq&open=AaEIaSL8IX282Dw4WMRq&pullRequest=3

Check warning on line 34 in StackDuel.Application.Tests/Queries/Languages/GetLanguages/GetLanguagesHandlerTests.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Call independent Assert statements from inside an Assert.Multiple

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaShHIX282Dw4WMTo&open=AaEIaShHIX282Dw4WMTo&pullRequest=3

Check warning on line 74 in StackDuel.Application.Tests/Queries/Games/GetMyActiveGames/GetMyActiveGamesHandlerTests.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Call independent Assert statements from inside an Assert.Multiple

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaSexIX282Dw4WMTR&open=AaEIaSexIX282Dw4WMTR&pullRequest=3

Check warning on line 230 in StackDuel.Application.Tests/Commands/Games/StartGame/StartGameHandlerTests.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Call independent Assert statements from inside an Assert.Multiple

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaSa6IX282Dw4WMS1&open=AaEIaSa6IX282Dw4WMS1&pullRequest=3

Check warning on line 164 in StackDuel.Infrastructure/Migrations/20260926052424_AddAchievements.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer 'static readonly' fields over constant array arguments if the called method is called repeatedly and is not mutating the passed array

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaRsnIX282Dw4WMQR&open=AaEIaRsnIX282Dw4WMQR&pullRequest=3

Check warning on line 3 in StackDuel.Api/Requests/Problem/GetAdminProblemsPageableRequest.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Value type property used as input in a controller action should be nullable, required or annotated with the JsonRequiredAttribute to avoid under-posting.

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaSLMIX282Dw4WMRf&open=AaEIaSLMIX282Dw4WMRf&pullRequest=3

Check warning on line 3 in StackDuel.Api/Requests/FeatureFlags/UpdateFeatureFlagRolloutRequest.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Value type property used as input in a controller action should be nullable, required or annotated with the JsonRequiredAttribute to avoid under-posting.

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaSJ9IX282Dw4WMRR&open=AaEIaSJ9IX282Dw4WMRR&pullRequest=3

Check failure on line 25 in scripts/setup-user-secrets.sh

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use '[[' instead of '[' for conditional tests. The '[[' construct is safer and more feature-rich.

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaSknIX282Dw4WMT_&open=AaEIaSknIX282Dw4WMT_&pullRequest=3

Check warning on line 5 in StackDuel.Api/Requests/Problem/AddProblemsToPoolRequest.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Value type property used as input in a controller action should be nullable, required or annotated with the JsonRequiredAttribute to avoid under-posting.

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaSLrIX282Dw4WMRn&open=AaEIaSLrIX282Dw4WMRn&pullRequest=3

Check warning on line 260 in StackDuel.Infrastructure/Migrations/20260909184707_AddCampaigns.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer 'static readonly' fields over constant array arguments if the called method is called repeatedly and is not mutating the passed array

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaRuxIX282Dw4WMQm&open=AaEIaRuxIX282Dw4WMQm&pullRequest=3

Check warning on line 322 in StackDuel.Infrastructure/ExecutionEngine/StepHandlers/Judge0ExecutionStepHandler.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Change type of parameter 'generationSpecs' from 'System.Collections.Generic.IReadOnlyDictionary<System.Guid, StackDuel.Domain.TestCaseGeneration.Entities.TestCaseGenerationSpec>' to 'System.Collections.Generic.Dictionary<System.Guid, StackDuel.Domain.TestCaseGeneration.Entities.TestCaseGenerationSpec>' for improved performance

See more on https://sonarcloud.io/project/issues?id=Stack-Duel_server&issues=AaEIaR0UIX282Dw4WMRB&open=AaEIaR0UIX282Dw4WMRB&pullRequest=3