elastic-transport 8.3.6 → 8.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 55e1343a320fd2a1dba29077ff4d9ad72b6adf62fe918856480f43e00e8a9927
-  data.tar.gz: da17629d377258be5b8564ba0bd82c3ef6119117e307a14ac60f8da14894e375
+  metadata.gz: 5634ceac3d52fa6af5174e9aad5ecf2613018e4a9977114305a9f3a42ee25909
+  data.tar.gz: 92acaa8ac6c08475e3dc91aa1132e1a719060b0bab0c05c94b3feae92f9cb83f
 SHA512:
-  metadata.gz: e375ab5339638b8fedd29d38ae0ff6384407e8256c9201c383ee084689e4781752fe4cc9b23830a2380185234acfbbc59533d91f653ec51fff9007a433b033b3
-  data.tar.gz: 25a3847030def992ed441260e8921509ad5375353ad3e564218e799341bc53942e30245e6dad79c6e38e1783b993298e6729951320bb785e17308a2f5f9ca19f
+  metadata.gz: 2cd0a8e98e8bf2cfa5964779042672f15ba4d968b0fadec444474dc2253584c9dc3cf5932ffb731415503a57719838e5aa05eec78e77e184fbc3eb762291c137
+  data.tar.gz: d7132ee663cd2984e5b34ea7827ff27010e69c66d6ad3dc2ca3571b6d4b516ab1004b7a45c7495e68e38a047defc66c947cd2c1a0a026086a3b718d6328dc1ed

data/.github/workflows/license.yml

@@ -7,7 +7,7 @@ jobs:
     - uses: actions/checkout@v4
     - uses: ruby/setup-ruby@v1
       with:
-        ruby-version: 3
+        ruby-version: '3.4'
     - name: Check license headers
       run: |
         ruby ./.github/check_license_headers.rb

data/.github/workflows/otel.yml

@@ -2,10 +2,10 @@ name: opentelemetry
 on:
   push:
     branches:
-      - 8.3
+      - main
   pull_request:
     branches:
-      - 8.3
+      - main
 jobs:
   test-otel:
     name: 'Test Open Telemetry'
@@ -16,7 +16,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        ruby: ['3.3', 'jruby-9.4']
+        ruby: ['3.4', 'jruby-9.4']
         es_version: ['9.2.0-SNAPSHOT']
     runs-on: ubuntu-latest
     steps:

data/.github/workflows/tests.yml

@@ -1,11 +1,12 @@
-name: 8.3 tests
+name: main tests
 on:
   push:
     branches:
-      - 8.3
+      - main
   pull_request:
     branches:
-      - 8.3
+      - main
+  workflow_dispatch:
 jobs:
   test:
     name: 'Main tests'
@@ -15,7 +16,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        ruby: ['3.1', '3.2', '3.3', '3.4', 'jruby-9.3', 'jruby-9.4']
+        ruby: ['3.2', '3.3', '3.4', 'jruby-9.3', 'jruby-9.4', 'jruby-10.0']
         es_version: ['8.18.8-SNAPSHOT', '8.19.5-SNAPSHOT', '9.0.8-SNAPSHOT', '9.1.5-SNAPSHOT', '9.2.0-SNAPSHOT']
     runs-on: ubuntu-latest
     steps:
@@ -53,7 +54,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        ruby: ['3.1', '3.2', '3.3', '3.4', 'jruby-9.3']
+        ruby: ['3.0', '3.1', '3.2', '3.3', 'jruby-9.3']
         es_version: ['8.19.5-SNAPSHOT']
     runs-on: ubuntu-latest
     steps:

data/CHANGELOG.md

