From fb0a516ecdb5da516600ba612d7808983cda5e42 Mon Sep 17 00:00:00 2001
From: Steve Gordon <sgordon@hotmail.co.uk>
Date: Thu, 21 Nov 2024 10:43:11 +0000
Subject: [PATCH] PR feedback - add null check on ctor

---
 .../Responses/Special/StreamResponseBase.cs          | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/src/Elastic.Transport/Responses/Special/StreamResponseBase.cs b/src/Elastic.Transport/Responses/Special/StreamResponseBase.cs
index 2c5478c..06ebb67 100644
--- a/src/Elastic.Transport/Responses/Special/StreamResponseBase.cs
+++ b/src/Elastic.Transport/Responses/Special/StreamResponseBase.cs
@@ -4,13 +4,14 @@
 
 using System;
 using System.IO;
+using Elastic.Transport.Extensions;
 
 namespace Elastic.Transport;
 
 /// <summary>
 /// A base class for implementing responses that access the raw response stream.
 /// </summary>
-public abstract class StreamResponseBase(Stream stream) : TransportResponse, IDisposable
+public abstract class StreamResponseBase : TransportResponse, IDisposable
 {
 	/// <inheritdoc/>
 	protected internal override bool LeaveOpen => true;
@@ -21,13 +22,20 @@ public abstract class StreamResponseBase(Stream stream) : TransportResponse, IDi
 	/// <remarks>
 	/// <b>MUST</b> be disposed to release the underlying HTTP connection for reuse.
 	/// </remarks>
-	protected Stream Stream { get; } = stream;
+	protected Stream Stream { get; }
 
 	/// <summary>
 	/// Indicates that the response has been disposed and it is not longer safe to access the stream.
 	/// </summary>
 	protected bool Disposed { get; private set; }
 
+	/// <inheritdoc cref="StreamResponseBase"/>
+	public StreamResponseBase(Stream responseStream)
+	{
+		responseStream.ThrowIfNull(nameof(responseStream));
+		Stream = responseStream;
+	}
+
 	/// <summary>
 	/// Disposes the underlying stream.
 	/// </summary>