CodeQL Scan #360
dougwaldronAnnotations
10 warnings
|
Analyze with CodeQL (csharp):
FMS.Infrastructure/Migrations/20240223163127_UpdateSPgetNearbyFacilities.cs#L46
Use 'Guid.NewGuid()' or 'Guid.Empty' or add arguments to this GUID instantiation. (https://rules.sonarsource.com/csharp/RSPEC-4581)
|
|
Analyze with CodeQL (csharp):
FMS/Helpers/URLHelper.cs#L40
'WebRequest.Create(string)' is obsolete: 'WebRequest, HttpWebRequest, ServicePoint, and WebClient are obsolete. Use HttpClient instead.' (https://aka.ms/dotnet-warnings/SYSLIB0014)
|
|
Analyze with CodeQL (csharp):
FMS/Helpers/URLHelper.cs#L35
Remove the unused private method 'RemoteSiteExists'. (https://rules.sonarsource.com/csharp/RSPEC-1144)
|
|
Analyze with CodeQL (csharp):
FMS/Pages/Account/Index.cshtml.cs#L21
'System.Exception' should not be thrown by user code. (https://rules.sonarsource.com/csharp/RSPEC-112)
|
|
Analyze with CodeQL (csharp):
FMS/Pages/Account/ExternalLogin.cshtml.cs#L180
Change this condition so that it does not always evaluate to 'True'. (https://rules.sonarsource.com/csharp/RSPEC-2589)
|
|
Analyze with CodeQL (csharp):
tests/FMS.Infrastructure.Tests/CabinetRepositoryTest.cs#L209
Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
|
|
Analyze with CodeQL (csharp):
tests/FMS.Infrastructure.Tests/CabinetRepositoryTest.cs#L190
Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
|
|
Analyze with CodeQL (csharp):
tests/FMS.Infrastructure.Tests/CabinetRepositoryTest.cs#L230
Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
|
|
Analyze with CodeQL (csharp):
tests/FMS.Infrastructure.Tests/CabinetRepositoryTest.cs#L249
Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
|
|
Analyze with CodeQL (csharp):
tests/FMS.Infrastructure.Tests/FileRepositoryTests.cs#L242
Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
|