stripe 10.1.0 → 12.6.0

data/lib/stripe/resources/treasury/transaction.rb

@@ -8,6 +8,19 @@ module Stripe
       extend Stripe::APIOperations::List
 
       OBJECT_NAME = "treasury.transaction"
+      def self.object_name
+        "treasury.transaction"
+      end
+
+      # Retrieves a list of Transaction objects.
+      def self.list(filters = {}, opts = {})
+        request_stripe_object(
+          method: :get,
+          path: "/v1/treasury/transactions",
+          params: filters,
+          opts: opts
+        )
+      end
     end
   end
 end

data/lib/stripe/resources/treasury/transaction_entry.rb

@@ -8,6 +8,19 @@ module Stripe
       extend Stripe::APIOperations::List
 
       OBJECT_NAME = "treasury.transaction_entry"
+      def self.object_name
+        "treasury.transaction_entry"
+      end
+
+      # Retrieves a list of TransactionEntry objects.
+      def self.list(filters = {}, opts = {})
+        request_stripe_object(
+          method: :get,
+          path: "/v1/treasury/transaction_entries",
+          params: filters,
+          opts: opts
+        )
+      end
 
       def self.resource_url
         "/v1/treasury/transaction_entries"

data/lib/stripe/resources/usage_record.rb

@@ -6,7 +6,12 @@ module Stripe
   # metered billing of subscription prices.
   #
   # Related guide: [Metered billing](https://stripe.com/docs/billing/subscriptions/metered-billing)
+  #
+  # This is our legacy usage-based billing API. See the [updated usage-based billing docs](https://docs.stripe.com/billing/subscriptions/usage-based).
   class UsageRecord < APIResource
     OBJECT_NAME = "usage_record"
+    def self.object_name
+      "usage_record"
+    end
   end
 end

data/lib/stripe/resources/usage_record_summary.rb

@@ -4,5 +4,8 @@
 module Stripe
   class UsageRecordSummary < StripeObject
     OBJECT_NAME = "usage_record_summary"
+    def self.object_name
+      "usage_record_summary"
+    end
   end
 end

data/lib/stripe/resources/webhook_endpoint.rb

@@ -2,13 +2,13 @@
 # frozen_string_literal: true
 
 module Stripe
-  # You can configure [webhook endpoints](https://stripe.com/docs/webhooks/) via the API to be
+  # You can configure [webhook endpoints](https://docs.stripe.com/webhooks/) via the API to be
   # notified about events that happen in your Stripe account or connected
   # accounts.
   #
   # Most users configure webhooks from [the dashboard](https://dashboard.stripe.com/webhooks), which provides a user interface for registering and testing your webhook endpoints.
   #
-  # Related guide: [Setting up webhooks](https://stripe.com/docs/webhooks/configure)
+  # Related guide: [Setting up webhooks](https://docs.stripe.com/webhooks/configure)
   class WebhookEndpoint < APIResource
     extend Stripe::APIOperations::Create
     include Stripe::APIOperations::Delete
@@ -16,5 +16,58 @@ module Stripe
     include Stripe::APIOperations::Save
 
     OBJECT_NAME = "webhook_endpoint"
+    def self.object_name
+      "webhook_endpoint"
+    end
+
+    # A webhook endpoint must have a url and a list of enabled_events. You may optionally specify the Boolean connect parameter. If set to true, then a Connect webhook endpoint that notifies the specified url about events from all connected accounts is created; otherwise an account webhook endpoint that notifies the specified url only about events from your account is created. You can also create webhook endpoints in the [webhooks settings](https://dashboard.stripe.com/account/webhooks) section of the Dashboard.
+    def self.create(params = {}, opts = {})
+      request_stripe_object(
+        method: :post,
+        path: "/v1/webhook_endpoints",
+        params: params,
+        opts: opts
+      )
+    end
+
+    # You can also delete webhook endpoints via the [webhook endpoint management](https://dashboard.stripe.com/account/webhooks) page of the Stripe dashboard.
+    def self.delete(id, params = {}, opts = {})
+      request_stripe_object(
+        method: :delete,
+        path: format("/v1/webhook_endpoints/%<id>s", { id: CGI.escape(id) }),
+        params: params,
+        opts: opts
+      )
+    end
+
+    # You can also delete webhook endpoints via the [webhook endpoint management](https://dashboard.stripe.com/account/webhooks) page of the Stripe dashboard.
+    def delete(params = {}, opts = {})
+      request_stripe_object(
+        method: :delete,
+        path: format("/v1/webhook_endpoints/%<webhook_endpoint>s", { webhook_endpoint: CGI.escape(self["id"]) }),
+        params: params,
+        opts: opts
+      )
+    end
+
+    # Returns a list of your webhook endpoints.
+    def self.list(filters = {}, opts = {})
+      request_stripe_object(
+        method: :get,
+        path: "/v1/webhook_endpoints",
+        params: filters,
+        opts: opts
+      )
+    end
+
+    # Updates the webhook endpoint. You may edit the url, the list of enabled_events, and the status of your endpoint.
+    def self.update(id, params = {}, opts = {})
+      request_stripe_object(
+        method: :post,
+        path: format("/v1/webhook_endpoints/%<id>s", { id: CGI.escape(id) }),
+        params: params,
+        opts: opts
+      )
+    end
   end
 end

