You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@james.apache.org by bt...@apache.org on 2023/05/18 10:10:27 UTC

[james-project] branch master updated: [PERF] Avoid using Non-Preemptive authentication for OpenSearch (#1564)

This is an automated email from the ASF dual-hosted git repository.

btellier pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/james-project.git


The following commit(s) were added to refs/heads/master by this push:
     new 0264c0f6a6 [PERF] Avoid using Non-Preemptive authentication for OpenSearch (#1564)
0264c0f6a6 is described below

commit 0264c0f6a60c77858212083f695b56ced12cc984
Author: Trần Hồng Quân <55...@users.noreply.github.com>
AuthorDate: Thu May 18 17:10:21 2023 +0700

    [PERF] Avoid using Non-Preemptive authentication for OpenSearch (#1564)
    
    Before, we were using Non-Preemptive authentication for OpenSearch that makes 1 normal OpenSearch request to be 2 round trips (1 authenticated 401 request and 1 normal request).
    This likely negatively affected our search performance.
    We should avoid that.
    
    rf: https://www.elastic.co/guide/en/elasticsearch/client/java-api-client/master/_basic_authentication.html
---
 .../main/java/org/apache/james/backends/opensearch/ClientProvider.java   | 1 -
 1 file changed, 1 deletion(-)

diff --git a/backends-common/opensearch/src/main/java/org/apache/james/backends/opensearch/ClientProvider.java b/backends-common/opensearch/src/main/java/org/apache/james/backends/opensearch/ClientProvider.java
index 28b09fa383..d253fc6d6b 100644
--- a/backends-common/opensearch/src/main/java/org/apache/james/backends/opensearch/ClientProvider.java
+++ b/backends-common/opensearch/src/main/java/org/apache/james/backends/opensearch/ClientProvider.java
@@ -169,7 +169,6 @@ public class ClientProvider implements Provider<ReactorOpenSearchClient> {
         }
 
         private void configureAuthentication(HttpAsyncClientBuilder builder) {
-            builder.disableAuthCaching();
             configuration.getCredential()
                 .ifPresent(credential -> {
                     CredentialsProvider credentialsProvider = new BasicCredentialsProvider();


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org