Fix code analysis warnings of dotnet 8

GeoWerkstatt · Jan 31, 2024 · ec8aa0c · ec8aa0c
1 parent 8dd3df5
commit ec8aa0c
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 13 deletions.
diff --git a/src/Controllers/ExportController.cs b/src/Controllers/ExportController.cs
@@ -38,6 +38,6 @@ public async Task<ContentResult> GetAsync(CancellationToken cancellationToken)
             cancellationToken).ConfigureAwait(false);
 
         Response.Headers.ContentDisposition = "attachment; filename=data_export.csv";
-        return Content(await reader.ReadToEndAsync().ConfigureAwait(false), "text/csv", Encoding.UTF8);
+        return Content(await reader.ReadToEndAsync(cancellationToken).ConfigureAwait(false), "text/csv", Encoding.UTF8);
     }
 }
diff --git a/src/Controllers/StandortController.cs b/src/Controllers/StandortController.cs
@@ -21,8 +21,10 @@ public async Task<IEnumerable<Standort>> GetAsync(
          string? gemeinde = null, string? gbnummer = null, string? bezeichnung = null, DateTime? erstellungsdatum = null, DateTime? mutationsdatum = null)
     {
         var standorte = Context.Standorte.Include(s => s.Bohrungen).AsQueryable();
-#pragma warning disable CA1304 // Specify CultureInfo
 
+#pragma warning disable CA1304 // Specify CultureInfo
+#pragma warning disable CA1311 // Specify a culture or use an invariant version
+#pragma warning disable CA1862 // Use the 'StringComparison' method overloads to perform case-insensitive string comparisons
         if (!string.IsNullOrEmpty(gemeinde))
         {
             standorte = standorte.Where(s => s.Gemeinde.ToLower().Contains(gemeinde.ToLower()));
@@ -36,8 +38,10 @@ public async Task<IEnumerable<Standort>> GetAsync(
         if (!string.IsNullOrEmpty(bezeichnung))
         {
             standorte = standorte.Where(s => s.Bezeichnung.ToLower().Contains(bezeichnung.ToLower()));
-#pragma warning restore CA1304 // Specify CultureInfo
         }
+#pragma warning restore CA1304 // Specify CultureInfo
+#pragma warning restore CA1311 // Specify a culture or use an invariant version
+#pragma warning restore CA1862 // Use the 'StringComparison' method overloads to perform case-insensitive string comparisons
 
         if (erstellungsdatum != null)
         {

diff --git a/src/EwsContextExtensions.cs b/src/EwsContextExtensions.cs
@@ -228,15 +228,15 @@ public static void SeedData(this EwsContext context)
         context.SaveChangesWithoutUpdatingChangeInformation();
 
         // Sync all database sequences
-        context.Database.ExecuteSqlRaw($"SELECT setval(pg_get_serial_sequence('bohrung.codetyp', 'codetyp_id'), {codetyp_ids - 1})");
-        context.Database.ExecuteSqlRaw($"SELECT setval(pg_get_serial_sequence('bohrung.code', 'code_id'), {code_ids - 1})");
-        context.Database.ExecuteSqlRaw($"SELECT setval(pg_get_serial_sequence('bohrung.codeschicht', 'codeschicht_id'),{codeschicht_ids - 1})");
-        context.Database.ExecuteSqlRaw($"SELECT setval(pg_get_serial_sequence('bohrung.standort', 'standort_id'),{standort_ids - 1})");
-        context.Database.ExecuteSqlRaw($"SELECT setval(pg_get_serial_sequence('bohrung.bohrung', 'bohrung_id'),{bohrung_ids - 1})");
-        context.Database.ExecuteSqlRaw($"SELECT setval(pg_get_serial_sequence('bohrung.bohrprofil', 'bohrprofil_id'),{bohrprofil_ids - 1})");
-        context.Database.ExecuteSqlRaw($"SELECT setval(pg_get_serial_sequence('bohrung.schicht', 'schicht_id'),{schicht_ids - 1})");
-        context.Database.ExecuteSqlRaw($"SELECT setval(pg_get_serial_sequence('bohrung.vorkommnis', 'vorkommnis_id'),{vorkommnis_ids - 1})");
-        context.Database.ExecuteSqlRaw($"SELECT setval(pg_get_serial_sequence('bohrung.user', 'user_id'),{user_ids - 1})");
+        context.Database.ExecuteSql($"SELECT setval(pg_get_serial_sequence('bohrung.codetyp', 'codetyp_id'), {codetyp_ids - 1})");
+        context.Database.ExecuteSql($"SELECT setval(pg_get_serial_sequence('bohrung.code', 'code_id'), {code_ids - 1})");
+        context.Database.ExecuteSql($"SELECT setval(pg_get_serial_sequence('bohrung.codeschicht', 'codeschicht_id'),{codeschicht_ids - 1})");
+        context.Database.ExecuteSql($"SELECT setval(pg_get_serial_sequence('bohrung.standort', 'standort_id'),{standort_ids - 1})");
+        context.Database.ExecuteSql($"SELECT setval(pg_get_serial_sequence('bohrung.bohrung', 'bohrung_id'),{bohrung_ids - 1})");
+        context.Database.ExecuteSql($"SELECT setval(pg_get_serial_sequence('bohrung.bohrprofil', 'bohrprofil_id'),{bohrprofil_ids - 1})");
+        context.Database.ExecuteSql($"SELECT setval(pg_get_serial_sequence('bohrung.schicht', 'schicht_id'),{schicht_ids - 1})");
+        context.Database.ExecuteSql($"SELECT setval(pg_get_serial_sequence('bohrung.vorkommnis', 'vorkommnis_id'),{vorkommnis_ids - 1})");
+        context.Database.ExecuteSql($"SELECT setval(pg_get_serial_sequence('bohrung.user', 'user_id'),{user_ids - 1})");
 
         transaction.Commit();
     }

diff --git a/src/Program.cs b/src/Program.cs
@@ -49,7 +49,7 @@
             ValidateAudience = false,
             ValidateLifetime = true,
             ValidateIssuerSigningKey = true,
-            IssuerSigningKey = new SymmetricSecurityKey(Encoding.ASCII.GetBytes(builder.Configuration["Auth:IssuerSigningKey"])),
+            IssuerSigningKey = new SymmetricSecurityKey(Encoding.ASCII.GetBytes(builder.Configuration["Auth:IssuerSigningKey"]!)),
         };
     });