data/lib/stripe/resources.rb

@@ -11,6 +11,11 @@ require "stripe/resources/apps/secret"
 require "stripe/resources/balance"
 require "stripe/resources/balance_transaction"
 require "stripe/resources/bank_account"
+require "stripe/resources/billing/alert"
+require "stripe/resources/billing/meter"
+require "stripe/resources/billing/meter_event"
+require "stripe/resources/billing/meter_event_adjustment"
+require "stripe/resources/billing/meter_event_summary"
 require "stripe/resources/billing_portal/configuration"
 require "stripe/resources/billing_portal/session"
 require "stripe/resources/capability"
@@ -18,6 +23,10 @@ require "stripe/resources/card"
 require "stripe/resources/cash_balance"
 require "stripe/resources/charge"
 require "stripe/resources/checkout/session"
+require "stripe/resources/climate/order"
+require "stripe/resources/climate/product"
+require "stripe/resources/climate/supplier"
+require "stripe/resources/confirmation_token"
 require "stripe/resources/country_spec"
 require "stripe/resources/coupon"
 require "stripe/resources/credit_note"
@@ -25,8 +34,11 @@ require "stripe/resources/credit_note_line_item"
 require "stripe/resources/customer"
 require "stripe/resources/customer_balance_transaction"
 require "stripe/resources/customer_cash_balance_transaction"
+require "stripe/resources/customer_session"
 require "stripe/resources/discount"
 require "stripe/resources/dispute"
+require "stripe/resources/entitlements/active_entitlement"
+require "stripe/resources/entitlements/feature"
 require "stripe/resources/ephemeral_key"
 require "stripe/resources/event"
 require "stripe/resources/exchange_rate"
@@ -36,16 +48,21 @@ require "stripe/resources/financial_connections/account"
 require "stripe/resources/financial_connections/account_owner"
 require "stripe/resources/financial_connections/account_ownership"
 require "stripe/resources/financial_connections/session"
+require "stripe/resources/financial_connections/transaction"
+require "stripe/resources/forwarding/request"
 require "stripe/resources/funding_instructions"
 require "stripe/resources/identity/verification_report"
 require "stripe/resources/identity/verification_session"
 require "stripe/resources/invoice"
 require "stripe/resources/invoice_item"
 require "stripe/resources/invoice_line_item"
+require "stripe/resources/invoice_rendering_template"
 require "stripe/resources/issuing/authorization"
 require "stripe/resources/issuing/card"
 require "stripe/resources/issuing/cardholder"
 require "stripe/resources/issuing/dispute"
+require "stripe/resources/issuing/personalization_design"
+require "stripe/resources/issuing/physical_bundle"
 require "stripe/resources/issuing/token"
 require "stripe/resources/issuing/transaction"
 require "stripe/resources/line_item"
@@ -61,6 +78,7 @@ require "stripe/resources/person"
 require "stripe/resources/plan"
 require "stripe/resources/price"
 require "stripe/resources/product"
+require "stripe/resources/product_feature"
 require "stripe/resources/promotion_code"
 require "stripe/resources/quote"
 require "stripe/resources/radar/early_fraud_warning"

data/lib/stripe/search_result_object.rb

@@ -7,6 +7,9 @@ module Stripe
     include Stripe::APIOperations::Request
 
     OBJECT_NAME = "search_result"
