Remove unused pre-compilation conditions

The conditions were used for pre-.NET 8 which are not supported anymore.

Author: Carl-Hugo

samples/WebApi.Minimal/Program.cs

@@ -9,15 +9,9 @@
 
 var builder = WebApplication.CreateBuilder(args);
 #if CHANGE_PROPERTY_NAME_POLICY
-#if NET8_0_OR_GREATER
 builder.Services.ConfigureHttpJsonOptions(options => {
     options.SerializerOptions.PropertyNamingPolicy = JsonNamingPolicy.SnakeCaseUpper;
 });
-#else
-builder.Services.Configure<JsonOptions>(options => {
-    options.JsonSerializerOptions.PropertyNamingPolicy = JsonNamingPolicy.CamelCase;
-});
-#endif
 #endif
 builder.AddExceptionMapper(builder =>
 {
@@ -74,9 +68,7 @@
     "---[Others]---",
     "/fallback",
     "/a-url-that-does-not-exist",
-#if NET7_0_OR_GREATER
     "/fluent-validation?name=&description=&range=0",
-#endif
 });
 app.MapGet("/BadRequestException", context => throw new BadRequestException());
 app.MapGet("/ConflictException", context => throw new ConflictException());
@@ -96,7 +88,6 @@
 app.MapGet("/MyUnauthorizedException", context => throw new MyUnauthorizedException(Random.Shared.Next(100) % 2 == 0 ? "John" : "Jane"));
 
 app.MapGet("/fallback", context => throw new Exception("An error that gets handled by the fallback handler."));
-#if NET7_0_OR_GREATER
 app.MapGet("/fluent-validation", ([AsParameters] Entity entity) =>
 {
     var validator = new EntityValidator();
@@ -107,7 +98,6 @@
     }
     return TypedResults.Ok(entity);
 });
-#endif
 app.Run();
 
 

src/ForEvolve.ExceptionMapper/DependencyInjection/ServiceCollectionExtensions.cs

@@ -53,23 +53,16 @@ public static WebApplicationBuilder AddExceptionMapper(this WebApplicationBuilde
 
     private static void AddSerializationHandler(this IServiceCollection services, IConfiguration configuration)
     {
-#if NET7_0_OR_GREATER
         services.ConfigureHttpJsonOptions(options => {
             options.SerializerOptions.DictionaryKeyPolicy = options.SerializerOptions.PropertyNamingPolicy;
         });
-#endif
         services
             .AddOptions<ProblemDetailsSerializationOptions>()
             .Bind(configuration.GetSection("ExceptionMapper:ProblemDetailsSerialization"))
             .ValidateOnStart()
         ;
         services.AddSingleton(sp => sp.GetRequiredService<IOptions<ProblemDetailsSerializationOptions>>().Value);
-#if NET7_0_OR_GREATER
         services.AddProblemDetails();
-#endif
-        // Workaround: binding a local copy of the DefaultProblemDetailsFactory because the .NET class is internal.
-        // Moreover, the only way to add the class is by calling the AddMvcCore method, which add way more services.
-        // So until we can add the DefaultProblemDetailsFactory
         services.TryAddSingleton<ProblemDetailsFactory, DefaultProblemDetailsFactory>();
         services.TryAddSingleton<IExceptionSerializer, ProblemDetailsSerializationHandler>();
     }

src/ForEvolve.ExceptionMapper/Serialization/DefaultProblemDetailsFactory.cs

@@ -1,4 +1,5 @@
-#if !NET9_0_OR_GREATER
+// Load this only for .NET 8.
+#if !NET9_0_OR_GREATER
 // Licensed to the .NET Foundation under one or more agreements.
 // The .NET Foundation licenses this file to you under the MIT license.
 
@@ -102,42 +103,3 @@ private void ApplyProblemDetailsDefaults(HttpContext httpContext, ProblemDetails
     }
 }
 #endif