@@ -1,8 +1,45 @@
-## 8.3.6
+## 8.4.1
 
 - Moves `require 'timeout'` to the classes that use it. Closed [#79](https://github.com/elastic/elastic-transport-ruby/issues/79).
 - Sanitize url on OTEL spans, thanks @steve21168. Pull Request: [#98](https://github.com/elastic/elastic-transport-ruby/pull/98)
 
+## 8.4.0
+
+- Adds support for **[Excon](https://github.com/excon/excon)** Faraday adapter. You can use the Excon adapter by setting it in when initializing an Elasticsearch client:
+
+```ruby
+require 'faraday/excon'
+
+Elasticsearch::Client.new(
+  host: host,
+  transport_options: transport_options,
+  adapter: :excon
+)
+```
+
+Excon may need some specific configuration for HTTPS/SSL. You may see certificate verification errors when trying to connect to Elastic. Excon has certificates bundled, but these can be customized:
+
+```ruby
+Excon.defaults[:ssl_ca_path] = path_to_certs
+ENV['SSL_CERT_DIR'] = path_to_certs
+Excon.defaults[:ssl_ca_file] = path_to_file
+ENV['SSL_CERT_FILE'] = path_to_file
+Excon.defaults[:ssl_verify_callback] = callback
+# (see OpenSSL::SSL::SSLContext#verify_callback)
+Excon.defaults[:ssl_verify_peer] = false # (less secure).
+```
+
+- Adds support for **[Async::HTTP::Faraday](https://github.com/socketry/async-http-faraday)**. You can use the adapter by setting it when initializing an Elasticsearch client:
+```ruby
+require 'async-http-faraday'
+
+Elasticsearch::Client.new(
+  host: host,
+  adapter: :async_http
+)
+```
+- Minor refactors, styling fixes.
+
 ## 8.3.5
 
 - Small updates in source code documentation.

data/CONTRIBUTING.md

@@ -43,6 +43,23 @@ $ rake docker:start[8.0.0-alpha1]
 
 You can find the available version in [Docker @ Elastic](https://www.docker.elastic.co/r/elasticsearch).
 
+You can also run Elasticsearch and Kibana locally with Docker with `start-local`:
+
+```bash
+curl -fsSL https://elastic.co/start-local | sh
+```
+
+This will run Elasticsearch in http://localhost:9200 and Kibana in http://localhost:5601. More information is available [here](https://www.elastic.co/guide/en/elasticsearch/reference/current/run-elasticsearch-locally.html).
+
+## Faraday 1 tests
+
+You can run the tests for Faraday v1 with the following commands:
+
+```bash
+BUNDLE_GEMFILE=Gemfile-faraday1.gemfile bundle install
+BUNDLE_GEMFILE=Gemfile-faraday1.gemfile rake test:all
+```
+
 # Contributing
 
 The process for contributing is:

data/Gemfile

@@ -21,13 +21,16 @@ source 'https://rubygems.org'
 gemspec
 
 group :development, :test do
+  gem 'faraday-excon'
   gem 'faraday-httpclient'
   gem 'faraday-net_http_persistent'
   gem 'faraday-typhoeus'
+  gem 'mutex_m' if RUBY_VERSION >= '3.4'
   gem 'opentelemetry-sdk', require: false if RUBY_VERSION >= '3.0'
   if defined?(JRUBY_VERSION)
     gem 'pry-nav'
   else
+    gem 'async-http-faraday'
     gem 'faraday-patron'
     gem 'oj'
     gem 'pry-byebug'

data/README.md

@@ -1,8 +1,7 @@
 # Elastic Transport
-[![8.1 tests](https://github.com/elastic/elastic-transport-ruby/actions/workflows/tests.yml/badge.svg?branch=8.1)](https://github.com/elastic/elastic-transport-ruby/actions/workflows/tests.yml)
 [![Run tests](https://github.com/elastic/elastic-transport-ruby/actions/workflows/tests.yml/badge.svg?branch=main)](https://github.com/elastic/elastic-transport-ruby/actions/workflows/tests.yml)
 
-This gem provides a low-level Ruby client for connecting to an [Elastic](http://elastic.co) cluster. It powers both the [Elasticsearch client](https://github.com/elasticsearch/elasticsearch-ruby/) and the [Elastic Enterprise Search](https://github.com/elastic/enterprise-search-ruby/) client.
+This gem provides a low-level Ruby client for connecting to an [Elastic](http://elastic.co) cluster. It powers the [Elasticsearch client](https://github.com/elasticsearch/elasticsearch-ruby/) and other Elastic projects.
 
 In the simplest form, connect to Elasticsearch running on `http://localhost:9200` without any configuration:
 
@@ -14,9 +13,9 @@ response = client.perform_request('GET', '_cluster/health')
 # => #<Elastic::Transport::Transport::Response:0x007fc5d506ce38 @status=200, @body={ ... } >
 ```
 
-**Refer to [the official documentation on Elastic Transport](https://www.elastic.co/guide/en/elasticsearch/client/ruby-api/current/transport.html).**
+**Refer to [the official documentation on Elastic Transport](https://www.elastic.co/docs/reference/elasticsearch/clients/ruby/transport).**
 
-**Refer to [Advanced Configuration](https://www.elastic.co/guide/en/elasticsearch/client/ruby-api/current/advanced-config.html) to read about more configuration options.**
+**Refer to [Advanced Configuration](https://www.elastic.co/docs/reference/elasticsearch/clients/ruby/advanced-config) to read about more configuration options.**
 
 ## Compatibility
 

data/lib/elastic/transport/client.rb

@@ -123,8 +123,8 @@ module Elastic
       #
       def initialize(arguments = {}, &block)
         @arguments = arguments.transform_keys(&:to_sym)
-        @arguments[:logger] ||= @arguments[:log]   ? DEFAULT_LOGGER.call() : nil
-        @arguments[:tracer] ||= @arguments[:trace] ? DEFAULT_TRACER.call() : nil
+        @arguments[:logger] ||= @arguments[:log]   ? DEFAULT_LOGGER.call : nil
+        @arguments[:tracer] ||= @arguments[:trace] ? DEFAULT_TRACER.call : nil
         @arguments[:reload_connections] ||= false
         @arguments[:retry_on_failure]   ||= false
         @arguments[:delay_on_retry]     ||= 0
@@ -134,13 +134,7 @@ module Elastic
         @arguments[:http]               ||= {}
         @arguments[:enable_meta_header] = arguments.fetch(:enable_meta_header, true)
 
-        @hosts ||= __extract_hosts(@arguments[:hosts] ||
-                                   @arguments[:host] ||
-                                   @arguments[:url] ||
-                                   @arguments[:urls] ||
-                                   ENV['ELASTICSEARCH_URL'] ||
-                                   DEFAULT_HOST)
-
+        @hosts ||= extract_hosts
         @send_get_body_as = @arguments[:send_get_body_as] || 'GET'
         @ca_fingerprint = @arguments.delete(:ca_fingerprint)
 
@@ -153,7 +147,7 @@ module Elastic
         else
           @transport_class = @arguments[:transport_class] || DEFAULT_TRANSPORT_CLASS
           @transport = if @transport_class == Transport::HTTP::Faraday
-                         @arguments[:adapter] ||= __auto_detect_adapter
+                         @arguments[:adapter] ||= auto_detect_adapter
                          set_meta_header # from include MetaHeader
                          @transport_class.new(hosts: @hosts, options: @arguments) do |faraday|
                            faraday.adapter(@arguments[:adapter])
@@ -246,7 +240,7 @@ module Elastic
       #
       # @api private
       #
-      def __extract_hosts(hosts_config)
+      def extract_hosts
         hosts = case hosts_config
                 when String
                   hosts_config.split(',').map { |h| h.strip! || h }
@@ -257,12 +251,20 @@ module Elastic
                 else
                   Array(hosts_config)
                 end
-
-        host_list = hosts.map { |host| __parse_host(host) }
+        host_list = hosts.map { |host| parse_host(host) }
         @arguments[:randomize_hosts] ? host_list.shuffle! : host_list
       end
 
-      def __parse_host(host)
+      def hosts_config
+        @arguments[:hosts] ||
+          @arguments[:host] ||
+          @arguments[:url] ||
+          @arguments[:urls] ||
+          ENV['ELASTICSEARCH_URL'] ||
+          DEFAULT_HOST
+      end
+
+      def parse_host(host)
         host_parts = case host
                      when String
                        if host =~ /^[a-z]+\:\/\//
@@ -313,12 +315,14 @@ module Elastic
       #
       # @api private
       #
-      def __auto_detect_adapter
+      def auto_detect_adapter
         if Gem::Version.new(Faraday::VERSION) >= Gem::Version.new(2)
           return :patron if defined?(Faraday::Adapter::Patron)
           return :typhoeus if defined?(Faraday::Adapter::Typhoeus)
           return :httpclient if defined?(Faraday::Adapter::HTTPClient)
           return :net_http_persistent if defined?(Faraday::Adapter::NetHttpPersistent)
+          return :excon if defined?(Faraday::Adapter::Excon)
+          return :async_http if defined?(Async::HTTP::Faraday)
         else
           return :patron if defined?(::Patron)
           return :typhoeus if defined?(::Typhoeus)

data/lib/elastic/transport/opentelemetry.rb

@@ -21,70 +21,77 @@ module Elastic
     #
     # @api private
     class OpenTelemetry
-      OTEL_TRACER_NAME = 'elasticsearch-api'
+      OTEL_TRACER_NAME = 'elasticsearch-api'.freeze
       # Valid values for the enabled config are 'true' and 'false'. Default is 'true'.
-      ENV_VARIABLE_ENABLED = 'OTEL_RUBY_INSTRUMENTATION_ELASTICSEARCH_ENABLED'
+      ENV_VARIABLE_ENABLED = 'OTEL_RUBY_INSTRUMENTATION_ELASTICSEARCH_ENABLED'.freeze
       # Describes how to handle search queries in the request body when assigned to
       # a span attribute.
       # Valid values are 'raw', 'omit', 'sanitize'. Default is 'omit'.
-      ENV_VARIABLE_BODY_STRATEGY = 'OTEL_RUBY_INSTRUMENTATION_ELASTICSEARCH_CAPTURE_SEARCH_QUERY'
-      ENV_VARIABLE_DEPRECATED_BODY_STRATEGY = 'OTEL_INSTRUMENTATION_ELASTICSEARCH_CAPTURE_SEARCH_QUERY'
-      DEFAULT_BODY_STRATEGY = 'omit'
+      ENV_VARIABLE_BODY_STRATEGY = 'OTEL_RUBY_INSTRUMENTATION_ELASTICSEARCH_CAPTURE_SEARCH_QUERY'.freeze
+      ENV_VARIABLE_DEPRECATED_BODY_STRATEGY = 'OTEL_INSTRUMENTATION_ELASTICSEARCH_CAPTURE_SEARCH_QUERY'.freeze
+      DEFAULT_BODY_STRATEGY = 'omit'.freeze
       # A string list of keys whose values are redacted. This is only relevant if the body strategy is
       # 'sanitize'. For example, a config 'sensitive-key,other-key' will redact the values at
       # 'sensitive-key' and 'other-key' in addition to the default keys.
-      ENV_VARIABLE_BODY_SANITIZE_KEYS = 'OTEL_RUBY_INSTRUMENTATION_ELASTICSEARCH_SEARCH_QUERY_SANITIZE_KEYS'
+      ENV_VARIABLE_BODY_SANITIZE_KEYS = 'OTEL_RUBY_INSTRUMENTATION_ELASTICSEARCH_SEARCH_QUERY_SANITIZE_KEYS'.freeze
 
       # A list of the Elasticsearch endpoints that qualify as "search" endpoints. The search query in
       # the request body may be captured for these endpoints, depending on the body capture strategy.
       SEARCH_ENDPOINTS = Set[
-        "search",
-        "async_search.submit",
-        "msearch",
-        "eql.search",
-        "terms_enum",
-        "search_template",
-        "msearch_template",
-        "render_search_template",
+        'search',
+        'async_search.submit',
+        'msearch',
+        'eql.search',
+        'terms_enum',
+        'search_template',
+        'msearch_template',
+        'render_search_template',
       ]
 
       # Initialize the Open Telemetry wrapper object. Takes the options originally passed to
       # Client#initialize.
       def initialize(opts)
-        @tracer = (opts[:opentelemetry_tracer_provider] || ::OpenTelemetry.tracer_provider).tracer(
-          OTEL_TRACER_NAME, Elastic::Transport::VERSION
-        )
-        @body_strategy = ENV[ENV_VARIABLE_DEPRECATED_BODY_STRATEGY] || ENV[ENV_VARIABLE_BODY_STRATEGY] ||
-                           DEFAULT_BODY_STRATEGY
+        @tracer = tracer_provider(opts).tracer(OTEL_TRACER_NAME, Elastic::Transport::VERSION)
+        @body_strategy = ENV[ENV_VARIABLE_DEPRECATED_BODY_STRATEGY] ||
+                         ENV[ENV_VARIABLE_BODY_STRATEGY] ||
+                         DEFAULT_BODY_STRATEGY
         @sanitize_keys = ENV[ENV_VARIABLE_BODY_SANITIZE_KEYS]&.split(',')&.collect! do |pattern|
           Regexp.new(pattern.gsub('*', '.*'))
         end
       end
       attr_accessor :tracer
 
+      def tracer_provider(opts)
+        opts[:opentelemetry_tracer_provider] || ::OpenTelemetry.tracer_provider
+      end
+
       # Process the request body. Applies the body strategy, which can be one of the following:
-      # 'omit': return nil
+      # 'omit' (DEFAULT_BODY_STRATEGY): return nil
       # 'sanitize': redact values at the default list of keys + any additional keys provided in
       # the OTEL_RUBY_INSTRUMENTATION_ELASTICSEARCH_SEARCH_QUERY_SANITIZE_KEYS env variable.
       # 'raw': return the original body, unchanged
       def process_body(body, endpoint)
-        unless @body_strategy == 'omit' || !SEARCH_ENDPOINTS.include?(endpoint)
-          if @body_strategy == 'sanitize'
-            Sanitizer.sanitize(body, @sanitize_keys).to_json
-          elsif @body_strategy == 'raw'
-            body&.is_a?(String) ? body : body.to_json
-          end
+        return if @body_strategy == DEFAULT_BODY_STRATEGY || !search_endpoint?(endpoint)
+
+        if @body_strategy == 'sanitize'
+          Sanitizer.sanitize(body, @sanitize_keys).to_json
+        elsif @body_strategy == 'raw'
+          body.is_a?(String) ? body : body.to_json
         end
       end
 
+      def search_endpoint?(endpoint)
+        SEARCH_ENDPOINTS.include?(endpoint)
+      end
+
       # Replaces values in a hash with 'REDACTED', given a set of keys to match on.
       class Sanitizer
         class << self
-          FILTERED = 'REDACTED'
+          FILTERED = 'REDACTED'.freeze
           DEFAULT_KEY_PATTERNS =
             %w[password passwd pwd secret *key *token* *session* *credit* *card* *auth* set-cookie].map! do |p|
-              Regexp.new(p.gsub('*', '.*'))
-            end
+            Regexp.new(p.gsub('*', '.*'))
+          end
 
           def sanitize(body, key_patterns = [])
             patterns = DEFAULT_KEY_PATTERNS

data/lib/elastic/transport/transport/base.rb

@@ -83,7 +83,7 @@ module Elastic
         # @return [Connections::Connection]
         # @see    Connections::Collection#get_connection
         #
-        def get_connection(options={})
+        def get_connection(options = {})
           resurrect_dead_connections! if Time.now > @last_request_at + @resurrect_after
 
           @counter_mtx.synchronize { @counter += 1 }
@@ -120,7 +120,7 @@ module Elastic
         # @return [Connections::Collection]
         # @api private
         #
-        def __rebuild_connections(arguments={})
+        def __rebuild_connections(arguments = {})
           @state_mutex.synchronize do
             @hosts       = arguments[:hosts]    || []
             @options     = arguments[:options]  || {}
@@ -232,7 +232,7 @@ module Elastic
         #
         # @api private
         #
-        def __convert_to_json(o=nil, options={})
+        def __convert_to_json(o = nil, options = {})
           o.is_a?(String) ? o : serializer.dump(o, options)
         end
 
@@ -379,11 +379,11 @@ module Elastic
         private
 
         USER_AGENT_STR = 'User-Agent'.freeze
-        USER_AGENT_REGEX = /user-?_?agent/
+        USER_AGENT_REGEX = /user-?_?agent/.freeze
         ACCEPT_ENCODING = 'Accept-Encoding'.freeze
         CONTENT_ENCODING = 'Content-Encoding'.freeze
         CONTENT_TYPE_STR = 'Content-Type'.freeze
-        CONTENT_TYPE_REGEX = /content-?_?type/
+        CONTENT_TYPE_REGEX = /content-?_?type/.freeze
         DEFAULT_CONTENT_TYPE = 'application/json'.freeze
         GZIP = 'gzip'.freeze
         GZIP_FIRST_TWO_BYTES = '1f8b'.freeze

data/lib/elastic/transport/transport/connections/connection.rb

@@ -155,7 +155,6 @@ module Elastic
             "<#{self.class.name} host: #{host} (#{dead? ? 'dead since ' + dead_since.to_s : 'alive'})>"
           end
         end
-
       end
     end
   end

data/lib/elastic/transport/transport/http/curb.rb

@@ -30,7 +30,7 @@ module Elastic
           # @return [Response]
           # @see    Transport::Base#perform_request
           #
-          def perform_request(method, path, params={}, body=nil, headers=nil, opts={})
+          def perform_request(method, path, params = {}, body = nil, headers = nil, opts = {})
             super do |connection, url|
               capture_otel_span_attributes(connection, url)
               connection.connection.url = connection.full_url(path, params)
@@ -51,7 +51,6 @@ module Elastic
                     connection.connection.headers = headers
                   end
                 end
-
               else raise ArgumentError, "Unsupported HTTP method: #{method}"
               end
 
@@ -73,7 +72,7 @@ module Elastic
           #
           # @return [Connections::Connection]
           #
-          def __build_connection(host, options={}, block=nil)
+          def __build_connection(host, options = {}, block = nil)
             client = ::Curl::Easy.new
 
             apply_headers(client, options)
@@ -87,7 +86,7 @@ module Elastic
 
             client.instance_eval(&block) if block
 
-            Connections::Connection.new :host => host, :connection => client
+            Connections::Connection.new(host: host, connection: client)
           end
 
           # Returns an array of implementation specific connection errors.

data/lib/elastic/transport/version.rb

@@ -17,6 +17,6 @@
 
 module Elastic
   module Transport
-    VERSION = '8.3.6'.freeze
+    VERSION = '8.4.1'.freeze
   end
 end

data/spec/elastic/transport/client_spec.rb

@@ -225,6 +225,38 @@ describe Elastic::Transport::Client do
       end
     end unless jruby?
 
+    context 'when the adapter is excon' do
+      let(:adapter) do
+        client.transport.connections.all.first.connection.builder.adapter
+      end
+
+      let(:client) do
+        require 'faraday/excon'
+
+        described_class.new(adapter: :excon, enable_meta_header: false)
+      end
+
+      it 'uses Faraday with the adapter' do
+        expect(adapter).to eq Faraday::Adapter::Excon
+      end
+    end if is_faraday_v2?
+
+    context 'when the adapter is async-http' do
+      let(:adapter) do
+        client.transport.connections.all.first.connection.builder.adapter
+      end
+
+      let(:client) do
+        require 'async/http/faraday'
+
+        described_class.new(adapter: :async_http, enable_meta_header: false)
+      end
+
+      it 'uses Faraday with the adapter' do
+        expect(adapter).to eq Async::HTTP::Faraday::Adapter
+      end
+    end unless jruby? || !is_faraday_v2?
+
     context 'when the adapter is specified as a string key' do
       let(:adapter) do
         client.transport.connections.all.first.connection.builder.adapter
@@ -1453,6 +1485,7 @@ describe Elastic::Transport::Client do
 
           context 'when using the HTTPClient adapter' do
             require 'faraday/httpclient'
+            require 'mutex_m' if RUBY_VERSION >= '3.4'
 
             let(:client) do
               described_class.new(hosts: ELASTICSEARCH_HOSTS, compression: true, adapter: :httpclient, enable_meta_header: false)

data/spec/elastic/transport/opentelemetry_spec.rb

@@ -25,8 +25,8 @@ if defined?(::OpenTelemetry)
     let(:span) { exporter.finished_spans[0] }
 
     let(:client) do
-      Elastic::Transport::Client.new(hosts: ELASTICSEARCH_HOSTS).tap do |_client|
-        allow(_client).to receive(:__build_connections)
+      Elastic::Transport::Client.new(hosts: ELASTICSEARCH_HOSTS).tap do |client|
+        allow(client).to receive(:__build_connections)
       end
     end
 
@@ -36,7 +36,8 @@ if defined?(::OpenTelemetry)
       let(:tracer_provider) do
         double('tracer_provider').tap do |tp|
           expect(tp).to receive(:tracer).with(
-            Elastic::Transport::OpenTelemetry::OTEL_TRACER_NAME, Elastic::Transport::VERSION
+            Elastic::Transport::OpenTelemetry::OTEL_TRACER_NAME,
+            Elastic::Transport::VERSION
           )
         end
       end
@@ -59,7 +60,7 @@ if defined?(::OpenTelemetry)
       it 'creates a span with path parameters' do
         client.perform_request(
           'POST', '/users/_create/abc', nil, { name: 'otel-test' }, nil,
-          defined_params: {'index' => 'users', 'id' => 'abc'}, endpoint: 'create'
+          defined_params: { 'index' => 'users', 'id' => 'abc' }, endpoint: 'create'
         )
 
         span = exporter.finished_spans.find { |s| s.name == 'create' }
@@ -96,7 +97,7 @@ if defined?(::OpenTelemetry)
 
     context 'when a request is instrumented' do
       let(:body) do
-        { query: { match: { password: { query: 'secret'} } } }
+        { query: { match: { password: { query: 'secret' } } } }
       end
 
       it 'creates a span and omits db.statement' do
@@ -214,9 +215,9 @@ if defined?(::OpenTelemetry)
 
         around(:example) do |ex|
           body_strategy = ENV[described_class::ENV_VARIABLE_BODY_STRATEGY]
-          ENV[described_class::ENV_VARIABLE_BODY_STRATEGY]  = 'omit'
+          ENV[described_class::ENV_VARIABLE_BODY_STRATEGY] = 'omit'
           ex.run
-          ENV[described_class::ENV_VARIABLE_BODY_STRATEGY] = body_strategy
+          ENV[described_class::ENV_VARIABLE_BODY_STRATEGY]= body_strategy
         end
 
         it 'does not include anything' do
@@ -308,11 +309,11 @@ if defined?(::OpenTelemetry)
           let(:key_patterns) { [/something/] }
 
           let(:body) do
-            { query: { match: { something: "test" } } }
+            { query: { match: { something: 'test' } } }
           end
 
           let(:expected_body) do
-            { query: { match: { something: "REDACTED" } } }
+            { query: { match: { something: 'REDACTED' } } }
           end
 
           it 'redacts sensitive values' do

data/test/unit/adapters_test.rb

@@ -72,6 +72,22 @@ class Elastic::Transport::ClientAdaptersUnitTest < Minitest::Test
       end
     end
 
+    should 'use Excon Faraday adapter' do
+      fork do
+        require 'faraday/excon'
+        client = Elastic::Transport::Client.new
+        assert_equal(client.transport.connections.first.connection.adapter, Faraday::Adapter::Excon)
+      end
+    end if is_faraday_v2?
+
+    should 'use Async HTTP Faraday adapter' do
+      fork do
+        require 'async/http/faraday'
+        client = Elastic::Transport::Client.new
+        assert_equal(client.transport.connections.first.connection.adapter, Async::HTTP::Faraday::Adapter)
+      end
+    end if is_faraday_v2?
+
     should 'use HTTPClient Faraday adapter' do
       fork do
         if is_faraday_v2?
@@ -81,7 +97,7 @@ class Elastic::Transport::ClientAdaptersUnitTest < Minitest::Test
         end
 
         client = Elastic::Transport::Client.new
-        assert_equal(Faraday::Adapter::HTTPClient, client.transport.connections.first.connection.adapter)
+        assert_equal(client.transport.connections.first.connection.adapter, Faraday::Adapter::HTTPClient)
       end
     end
   end unless jruby?

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: elastic-transport
 version: !ruby/object:Gem::Version
-  version: 8.3.6
+  version: 8.4.1
 platform: ruby
 authors:
 - Elastic Client Library Maintainers