+    def self.object_name
+      "search_result"
+    end
 
     # This accessor allows a `SearchResultObject` to inherit various filters
     # that were given to a predecessor. This allows for things like consistent
@@ -80,7 +83,7 @@ module Stripe
 
       params = filters.merge(page: next_page).merge(params)
 
-      _search(url, params, opts)
+      request_stripe_object(method: :get, path: url, params: params, opts: opts)
     end
   end
 end

data/lib/stripe/singleton_api_resource.rb

@@ -10,15 +10,32 @@ module Stripe
       end
       # Namespaces are separated in object names with periods (.) and in URLs
       # with forward slashes (/), so replace the former with the latter.
-      "/v1/#{self::OBJECT_NAME.downcase.tr('.', '/')}"
+      "/v1/#{object_name.downcase.tr('.', '/')}"
     end
 
     def resource_url
       self.class.resource_url
     end
 
-    def self.retrieve(opts = {})
-      instance = new(nil, Util.normalize_opts(opts))
+    def self.retrieve(params_or_opts = {}, definitely_opts = nil)
+      opts = nil
+      params = nil
+      if definitely_opts.nil?
+        unrecognized_key = params_or_opts.keys.find { |k| !Util::OPTS_USER_SPECIFIED.include?(k) }
+        if unrecognized_key
+          raise ArgumentError,
+                "Unrecognized request option: #{unrecognized_key}. Did you mean to specify this as retrieve params? " \
+                "If so, you must explicitly pass an opts hash as a second argument. " \
+                "For example: .retrieve({#{unrecognized_key}: 'foo'}, {})"
+        end
+
+        opts = params_or_opts
+      else
+        opts = definitely_opts
+        params = params_or_opts
+      end
+
+      instance = new(params, Util.normalize_opts(opts))
       instance.refresh
       instance
     end

data/lib/stripe/stripe_client.rb

@@ -18,6 +18,11 @@ module Stripe
       @system_profiler = SystemProfiler.new
       @last_request_metrics = nil
 
+      # The following attribute is only used to log whether or not
+      # StripeClient#request has been called. To be removed in a
+      # future major version.
+      @usage = []
+
       @config = case config_arg
                 when Hash
                   Stripe.config.reverse_duplicate_merge(config_arg)
@@ -36,8 +41,7 @@ module Stripe
                 end
     end
 
-    attr_reader :config
-    attr_reader :options
+    attr_reader :config, :options
 
     # Gets a currently active `StripeClient`. Set for the current thread when
     # `StripeClient#request` is being run so that API operations being executed
@@ -80,9 +84,7 @@ module Stripe
             end
           end
 
-          if thread_context.default_connection_managers.empty?
-            pruned_contexts << thread_context
-          end
+          pruned_contexts << thread_context if thread_context.default_connection_managers.empty?
         end
 
         @thread_contexts_with_connection_managers.subtract(pruned_contexts)
@@ -113,14 +115,14 @@ module Stripe
     # both socket errors that may represent an intermittent problem and some
     # special HTTP statuses.
     def self.should_retry?(error,
-                           method:, num_retries:, config: Stripe.config)
+                           num_retries:, config: Stripe.config)
       return false if num_retries >= config.max_network_retries
 
       case error
       when Net::OpenTimeout, Net::ReadTimeout
         # Retry on timeout-related problems (either on open or read).
         true
-      when EOFError, Errno::ECONNREFUSED, Errno::ECONNRESET,
+      when EOFError, Errno::ECONNREFUSED, Errno::ECONNRESET, # rubocop:todo Lint/DuplicateBranch
             Errno::EHOSTUNREACH, Errno::ETIMEDOUT, SocketError
         # Destination refused the connection, the connection was reset, or a
         # variety of other connection failures. This could occur from a single
@@ -146,15 +148,12 @@ module Stripe
         # These 429s are safe to retry.
         return true if error.http_status == 429 && error.code == "lock_timeout"
 
-        # 500 Internal Server Error
+        # Retry on 500, 503, and other internal errors.
         #
-        # We only bother retrying these for non-POST requests. POSTs end up
-        # being cached by the idempotency layer so there's no purpose in
-        # retrying them.
-        return true if error.http_status == 500 && method != :post
-
-        # 503 Service Unavailable
-        error.http_status == 503
+        # Note that we expect the stripe-should-retry header to be false
+        # in most cases when a 500 is returned, since our idempotency framework
+        # would typically replay it anyway.
+        true if error.http_status >= 500
       else
         false
       end