-#if NET6_0
-public class ProblemDetailsOptions
-{
-    /// <summary>
-    /// The operation that customizes the current <see cref="Mvc.ProblemDetails"/> instance.
-    /// </summary>
-    public Action<ProblemDetailsContext>? CustomizeProblemDetails { get; set; }
-}
-
-public class ProblemDetailsContext
-{
-    private ProblemDetails? _problemDetails;
-
-    /// <summary>
-    /// The <see cref="HttpContext"/> associated with the current request being processed by the filter.
-    /// </summary>
-    public HttpContext? HttpContext { get; init; }
-
-    /// <summary>
-    /// A collection of additional arbitrary metadata associated with the current request endpoint.
-    /// </summary>
-    public EndpointMetadataCollection? AdditionalMetadata { get; init; }
-
-    /// <summary>
-    /// An instance of <see cref="ProblemDetails"/> that will be
-    /// used during the response payload generation.
-    /// </summary>
-    public ProblemDetails ProblemDetails
-    {
-        get => _problemDetails ??= new ProblemDetails();
-        set => _problemDetails = value;
-    }
-
-    /// <summary>
-    /// The exception causing the problem or <c>null</c> if no exception information is available.
-    /// </summary>
-    public Exception? Exception { get; init; }
-}
-#endif

src/ForEvolve.ExceptionMapper/Serialization/Json/ProblemDetailsSerializationHandler.cs

@@ -17,15 +17,11 @@ public class ProblemDetailsSerializationHandler : IExceptionSerializer
     private readonly ProblemDetailsFactory _problemDetailsFactory;
     private readonly IHostEnvironment _hostEnvironment;
     private readonly ProblemDetailsSerializationOptions _options;
-#if NET7_0_OR_GREATER
     private readonly IProblemDetailsService _problemDetailsService;
-#endif
     private readonly JsonSerializerOptions _jsonSerializerOptions;
 
     public ProblemDetailsSerializationHandler(
-#if NET7_0_OR_GREATER
         IProblemDetailsService problemDetailsService,
-#endif
         ProblemDetailsFactory problemDetailsFactory,
         IHostEnvironment hostEnvironment,
         ProblemDetailsSerializationOptions options,
@@ -34,9 +30,7 @@ public ProblemDetailsSerializationHandler(
         _problemDetailsFactory = problemDetailsFactory ?? throw new ArgumentNullException(nameof(problemDetailsFactory));
         _hostEnvironment = hostEnvironment ?? throw new ArgumentNullException(nameof(hostEnvironment));
         _options = options ?? throw new ArgumentNullException(nameof(options));
-#if NET7_0_OR_GREATER
         _problemDetailsService = problemDetailsService ?? throw new ArgumentNullException(nameof(problemDetailsService));
-#endif
         _jsonSerializerOptions = jsonOptions.Value.SerializerOptions ?? throw new ArgumentNullException(nameof(jsonOptions));
     }
 
@@ -113,27 +107,13 @@ public async Task ExecuteAsync(ExceptionHandlingContext ctx)
         }
 
         // Output the problem details
-#if NET7_0_OR_GREATER
         var problemDetailsContext = new ProblemDetailsContext
         {
             HttpContext = ctx.HttpContext,
-#if NET8_0_OR_GREATER
             Exception = ctx.Error,
-#endif
             ProblemDetails = problemDetails,
         };
         await _problemDetailsService.WriteAsync(problemDetailsContext);
-#else
-#pragma warning disable CS0618 // Type or member is obsolete
-        ctx.HttpContext.Response.ContentType = _options.ContentType;
-        await JsonSerializer.SerializeAsync(
-            ctx.HttpContext.Response.Body,
-            problemDetails,
-            _jsonSerializerOptions,
-            cancellationToken: ctx.HttpContext.RequestAborted
-        );
-#pragma warning restore CS0618 // Type or member is obsolete
-#endif
     }
 
     private string FormatName(string name)

src/ForEvolve.ExceptionMapper/Serialization/Json/ProblemDetailsSerializationOptions.cs

@@ -6,10 +6,4 @@ public class ProblemDetailsSerializationOptions
 {
     public bool SerializeExceptions { get; set; } = true;
     public Func<ExceptionHandlingContext, bool> DisplayDebugInformation { get; set; } = (ExceptionHandlingContext ctx) => false;
-#if NET6_0
-    private const string _obsoleteMessage = "This property was removed when targeting .NET 7+. The library now leverages the `IProblemDetailsService` interface instead.";
-
-    [Obsolete(_obsoleteMessage)]
-    public string ContentType { get; set; } = "application/problem+json; charset=utf-8";
-#endif
 }