From 58f08aaad267a2365cea2786a4b5bf357315fe50 Mon Sep 17 00:00:00 2001
From: Florian Bernd <git@flobernd.de>
Date: Tue, 28 Apr 2026 11:02:26 +0200
Subject: [PATCH] Remove obsolete ElasticsearchClientProductRegistration
 overrides (#8891)

(cherry picked from commit 5b708a27f345006e3e992b635d3b86e6ba08a602)
---
 .../ElasticsearchClientProductRegistration.cs | 24 -------------------
 .../Elastic.Clients.Elasticsearch.csproj      |  2 +-
 2 files changed, 1 insertion(+), 25 deletions(-)

diff --git a/src/Elastic.Clients.Elasticsearch/Core/ElasticsearchClientProductRegistration.cs b/src/Elastic.Clients.Elasticsearch/Core/ElasticsearchClientProductRegistration.cs
index 6c5e75b814e..725779fd326 100644
--- a/src/Elastic.Clients.Elasticsearch/Core/ElasticsearchClientProductRegistration.cs
+++ b/src/Elastic.Clients.Elasticsearch/Core/ElasticsearchClientProductRegistration.cs
@@ -3,8 +3,6 @@
 // See the LICENSE file in the project root for more information.
 
 using System;
-using System.Diagnostics.CodeAnalysis;
-using Elastic.Transport;
 using Elastic.Transport.Products.Elasticsearch;
 
 namespace Elastic.Clients.Elasticsearch;
@@ -14,26 +12,4 @@ internal sealed class ElasticsearchClientProductRegistration : ElasticsearchProd
 	public ElasticsearchClientProductRegistration(Type markerType) : base(markerType) { }
 
 	public static ElasticsearchClientProductRegistration DefaultForElasticsearchClientsElasticsearch { get; } = new(typeof(ElasticsearchClient));
-
-	/// <summary>
-	///     Elastic.Clients.Elasticsearch handles 404 in its <see cref="ElasticsearchResponse.IsValidResponse" />, we do not want the low level client throwing
-	///     exceptions
-	///     when <see cref="IRequestConfiguration.ThrowExceptions" /> is enabled for 404's. The client is in charge of
-	///     composing paths
-	///     so a 404 never signals a wrong URL but a missing entity.
-	/// </summary>
-	public override bool HttpStatusCodeClassifier(HttpMethod method, int statusCode) =>
-		statusCode is >= 200 and < 300 or 404;
-
-	/// <summary>
-	///     Makes the low level transport aware of Elastic.Clients.Elasticsearch's <see cref="ElasticsearchResponse" />
-	///     so that it can peek in to its exposed error when reporting failures.
-	/// </summary>
-	public override bool TryGetServerErrorReason<TResponse>(TResponse response, [NotNullWhen(returnValue: true)] out string? reason)
-	{
-		if (response is not ElasticsearchResponse r)
-			return base.TryGetServerErrorReason(response, out reason);
-		reason = r.ElasticsearchServerError?.Error?.ToString();
-		return !string.IsNullOrEmpty(reason);
-	}
 }
diff --git a/src/Elastic.Clients.Elasticsearch/Elastic.Clients.Elasticsearch.csproj b/src/Elastic.Clients.Elasticsearch/Elastic.Clients.Elasticsearch.csproj
index aeab1d403da..9cf454fe07e 100644
--- a/src/Elastic.Clients.Elasticsearch/Elastic.Clients.Elasticsearch.csproj
+++ b/src/Elastic.Clients.Elasticsearch/Elastic.Clients.Elasticsearch.csproj
@@ -32,7 +32,7 @@
 
   <ItemGroup>
     <PackageReference Include="Elastic.Esql" Version="0.11.0" />
-    <PackageReference Include="Elastic.Transport" Version="0.16.0" />
+    <PackageReference Include="Elastic.Transport" Version="0.17.0" />
     <PackageReference Include="PolySharp" Version="1.15.0">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>