@@ -191,7 +190,9 @@ module Stripe
     #     client = StripeClient.new
     #     charge, resp = client.request { Charge.create }
     #
+
     def request
+      @usage = ["stripe_client_request"]
       old_stripe_client = self.class.current_thread_context.active_client
       self.class.current_thread_context.active_client = self
 
@@ -206,15 +207,19 @@ module Stripe
         res = yield
         [res, self.class.current_thread_context.last_responses[object_id]]
       ensure
+        @usage = []
         self.class.current_thread_context.active_client = old_stripe_client
         self.class.current_thread_context.last_responses.delete(object_id)
       end
     end
+    deprecate :request, "the `last_response` property on the returned resource (see " \
+                        "https://github.com/stripe/stripe-ruby?tab=readme-ov-file#accessing-a-response-object " \
+                        "for usage examples)", 2024, 6
 
     def execute_request(method, path,
-                        api_base: nil, api_key: nil, headers: {}, params: {})
+                        api_base: nil, api_key: nil, headers: {}, params: {}, usage: [])
       http_resp, api_key = execute_request_internal(
-        method, path, api_base, api_key, headers, params
+        method, path, api_base, api_key, headers, params, usage
       )
 
       begin
@@ -243,7 +248,7 @@ module Stripe
     # passed, then a StripeStreamResponse is returned containing an IO stream
     # with the response body.
     def execute_request_stream(method, path,
-                               api_base: nil, api_key: nil,
+                               api_base: nil, api_key: nil, usage: [],
                                headers: {}, params: {},
                                &read_body_chunk_block)
       unless block_given?
@@ -252,7 +257,7 @@ module Stripe
       end
 
       http_resp, api_key = execute_request_internal(
-        method, path, api_base, api_key, headers, params, &read_body_chunk_block
+        method, path, api_base, api_key, headers, params, usage, &read_body_chunk_block
       )
 
       # When the read_body_chunk_block is given, we no longer have access to the
@@ -287,21 +292,21 @@ module Stripe
 
     ERROR_MESSAGE_CONNECTION =
       "Unexpected error communicating when trying to connect to " \
-        "Stripe (%s). You may be seeing this message because your DNS is not " \
-        "working or you don't have an internet connection.  To check, try " \
-        "running `host stripe.com` from the command line."
+      "Stripe (%s). You may be seeing this message because your DNS is not " \
+      "working or you don't have an internet connection.  To check, try " \
+      "running `host stripe.com` from the command line."
     ERROR_MESSAGE_SSL =
       "Could not establish a secure connection to Stripe (%s), you " \
-        "may need to upgrade your OpenSSL version. To check, try running " \
-        "`openssl s_client -connect api.stripe.com:443` from the command " \
-        "line."
+      "may need to upgrade your OpenSSL version. To check, try running " \
+      "`openssl s_client -connect api.stripe.com:443` from the command " \
+      "line."
 
     # Common error suffix sared by both connect and read timeout messages.
     ERROR_MESSAGE_TIMEOUT_SUFFIX =
       "Please check your internet connection and try again. " \
