Skip to content

Commit f760e8f

Browse files
bkoelmanbkoelman
committed
This reverts commit a00ab50.
1 parent a00ab50 commit f760e8f

File tree

15 files changed

+49
-82
lines changed

15 files changed

+49
-82
lines changed

‎benchmarks/Deserialization/DeserializationBenchmarkBase.cs‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
using System.ComponentModel.Design;
2+
using System.Text.Json;
23
using JetBrains.Annotations;
34
using JsonApiDotNetCore.Configuration;
45
using JsonApiDotNetCore.Middleware;
56
using JsonApiDotNetCore.Resources;
67
using JsonApiDotNetCore.Resources.Annotations;
7-
using JsonApiDotNetCore.Serialization;
88
using JsonApiDotNetCore.Serialization.JsonConverters;
99
using JsonApiDotNetCore.Serialization.Request.Adapters;
1010
using Microsoft.Extensions.Logging.Abstractions;
@@ -13,15 +13,15 @@ namespace Benchmarks.Deserialization;
1313

1414
public abstract class DeserializationBenchmarkBase
1515
{
16-
protected readonly JsonApiSerializationContext SerializationReadContext;
16+
protected readonly JsonSerializerOptions SerializerReadOptions;
1717
protected readonly DocumentAdapter DocumentAdapter;
1818

1919
protected DeserializationBenchmarkBase()
2020
{
2121
var options = new JsonApiOptions();
2222
IResourceGraph resourceGraph = new ResourceGraphBuilder(options, NullLoggerFactory.Instance).Add<IncomingResource, int>().Build();
2323
options.SerializerOptions.Converters.Add(new ResourceObjectConverter(resourceGraph));
24-
SerializationReadContext = ((IJsonApiOptions)options).SerializationReadContext;
24+
SerializerReadOptions = ((IJsonApiOptions)options).SerializerReadOptions;
2525

2626
var serviceContainer = new ServiceContainer();
2727
var resourceFactory = new ResourceFactory(serviceContainer);

‎benchmarks/Deserialization/OperationsDeserializationBenchmarks.cs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -270,7 +270,7 @@ public class OperationsDeserializationBenchmarks : DeserializationBenchmarkBase
270270
[Benchmark]
271271
public object? DeserializeOperationsRequest()
272272
{
273-
Document document = JsonSerializer.Deserialize(RequestBody, SerializationReadContext.Document)!;
273+
var document = JsonSerializer.Deserialize<Document>(RequestBody, SerializerReadOptions)!;
274274
return DocumentAdapter.Convert(document);
275275
}
276276

‎benchmarks/Deserialization/ResourceDeserializationBenchmarks.cs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,7 @@ public class ResourceDeserializationBenchmarks : DeserializationBenchmarkBase
133133
[Benchmark]
134134
public object? DeserializeResourceRequest()
135135
{
136-
Document document = JsonSerializer.Deserialize(RequestBody, SerializationReadContext.Document)!;
136+
var document = JsonSerializer.Deserialize<Document>(RequestBody, SerializerReadOptions)!;
137137
return DocumentAdapter.Convert(document);
138138
}
139139

‎benchmarks/Serialization/OperationsSerializationBenchmarks.cs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ private static IEnumerable<OperationContainer> CreateResponseOperations(IJsonApi
116116
public string SerializeOperationsResponse()
117117
{
118118
Document responseDocument = ResponseModelAdapter.Convert(_responseOperations);
119-
return JsonSerializer.Serialize(responseDocument, SerializationWriteContext.Document);
119+
return JsonSerializer.Serialize(responseDocument, SerializerWriteOptions);
120120
}
121121

122122
protected override JsonApiRequest CreateJsonApiRequest(IResourceGraph resourceGraph)

‎benchmarks/Serialization/ResourceSerializationBenchmarks.cs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ private static OutgoingResource CreateResponseResource()
107107
public string SerializeResourceResponse()
108108
{
109109
Document responseDocument = ResponseModelAdapter.Convert(ResponseResource);
110-
return JsonSerializer.Serialize(responseDocument, SerializationWriteContext.Document);
110+
return JsonSerializer.Serialize(responseDocument, SerializerWriteOptions);
111111
}
112112

113113
protected override JsonApiRequest CreateJsonApiRequest(IResourceGraph resourceGraph)

‎benchmarks/Serialization/SerializationBenchmarkBase.cs‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
using System.Text.Json;
12
using System.Text.Json.Serialization;
23
using Benchmarks.Tools;
34
using JetBrains.Annotations;
@@ -7,15 +8,14 @@
78
using JsonApiDotNetCore.Queries.Internal;
89
using JsonApiDotNetCore.Resources;
910
using JsonApiDotNetCore.Resources.Annotations;
10-
using JsonApiDotNetCore.Serialization;
1111
using JsonApiDotNetCore.Serialization.Response;
1212
using Microsoft.Extensions.Logging.Abstractions;
1313

1414
namespace Benchmarks.Serialization;
1515

1616
public abstract class SerializationBenchmarkBase
1717
{
18-
protected readonly JsonApiSerializationContext SerializationWriteContext;
18+
protected readonly JsonSerializerOptions SerializerWriteOptions;
1919
protected readonly IResponseModelAdapter ResponseModelAdapter;
2020
protected readonly IResourceGraph ResourceGraph;
2121

@@ -33,7 +33,7 @@ protected SerializationBenchmarkBase()
3333
};
3434

3535
ResourceGraph = new ResourceGraphBuilder(options, NullLoggerFactory.Instance).Add<OutgoingResource, int>().Build();
36-
SerializationWriteContext = ((IJsonApiOptions)options).SerializationWriteContext;
36+
SerializerWriteOptions = ((IJsonApiOptions)options).SerializerWriteOptions;
3737

3838
// ReSharper disable VirtualMemberCallInConstructor
3939
JsonApiRequest request = CreateJsonApiRequest(ResourceGraph);

‎src/JsonApiDotNetCore/Configuration/IJsonApiOptions.cs‎

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,13 @@
11
using System.Data;
22
using System.Text.Json;
3-
using JetBrains.Annotations;
43
using JsonApiDotNetCore.Resources.Annotations;
5-
using JsonApiDotNetCore.Serialization;
64
using JsonApiDotNetCore.Serialization.Objects;
75

86
namespace JsonApiDotNetCore.Configuration;
97

108
/// <summary>
119
/// Global options that configure the behavior of JsonApiDotNetCore.
1210
/// </summary>
13-
[PublicAPI]
1411
public interface IJsonApiOptions
1512
{
1613
/// <summary>
@@ -159,27 +156,13 @@ public interface IJsonApiOptions
159156
/// </example>
160157
JsonSerializerOptions SerializerOptions { get; }
161158

162-
/// <summary>
163-
/// Gets the source-generated JSON serialization context used for deserializing request bodies. This value is based on <see cref="SerializerOptions" />
164-
/// and is intended for internal use.
165-
/// </summary>
166-
JsonApiSerializationContext SerializationReadContext { get; }
167-
168159
/// <summary>
169160
/// Gets the settings used for deserializing request bodies. This value is based on <see cref="SerializerOptions" /> and is intended for internal use.
170161
/// </summary>
171-
[Obsolete("Use SerializationReadContext.Options instead.")]
172162
JsonSerializerOptions SerializerReadOptions { get; }
173163

174-
/// <summary>
175-
/// Gets the source-generated JSON serialization context used for serializing response bodies. This value is based on <see cref="SerializerOptions" />
176-
/// and is intended for internal use.
177-
/// </summary>
178-
JsonApiSerializationContext SerializationWriteContext { get; }
179-
180164
/// <summary>
181165
/// Gets the settings used for serializing response bodies. This value is based on <see cref="SerializerOptions" /> and is intended for internal use.
182166
/// </summary>
183-
[Obsolete("Use SerializationWriteContext.Options instead.")]
184167
JsonSerializerOptions SerializerWriteOptions { get; }
185168
}

‎src/JsonApiDotNetCore/Configuration/JsonApiOptions.cs‎

Lines changed: 8 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
using System.Text.Json;
44
using JetBrains.Annotations;
55
using JsonApiDotNetCore.Resources.Annotations;
6-
using JsonApiDotNetCore.Serialization;
76
using JsonApiDotNetCore.Serialization.JsonConverters;
87

98
namespace JsonApiDotNetCore.Configuration;
@@ -12,20 +11,14 @@ namespace JsonApiDotNetCore.Configuration;
1211
[PublicAPI]
1312
public sealed class JsonApiOptions : IJsonApiOptions
1413
{
15-
private readonly Lazy<JsonApiSerializationContext> _lazySerializerReadContext;
16-
private readonly Lazy<JsonApiSerializationContext> _lazySerializerWriteContext;
14+
private readonly Lazy<JsonSerializerOptions> _lazySerializerWriteOptions;
15+
private readonly Lazy<JsonSerializerOptions> _lazySerializerReadOptions;
1716

1817
/// <inheritdoc />
19-
JsonApiSerializationContext IJsonApiOptions.SerializationReadContext => _lazySerializerReadContext.Value;
18+
JsonSerializerOptions IJsonApiOptions.SerializerReadOptions => _lazySerializerReadOptions.Value;
2019

2120
/// <inheritdoc />
22-
JsonSerializerOptions IJsonApiOptions.SerializerReadOptions => ((IJsonApiOptions)this).SerializationReadContext.Options;
23-
24-
/// <inheritdoc />
25-
JsonApiSerializationContext IJsonApiOptions.SerializationWriteContext => _lazySerializerWriteContext.Value;
26-
27-
/// <inheritdoc />
28-
JsonSerializerOptions IJsonApiOptions.SerializerWriteOptions => ((IJsonApiOptions)this).SerializationWriteContext.Options;
21+
JsonSerializerOptions IJsonApiOptions.SerializerWriteOptions => _lazySerializerWriteOptions.Value;
2922

3023
/// <inheritdoc />
3124
public string? Namespace { get; set; }
@@ -117,16 +110,16 @@ static JsonApiOptions()
117110

118111
public JsonApiOptions()
119112
{
120-
_lazySerializerReadContext = new Lazy<JsonApiSerializationContext>(() => new JsonApiSerializationContext(new JsonSerializerOptions(SerializerOptions)),
121-
LazyThreadSafetyMode.ExecutionAndPublication);
113+
_lazySerializerReadOptions =
114+
new Lazy<JsonSerializerOptions>(() => new JsonSerializerOptions(SerializerOptions), LazyThreadSafetyMode.ExecutionAndPublication);
122115

123-
_lazySerializerWriteContext = new Lazy<JsonApiSerializationContext>(() => new JsonApiSerializationContext(new JsonSerializerOptions(SerializerOptions)
116+
_lazySerializerWriteOptions = new Lazy<JsonSerializerOptions>(() => new JsonSerializerOptions(SerializerOptions)
124117
{
125118
Converters =
126119
{
127120
new WriteOnlyDocumentConverter(),
128121
new WriteOnlyRelationshipObjectConverter()
129122
}
130-
}), LazyThreadSafetyMode.ExecutionAndPublication);
123+
}, LazyThreadSafetyMode.ExecutionAndPublication);
131124
}
132125
}

‎src/JsonApiDotNetCore/Middleware/JsonApiMiddleware.cs‎

Lines changed: 13 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44
using JsonApiDotNetCore.Configuration;
55
using JsonApiDotNetCore.Diagnostics;
66
using JsonApiDotNetCore.Resources.Annotations;
7-
using JsonApiDotNetCore.Serialization;
87
using JsonApiDotNetCore.Serialization.Objects;
98
using Microsoft.AspNetCore.Http;
109
using Microsoft.AspNetCore.Http.Extensions;
@@ -45,7 +44,7 @@ public async Task InvokeAsync(HttpContext httpContext, IControllerResourceMappin
4544

4645
using (CodeTimingSessionManager.Current.Measure("JSON:API middleware"))
4746
{
48-
if (!await ValidateIfMatchHeaderAsync(httpContext, options.SerializationWriteContext))
47+
if (!await ValidateIfMatchHeaderAsync(httpContext, options.SerializerWriteOptions))
4948
{
5049
return;
5150
}
@@ -55,8 +54,8 @@ public async Task InvokeAsync(HttpContext httpContext, IControllerResourceMappin
5554

5655
if (primaryResourceType != null)
5756
{
58-
if (!await ValidateContentTypeHeaderAsync(HeaderConstants.MediaType, httpContext, options.SerializationWriteContext) ||
59-
!await ValidateAcceptHeaderAsync(MediaType, httpContext, options.SerializationWriteContext))
57+
if (!await ValidateContentTypeHeaderAsync(HeaderConstants.MediaType, httpContext, options.SerializerWriteOptions) ||
58+
!await ValidateAcceptHeaderAsync(MediaType, httpContext, options.SerializerWriteOptions))
6059
{
6160
return;
6261
}
@@ -67,8 +66,8 @@ public async Task InvokeAsync(HttpContext httpContext, IControllerResourceMappin
6766
}
6867
else if (IsRouteForOperations(routeValues))
6968
{
70-
if (!await ValidateContentTypeHeaderAsync(HeaderConstants.AtomicOperationsMediaType, httpContext, options.SerializationWriteContext) ||
71-
!await ValidateAcceptHeaderAsync(AtomicOperationsMediaType, httpContext, options.SerializationWriteContext))
69+
if (!await ValidateContentTypeHeaderAsync(HeaderConstants.AtomicOperationsMediaType, httpContext, options.SerializerWriteOptions) ||
70+
!await ValidateAcceptHeaderAsync(AtomicOperationsMediaType, httpContext, options.SerializerWriteOptions))
7271
{
7372
return;
7473
}
@@ -92,11 +91,11 @@ public async Task InvokeAsync(HttpContext httpContext, IControllerResourceMappin
9291
}
9392
}
9493

95-
private async Task<bool> ValidateIfMatchHeaderAsync(HttpContext httpContext, JsonApiSerializationContext serializationContext)
94+
private async Task<bool> ValidateIfMatchHeaderAsync(HttpContext httpContext, JsonSerializerOptions serializerOptions)
9695
{
9796
if (httpContext.Request.Headers.ContainsKey(HeaderNames.IfMatch))
9897
{
99-
await FlushResponseAsync(httpContext.Response, serializationContext, new ErrorObject(HttpStatusCode.PreconditionFailed)
98+
await FlushResponseAsync(httpContext.Response, serializerOptions, new ErrorObject(HttpStatusCode.PreconditionFailed)
10099
{
101100
Title = "Detection of mid-air edit collisions using ETags is not supported.",
102101
Source = new ErrorSource
@@ -121,14 +120,13 @@ private async Task<bool> ValidateIfMatchHeaderAsync(HttpContext httpContext, Jso
121120
: null;
122121
}
123122

124-
private static async Task<bool> ValidateContentTypeHeaderAsync(string allowedContentType, HttpContext httpContext,
125-
JsonApiSerializationContext serializationContext)
123+
private static async Task<bool> ValidateContentTypeHeaderAsync(string allowedContentType, HttpContext httpContext, JsonSerializerOptions serializerOptions)
126124
{
127125
string? contentType = httpContext.Request.ContentType;
128126

129127
if (contentType != null && contentType != allowedContentType)
130128
{
131-
await FlushResponseAsync(httpContext.Response, serializationContext, new ErrorObject(HttpStatusCode.UnsupportedMediaType)
129+
await FlushResponseAsync(httpContext.Response, serializerOptions, new ErrorObject(HttpStatusCode.UnsupportedMediaType)
132130
{
133131
Title = "The specified Content-Type header value is not supported.",
134132
Detail = $"Please specify '{allowedContentType}' instead of '{contentType}' for the Content-Type header value.",
@@ -145,7 +143,7 @@ private static async Task<bool> ValidateContentTypeHeaderAsync(string allowedCon
145143
}
146144

147145
private static async Task<bool> ValidateAcceptHeaderAsync(MediaTypeHeaderValue allowedMediaTypeValue, HttpContext httpContext,
148-
JsonApiSerializationContext serializationContext)
146+
JsonSerializerOptions serializerOptions)
149147
{
150148
string[] acceptHeaders = httpContext.Request.Headers.GetCommaSeparatedValues("Accept");
151149

@@ -178,7 +176,7 @@ private static async Task<bool> ValidateAcceptHeaderAsync(MediaTypeHeaderValue a
178176

179177
if (!seenCompatibleMediaType)
180178
{
181-
await FlushResponseAsync(httpContext.Response, serializationContext, new ErrorObject(HttpStatusCode.NotAcceptable)
179+
await FlushResponseAsync(httpContext.Response, serializerOptions, new ErrorObject(HttpStatusCode.NotAcceptable)
182180
{
183181
Title = "The specified Accept header value does not contain any supported media types.",
184182
Detail = $"Please include '{allowedMediaTypeValue}' in the Accept header values.",
@@ -194,7 +192,7 @@ private static async Task<bool> ValidateAcceptHeaderAsync(MediaTypeHeaderValue a
194192
return true;
195193
}
196194

197-
private static async Task FlushResponseAsync(HttpResponse httpResponse, JsonApiSerializationContext serializationContext, ErrorObject error)
195+
private static async Task FlushResponseAsync(HttpResponse httpResponse, JsonSerializerOptions serializerOptions, ErrorObject error)
198196
{
199197
httpResponse.ContentType = HeaderConstants.MediaType;
200198
httpResponse.StatusCode = (int)error.StatusCode;
@@ -204,7 +202,7 @@ private static async Task FlushResponseAsync(HttpResponse httpResponse, JsonApiS
204202
Errors = error.AsList()
205203
};
206204

207-
await JsonSerializer.SerializeAsync(httpResponse.Body, errorDocument, serializationContext.Document);
205+
await JsonSerializer.SerializeAsync(httpResponse.Body, errorDocument, serializerOptions);
208206
await httpResponse.Body.FlushAsync();
209207
}
210208

‎src/JsonApiDotNetCore/Serialization/JsonApiSerializationContext.cs‎

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

0 commit comments

Comments
 (0)