Skip to content

Commit 3fa8078

Browse files
committed
Null check bug fixes
1 parent efe6b75 commit 3fa8078

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

src/Elastic.Transport/Responses/DefaultResponseFactory.cs

+1-3
Original file line numberDiff line numberDiff line change
@@ -79,13 +79,11 @@ private async ValueTask<TResponse> CreateCoreAsync<TResponse>(
7979
IReadOnlyDictionary<TcpState, int>? tcpStats,
8080
CancellationToken cancellationToken = default) where TResponse : TransportResponse, new()
8181
{
82-
responseStream.ThrowIfNull(nameof(responseStream));
83-
8482
var details = InitializeApiCallDetails(endpoint, boundConfiguration, postData, ex, statusCode, headers, contentType, threadPoolStats, tcpStats, contentLength);
8583

8684
TResponse? response = null;
8785

88-
if (MayHaveBody(statusCode, endpoint.Method, contentLength)
86+
if (responseStream is not null && MayHaveBody(statusCode, endpoint.Method, contentLength)
8987
&& TryResolveBuilder<TResponse>(boundConfiguration.ResponseBuilders, boundConfiguration.ProductResponseBuilders, out var builder))
9088
{
9189
var ownsStream = false;

src/Elastic.Transport/Responses/Special/StreamResponse.cs

+1-1
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ protected virtual void Dispose(bool disposing)
5353
if (LinkedDisposables is not null)
5454
{
5555
foreach (var disposable in LinkedDisposables)
56-
disposable.Dispose();
56+
disposable?.Dispose();
5757
}
5858
}
5959

0 commit comments

Comments
 (0)