From 3d17be173b925d775ece84246c91ae31c1818737 Mon Sep 17 00:00:00 2001
From: Jonathan Mendez <jonnathan.mendez@gmail.com>
Date: Thu, 16 Jul 2026 12:39:29 -0600
Subject: [PATCH] Make searchClient a separate bean so Studio context can
 override to avoid duplicating instances

---
 .../engine/services/main-services-context.xml | 27 ++++++++++---------
 1 file changed, 14 insertions(+), 13 deletions(-)

diff --git a/src/main/resources/crafter/engine/services/main-services-context.xml b/src/main/resources/crafter/engine/services/main-services-context.xml
index c71a0114..25b409fb 100644
--- a/src/main/resources/crafter/engine/services/main-services-context.xml
+++ b/src/main/resources/crafter/engine/services/main-services-context.xml
@@ -720,20 +720,21 @@
     <alias name="crafter.searchService" alias="crafter.elasticsearchService"/>
     <alias name="crafter.searchClient" alias="crafter.elasticsearchClient"/>
 
+
+	<bean id="searchClient" class="org.craftercms.search.opensearch.spring.OpenSearchClientFactory">
+		<constructor-arg name="serverUrls" ref="crafter.searchUrls"/>
+		<property name="username" ref="crafter.searchUsername"/>
+		<property name="password" ref="crafter.searchPassword"/>
+		<property name="connectTimeout" ref="crafter.searchConnectTimeout"/>
+		<property name="socketTimeout" ref="crafter.searchSocketTimeout"/>
+		<property name="threadCount" ref="crafter.searchThreadCount"/>
+		<property name="socketKeepAlive" ref="crafter.searchSocketKeepAlive"/>
+		<property name="maxTotalConnections" value="${crafter.engine.search.maxTotalConnections}"/>
+		<property name="maxConnectionsPerRoute" value="${crafter.engine.search.maxConnectionsPerRoute}"/>
+	</bean>
+
     <bean id="crafter.searchClient" class="org.craftercms.engine.search.SiteAwareOpenSearchClient">
-        <constructor-arg name="client">
-            <bean class="org.craftercms.search.opensearch.spring.OpenSearchClientFactory">
-                <constructor-arg name="serverUrls" ref="crafter.searchUrls"/>
-                <property name="username" ref="crafter.searchUsername"/>
-                <property name="password" ref="crafter.searchPassword"/>
-                <property name="connectTimeout" ref="crafter.searchConnectTimeout"/>
-                <property name="socketTimeout" ref="crafter.searchSocketTimeout"/>
-                <property name="threadCount" ref="crafter.searchThreadCount"/>
-                <property name="socketKeepAlive" ref="crafter.searchSocketKeepAlive"/>
-                <property name="maxTotalConnections" value="${crafter.engine.search.maxTotalConnections}"/>
-                <property name="maxConnectionsPerRoute" value="${crafter.engine.search.maxConnectionsPerRoute}"/>
-            </bean>
-        </constructor-arg>
+        <constructor-arg name="client" ref="searchClient"/>
         <constructor-arg name="indexIdFormat" value="${crafter.engine.search.index.format}"/>
         <constructor-arg name="enableTranslation" value="${crafter.engine.translation.enable}"/>
         <constructor-arg name="enableDefaultFilters" value="${crafter.engine.search.defaultFilters.enabled}"/>