-        "If this problem persists, you should check Stripe's service " \
-        "status at https://status.stripe.com, or let us know at " \
-        "support@stripe.com."
+      "If this problem persists, you should check Stripe's service " \
+      "status at https://status.stripe.com, or let us know at " \
+      "support@stripe.com."
 
     ERROR_MESSAGE_TIMEOUT_CONNECT = (
       "Timed out connecting to Stripe (%s). " +
@@ -361,7 +366,7 @@ module Stripe
       # garbage in `Thread.current`.
       attr_accessor :last_responses
 
-      # A map of connection mangers for the thread. Normally shared between
+      # A map of connection managers for the thread. Normally shared between
       # all `StripeClient` objects on a particular thread, and created so as to
       # minimize the number of open connections that an application needs.
       def default_connection_managers
@@ -431,7 +436,7 @@ module Stripe
     end
 
     private def execute_request_internal(method, path,
-                                         api_base, api_key, headers, params,
+                                         api_base, api_key, headers, params, usage,
                                          &read_body_chunk_block)
       raise ArgumentError, "method should be a symbol" \
       unless method.is_a?(Symbol)
@@ -457,6 +462,7 @@ module Stripe
 
       headers = request_headers(api_key, method)
                 .update(Util.normalize_headers(headers))
+
       url = api_url(path, api_base)
 
       # Merge given query parameters with any already encoded in the path.
@@ -488,14 +494,12 @@ module Stripe
       response_block =
         if block_given?
           lambda do |response|
-            unless should_handle_as_error(response.code.to_i)
-              response.read_body(&read_body_chunk_block)
-            end
+            response.read_body(&read_body_chunk_block) unless should_handle_as_error(response.code.to_i)
           end
         end
 
       http_resp =
-        execute_request_with_rescues(method, api_base, headers, context) do
+        execute_request_with_rescues(api_base, headers, usage, context) do
           self.class
               .default_connection_manager(config)
               .execute_request(method, url,
@@ -515,18 +519,18 @@ module Stripe
     private def check_api_key!(api_key)
       unless api_key
         raise AuthenticationError, "No API key provided. " \
-          'Set your API key using "Stripe.api_key = <API-KEY>". ' \
-          "You can generate API keys from the Stripe web interface. " \
-          "See https://stripe.com/api for details, or email " \
-          "support@stripe.com if you have any questions."
+                                   'Set your API key using "Stripe.api_key = <API-KEY>". ' \
+                                   "You can generate API keys from the Stripe web interface. " \
+                                   "See https://stripe.com/api for details, or email " \
+                                   "support@stripe.com if you have any questions."
       end
 
       return unless api_key =~ /\s/
 
       raise AuthenticationError, "Your API key is invalid, as it contains " \
-        "whitespace. (HINT: You can double-check your API key from the " \
-        "Stripe web interface. See https://stripe.com/api for details, or " \
-        "email support@stripe.com if you have any questions.)"
+                                 "whitespace. (HINT: You can double-check your API key from the " \
+                                 "Stripe web interface. See https://stripe.com/api for details, or " \
+                                 "email support@stripe.com if you have any questions.)"
     end
 
     # Encodes a set of body parameters using multipart if `Content-Type` is set
@@ -565,7 +569,7 @@ module Stripe
       http_status >= 400
     end
 
-    private def execute_request_with_rescues(method, api_base, headers, context)
+    private def execute_request_with_rescues(api_base, headers, usage, context)
       num_retries = 0
 
       begin
@@ -582,9 +586,7 @@ module Stripe
         http_status = resp.code.to_i
         context = context.dup_from_response_headers(resp)
 
-        if should_handle_as_error(http_status)
-          handle_error_response(resp, context)
-        end
+        handle_error_response(resp, context) if should_handle_as_error(http_status)
 
         log_response(context, request_start, http_status, resp.body, resp)
         notify_request_end(context, request_duration, http_status,
@@ -593,7 +595,7 @@ module Stripe
         if config.enable_telemetry? && context.request_id
           request_duration_ms = (request_duration * 1000).to_i
           @last_request_metrics =
-            StripeRequestMetrics.new(context.request_id, request_duration_ms)
+            StripeRequestMetrics.new(context.request_id, request_duration_ms, usage: usage + @usage)
         end
 
       # We rescue all exceptions from a request so that we have an easy spot to
@@ -618,7 +620,6 @@ module Stripe
                            user_data, resp, headers)
 
         if self.class.should_retry?(e,
-                                    method: method,
                                     num_retries: num_retries,
                                     config: config)
           num_retries += 1
@@ -634,7 +635,7 @@ module Stripe
 
         # Only handle errors when we know we can do so, and re-raise otherwise.
         # This should be pretty infrequent.
-        else
+        else # rubocop:todo Lint/DuplicateBranch
           raise
         end
       end
@@ -844,7 +845,7 @@ module Stripe
 
       if errors.nil?
         message = "Unexpected error #{error.class.name} communicating " \
-          "with Stripe. Please let us know at support@stripe.com."
+                  "with Stripe. Please let us know at support@stripe.com."
       end
 
       api_base ||= config.api_base
@@ -858,9 +859,7 @@ module Stripe
 
     private def request_headers(api_key, method)
       user_agent = "Stripe/v1 RubyBindings/#{Stripe::VERSION}"
-      unless Stripe.app_info.nil?
-        user_agent += " " + format_app_info(Stripe.app_info)
-      end
+      user_agent += " " + format_app_info(Stripe.app_info) unless Stripe.app_info.nil?
 
       headers = {
         "User-Agent" => user_agent,
@@ -881,6 +880,7 @@ module Stripe
       end
 
       headers["Stripe-Version"] = config.api_version if config.api_version
+
       headers["Stripe-Account"] = config.stripe_account if config.stripe_account
 
       user_agent = @system_profiler.user_agent
@@ -963,15 +963,7 @@ module Stripe
     # that we can log certain information. It's useful because it means that we
     # don't have to pass around as many parameters.
     class RequestLogContext
-      attr_accessor :body
-      attr_accessor :account
-      attr_accessor :api_key
-      attr_accessor :api_version
-      attr_accessor :idempotency_key
-      attr_accessor :method
-      attr_accessor :path
-      attr_accessor :query
-      attr_accessor :request_id
+      attr_accessor :body, :account, :api_key, :api_version, :idempotency_key, :method, :path, :query, :request_id
 
       # The idea with this method is that we might want to update some of
       # context information because a response that we've received from the API
@@ -1055,13 +1047,19 @@ module Stripe
       # Request duration in milliseconds
       attr_accessor :request_duration_ms
 
-      def initialize(request_id, request_duration_ms)
+      # list of names of tracked behaviors associated with this request
+      attr_accessor :usage
+
+      def initialize(request_id, request_duration_ms, usage: [])
         self.request_id = request_id
         self.request_duration_ms = request_duration_ms
+        self.usage = usage
       end
 
       def payload
-        { request_id: request_id, request_duration_ms: request_duration_ms }
+        ret = { request_id: request_id, request_duration_ms: request_duration_ms }
+        ret[:usage] = usage if !usage.nil? && !usage.empty?
+        ret
       end
     end
   end

data/lib/stripe/stripe_configuration.rb

@@ -25,26 +25,12 @@ module Stripe
   # If `.logger` is set, the value of `.log_level` is ignored. The decision on
   # what levels to print is entirely deferred to the logger.
   class StripeConfiguration
-    attr_accessor :api_key
-    attr_accessor :api_version
-    attr_accessor :client_id
-    attr_accessor :enable_telemetry
-    attr_accessor :logger
-    attr_accessor :stripe_account
-
-    attr_reader :api_base
-    attr_reader :uploads_base
-    attr_reader :connect_base
-    attr_reader :ca_bundle_path
-    attr_reader :log_level
-    attr_reader :initial_network_retry_delay
-    attr_reader :max_network_retries
-    attr_reader :max_network_retry_delay
-    attr_reader :open_timeout
-    attr_reader :read_timeout
-    attr_reader :write_timeout
-    attr_reader :proxy
-    attr_reader :verify_ssl_certs
+    attr_accessor :api_key, :api_version, :client_id, :enable_telemetry, :logger, :stripe_account
+
+    attr_reader :api_base, :uploads_base, :connect_base, :ca_bundle_path, :log_level, :initial_network_retry_delay,
+                # rubocop:todo Layout/LineLength
+                :max_network_retries, :max_network_retry_delay, :open_timeout, :read_timeout, :write_timeout, :proxy, :verify_ssl_certs
+    # rubocop:enable Layout/LineLength
 
     def self.setup
       new.tap do |instance|
@@ -95,8 +81,8 @@ module Stripe
 
       if !val.nil? && !levels.include?(val)
         raise ArgumentError,
-              "log_level should only be set to `nil`, `debug`, `info`," \
-              " or `error`"
+              "log_level should only be set to `nil`, `debug`, `info`, " \
+              "or `error`"
       end
       @log_level = val
     end
@@ -124,9 +110,7 @@ module Stripe
     end
 
     def write_timeout=(write_timeout)
-      unless Net::HTTP.instance_methods.include?(:write_timeout=)
-        raise NotImplementedError
-      end
+      raise NotImplementedError unless Net::HTTP.instance_methods.include?(:write_timeout=)
 
       @write_timeout = write_timeout
       StripeClient.clear_all_connection_managers(config: self)
@@ -177,10 +161,10 @@ module Stripe
     # and is itself not thread safe.
     def ca_store
       @ca_store ||= begin
-                      store = OpenSSL::X509::Store.new
-                      store.add_file(ca_bundle_path)
-                      store
-                    end
+        store = OpenSSL::X509::Store.new
+        store.add_file(ca_bundle_path)
+        store
+      end
     end
 
     def enable_telemetry?