Skip to content

Commit 4ed3e91

Browse files
committed
Refactor ProblemConstants usage in ServiceCollectionExtensions; streamline extension key references
1 parent 7080b93 commit 4ed3e91

2 files changed

Lines changed: 2 additions & 69 deletions

File tree

ManagedCode.Communication.Extensions/Extensions/ServiceCollectionExtensions.cs

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
using Microsoft.AspNetCore.WebUtilities;
77
using Microsoft.Extensions.DependencyInjection;
88
using Microsoft.Extensions.Hosting;
9-
using ManagedCode.Communication.Extensions.Constants;
9+
using static ManagedCode.Communication.Extensions.Constants.ProblemConstants;
1010

1111
namespace ManagedCode.Communication.Extensions.Extensions;
1212

@@ -50,22 +50,13 @@ public static IServiceCollection AddDefaultProblemDetails(this IServiceCollectio
5050
context.ProblemDetails.Type ??= $"https://httpstatuses.io/{statusCode}";
5151
context.ProblemDetails.Title ??= ReasonPhrases.GetReasonPhrase(statusCode);
5252
context.ProblemDetails.Instance ??= context.HttpContext.Request.Path;
53-
context.ProblemDetails.Extensions.TryAdd(ProblemConstants.ExtensionKeys.TraceId, Activity.Current?.Id ?? context.HttpContext.TraceIdentifier);
53+
context.ProblemDetails.Extensions.TryAdd(ExtensionKeys.TraceId, Activity.Current?.Id ?? context.HttpContext.TraceIdentifier);
5454
};
5555
});
5656

5757
return services;
5858
}
5959

60-
public static IServiceCollection AddCommunicationExceptionHandler(this IServiceCollection services)
61-
{
62-
// Ensures that the ProblemDetails service is registered.
63-
services.AddProblemDetails();
64-
65-
services.AddExceptionHandler<CommunicationExceptionHandler>();
66-
return services;
67-
}
68-
6960
public static IServiceCollection AddCommunicationFilters<TExceptionFilter, TModelValidationFilter, THubExceptionFilter>(
7061
this IServiceCollection services)
7162
where TExceptionFilter : ExceptionFilterBase

ManagedCode.Communication.Extensions/sdf.cs

Lines changed: 0 additions & 58 deletions
This file was deleted.

0 commit comments

Comments
 (0)