Skip to content

Commit f084143

Browse files
committed
refactor: use built-in json options in Minimal APIs
1 parent fde75c1 commit f084143

File tree

4 files changed

+8
-9
lines changed

4 files changed

+8
-9
lines changed

src/Cnblogs.Architecture.Ddd.Cqrs.AspNetCore/CommandEndpointHandler.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
using Cnblogs.Architecture.Ddd.Cqrs.Abstractions;
1+
using Cnblogs.Architecture.Ddd.Cqrs.Abstractions;
22
using MediatR;
33
using Microsoft.AspNetCore.Http;
44
using Microsoft.Extensions.Options;
@@ -47,8 +47,8 @@ public class CommandEndpointHandler(IMediator mediator, IOptions<CqrsHttpOptions
4747
if (commandResponse is IObjectResponse objectResponse)
4848
{
4949
return context.HttpContext.Request.Headers.CqrsVersion() > 1
50-
? Results.Extensions.Cqrs(response, _options.DefaultJsonSerializerOptions)
51-
: Results.Json(objectResponse.GetResult(), _options.DefaultJsonSerializerOptions);
50+
? Results.Extensions.Cqrs(response)
51+
: Results.Json(objectResponse.GetResult());
5252
}
5353

5454
return Results.NoContent();

src/Cnblogs.Architecture.Ddd.Cqrs.AspNetCore/CqrsHttpOptionsInjector.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,8 +50,8 @@ public static CqrsInjector UseCustomCommandErrorResponseMapper(
5050
/// <returns></returns>
5151
public static CqrsInjector AddLongToStringJsonConverter(this CqrsInjector injector)
5252
{
53-
injector.Services.Configure<CqrsHttpOptions>(
54-
o => o.DefaultJsonSerializerOptions.Converters.Add(new LongToStringConverter()));
53+
injector.Services.ConfigureHttpJsonOptions(
54+
o => o.SerializerOptions.Converters.Add(new LongToStringConverter()));
5555
return injector;
5656
}
5757
}

src/Cnblogs.Architecture.Ddd.Cqrs.AspNetCore/QueryEndpointHandler.cs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
using MediatR;
1+
using MediatR;
22
using Microsoft.AspNetCore.Http;
33
using Microsoft.Extensions.Options;
44

@@ -24,8 +24,9 @@ public class QueryEndpointHandler(IMediator mediator, IOptions<CqrsHttpOptions>
2424
}
2525

2626
var response = await mediator.Send(query);
27+
Console.WriteLine("QueryEndpointHandler");
2728
return response == null
2829
? Results.NotFound()
29-
: Results.Json(response, cqrsHttpOptions.Value.DefaultJsonSerializerOptions);
30+
: Results.Json(response);
3031
}
3132
}

test/Cnblogs.Architecture.IntegrationTests/MinimalApiTests.cs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,6 @@ public async Task ResponseJsonWithChineseChars_RemainUnencodedAsync()
1515
var response = await builder.CreateClient().GetAsync("/api/v1/articles");
1616
var content = await response.Content.ReadAsStringAsync();
1717

18-
Console.WriteLine(content);
19-
2018
// Assert
2119
Assert.True(response.IsSuccessStatusCode);
2220
Assert.Contains("开发者", content);

0 commit comments

Comments
 (0)