castle-rb 7.2.0 → 8.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5ada9f9e25a4da59d7577cb10848d45697a6709648846011fc803167a8b96203
-  data.tar.gz: 2b0927e4ae6f686a35c9c5ac39e510f46f8fc06506932441b421a845ba296f27
+  metadata.gz: e9dc06eac5803e36c8576d9938442ba1c1f6bb852420c7a736044127deea50b8
+  data.tar.gz: 5d89738c670359a0345d51b41d0137d614622002711f20665238a4c289f28bab
 SHA512:
-  metadata.gz: 5ff2d32ab3ae269adbcd5380ba7488e06c0d9ae0ab463cd73856eb65b67bb4559daa1118344bc1a6896b097f00f9e5cc35c80ce68add430785769c2502e261b0
-  data.tar.gz: e48b7c5332de7d74ad8d13a89f52b2fd756d1c2b77600af86b86d74c78789916ac2972983781708be097d9d6f4067705b59ba26eaca2dc3e45720eee4ced6f08
+  metadata.gz: 118236690b20da223208e043cb4aafea5c1d9870e0d2e44d87106a45122652ca894f8bfbc79bf465e474923a026b9b2253a6c7a1ab2c2f5157b0b21d51459d6c
+  data.tar.gz: 223933e1de62111635ba5d22fed064541a1d54aee9c9374fc8524a808b27b623245ee0d5240a42d0eff4ac72e214cc4f61ce44bb72f15591d3944790d8bc3e0d

data/lib/castle/api/authenticate.rb

@@ -7,26 +7,17 @@ module Castle
         # @param options [Hash]
         # return [Hash]
         def call(options = {})
-          unless options[:no_symbolize]
-            options = Castle::Utils::DeepSymbolizeKeys.call(options || {})
-          end
+          options = Castle::Utils::DeepSymbolizeKeys.call(options || {}) unless options[:no_symbolize]
           options.delete(:no_symbolize)
           http = options.delete(:http)
           config = options.delete(:config) || Castle.config
 
-          response =
-            Castle::API.call(Castle::Commands::Authenticate.build(options), {}, http, config)
+          response = Castle::API.call(Castle::Commands::Authenticate.build(options), {}, http, config)
           response.merge(failover: false, failover_reason: nil)
         rescue Castle::RequestError, Castle::InternalServerError => e
           unless config.failover_strategy == :throw
             strategy = (config || Castle.config).failover_strategy
-            return(
-              Castle::Failover::PrepareResponse.new(
-                options[:user_id],
-                reason: e.to_s,
-                strategy: strategy
-              ).call
-            )
+            return(Castle::Failover::PrepareResponse.new(options[:user_id], reason: e.to_s, strategy: strategy).call)
           end
 
           raise e

data/lib/castle/api/end_impersonation.rb

@@ -7,9 +7,7 @@ module Castle
       class << self
         # @param options [Hash]
         def call(options = {})
-          unless options[:no_symbolize]
-            options = Castle::Utils::DeepSymbolizeKeys.call(options || {})
-          end
+          options = Castle::Utils::DeepSymbolizeKeys.call(options || {}) unless options[:no_symbolize]
           options.delete(:no_symbolize)
           http = options.delete(:http)
           config = options.delete(:config) || Castle.config

data/lib/castle/api/filter.rb

@@ -8,9 +8,7 @@ module Castle
         # @param options [Hash]
         # return [Hash]
         def call(options = {})
-          unless options[:no_symbolize]
-            options = Castle::Utils::DeepSymbolizeKeys.call(options || {})
-          end
+          options = Castle::Utils::DeepSymbolizeKeys.call(options || {}) unless options[:no_symbolize]
           options.delete(:no_symbolize)
           http = options.delete(:http)
           config = options.delete(:config) || Castle.config
@@ -20,13 +18,7 @@ module Castle
         rescue Castle::RequestError, Castle::InternalServerError => e
           unless config.failover_strategy == :throw
             strategy = (config || Castle.config).failover_strategy
-            return(
-              Castle::Failover::PrepareResponse.new(
-                options[:user][:id],
-                reason: e.to_s,
-                strategy: strategy
-              ).call
-            )
+            return(Castle::Failover::PrepareResponse.new(options[:user][:id], reason: e.to_s, strategy: strategy).call)
           end
 
           raise e

data/lib/castle/api/log.rb

@@ -8,9 +8,7 @@ module Castle
         # @param options [Hash]
         # return [Hash]
         def call(options = {})
-          unless options[:no_symbolize]
-            options = Castle::Utils::DeepSymbolizeKeys.call(options || {})
-          end
+          options = Castle::Utils::DeepSymbolizeKeys.call(options || {}) unless options[:no_symbolize]
           options.delete(:no_symbolize)
           http = options.delete(:http)
           config = options.delete(:config) || Castle.config
@@ -20,13 +18,7 @@ module Castle
         rescue Castle::RequestError, Castle::InternalServerError => e
           unless config.failover_strategy == :throw
             strategy = (config || Castle.config).failover_strategy
-            return(
-              Castle::Failover::PrepareResponse.new(
-                options[:user][:id],
-                reason: e.to_s,
-                strategy: strategy
-              ).call
-            )
+            return(Castle::Failover::PrepareResponse.new(options[:user][:id], reason: e.to_s, strategy: strategy).call)
           end
 
           raise e

data/lib/castle/api/risk.rb

@@ -8,9 +8,7 @@ module Castle
         # @param options [Hash]
         # return [Hash]
         def call(options = {})
-          unless options[:no_symbolize]
-            options = Castle::Utils::DeepSymbolizeKeys.call(options || {})
-          end
+          options = Castle::Utils::DeepSymbolizeKeys.call(options || {}) unless options[:no_symbolize]
           options.delete(:no_symbolize)
           http = options.delete(:http)
           config = options.delete(:config) || Castle.config
@@ -20,13 +18,7 @@ module Castle
         rescue Castle::RequestError, Castle::InternalServerError => e
           unless config.failover_strategy == :throw
             strategy = (config || Castle.config).failover_strategy
-            return(
-              Castle::Failover::PrepareResponse.new(
-                options[:user][:id],
-                reason: e.to_s,
-                strategy: strategy
-              ).call
-            )
+            return(Castle::Failover::PrepareResponse.new(options[:user][:id], reason: e.to_s, strategy: strategy).call)
           end
 
           raise e

data/lib/castle/api/start_impersonation.rb

@@ -7,9 +7,7 @@ module Castle
       class << self
         # @param options [Hash]
         def call(options = {})
-          unless options[:no_symbolize]
-            options = Castle::Utils::DeepSymbolizeKeys.call(options || {})
-          end
+          options = Castle::Utils::DeepSymbolizeKeys.call(options || {}) unless options[:no_symbolize]
           options.delete(:no_symbolize)
           http = options.delete(:http)
           config = options.delete(:config) || Castle.config

data/lib/castle/api/track.rb

@@ -6,9 +6,7 @@ module Castle
       class << self
         # @param options [Hash]
         def call(options = {})
-          unless options[:no_symbolize]
-            options = Castle::Utils::DeepSymbolizeKeys.call(options || {})
-          end
+          options = Castle::Utils::DeepSymbolizeKeys.call(options || {}) unless options[:no_symbolize]
           options.delete(:no_symbolize)
           http = options.delete(:http)
           config = options.delete(:config) || Castle.config

data/lib/castle/client.rb

@@ -123,11 +123,7 @@ module Castle
 
     # @param user_id [String, Boolean]
     def generate_do_not_track_response(user_id)
-      Castle::Failover::PrepareResponse.new(
-        user_id,
-        strategy: :allow,
-        reason: 'Castle is set to do not track.'
-      ).call
+      Castle::Failover::PrepareResponse.new(user_id, strategy: :allow, reason: 'Castle is set to do not track.').call
     end
 
     # @param options [Hash]

data/lib/castle/commands/log.rb

@@ -10,11 +10,7 @@ module Castle
         def build(options = {})
           context = Castle::Context::Sanitize.call(options[:context])
 
-          Castle::Command.new(
-            'log',
-            options.merge(context: context, sent_at: Castle::Utils::GetTimestamp.call),
-            :post
-          )
+          Castle::Command.new('log', options.merge(context: context, sent_at: Castle::Utils::GetTimestamp.call), :post)
         end
       end
     end

data/lib/castle/commands/risk.rb

@@ -10,11 +10,7 @@ module Castle
         def build(options = {})
           context = Castle::Context::Sanitize.call(options[:context])
 
-          Castle::Command.new(
-            'risk',
-            options.merge(context: context, sent_at: Castle::Utils::GetTimestamp.call),
-            :post
-          )
+          Castle::Command.new('risk', options.merge(context: context, sent_at: Castle::Utils::GetTimestamp.call), :post)
         end
       end
     end

data/lib/castle/configuration.rb

@@ -104,9 +104,7 @@ module Castle
     # sets trusted proxies
     # @param value [Array<String,Regexp>]
     def trusted_proxies=(value)
-      unless value.is_a?(Array)
-        raise Castle::ConfigurationError, 'trusted proxies must be an Array'
-      end
+      raise Castle::ConfigurationError, 'trusted proxies must be an Array' unless value.is_a?(Array)
 
       @trusted_proxies = value
     end
@@ -121,8 +119,7 @@ module Castle
     end
 
     def failover_strategy=(value)
-      @failover_strategy =
-        Castle::Failover::STRATEGIES.detect { |strategy| strategy == value.to_sym }
+      @failover_strategy = Castle::Failover::STRATEGIES.detect { |strategy| strategy == value.to_sym }
       raise Castle::ConfigurationError, 'unrecognized failover strategy' if @failover_strategy.nil?
     end
 

data/lib/castle/core/process_response.rb

@@ -9,7 +9,8 @@ module Castle
         401 => Castle::UnauthorizedError,
         403 => Castle::ForbiddenError,
         404 => Castle::NotFoundError,
-        419 => Castle::UserUnauthorizedError
+        419 => Castle::UserUnauthorizedError,
+        429 => Castle::RateLimitError
       }.freeze
 
       INVALID_REQUEST_TOKEN = 'invalid_request_token'
@@ -51,9 +52,9 @@ module Castle
               if parsed_body.is_a?(Hash) && parsed_body.key?(:type)
                 if parsed_body[:type] == INVALID_REQUEST_TOKEN
                   raise Castle::InvalidRequestTokenError, parsed_body[:message]
-                else
-                  raise Castle::InvalidParametersError, parsed_body[:message]
                 end
+
+                raise Castle::InvalidParametersError, parsed_body[:message]
               end
             rescue JSON::ParserError
             end

data/lib/castle/core/send_request.rb

@@ -15,9 +15,7 @@ module Castle
         # @param http [Net::HTTP]
         # @param config [Castle::Configuration, Castle::SingletonConfiguration, nil]
         def call(command, headers, http = nil, config = nil)
-          (http || Castle::Core::GetConnection.call).request(
-            build(command, headers.merge(DEFAULT_HEADERS), config)
-          )
+          (http || Castle::Core::GetConnection.call).request(build(command, headers.merge(DEFAULT_HEADERS), config))
         end
 
         # @param command [String]

data/lib/castle/errors.rb

@@ -61,6 +61,10 @@ module Castle
   class UnauthorizedError < Castle::ApiError
   end
 
+  # api error too many requests 429
+  class RateLimitError < Castle::ApiError
+  end
+
   # all internal server errors
   class InternalServerError < Castle::ApiError
   end

data/lib/castle/headers/extract.rb

@@ -23,9 +23,7 @@ module Castle
       # Serialize HTTP headers
       # @return [Hash]
       def call
-        @headers.each_with_object({}) do |(name, value), acc|
-          acc[name] = header_value(name, value)
-        end
+        @headers.each_with_object({}) { |(name, value), acc| acc[name] = header_value(name, value) }
       end
 
       private

data/lib/castle/headers/filter.rb

@@ -12,8 +12,7 @@ module Castle
       HTTP(?:_|-).*|
         CONTENT(?:_|-)LENGTH|
       REMOTE(?:_|-)ADDR
-      $/xi
-        .freeze
+      $/xi.freeze
 
       private_constant :VALUABLE_HEADERS
 
@@ -32,7 +31,7 @@ module Castle
             next unless header_name.match(VALUABLE_HEADERS)
 
             formatted_name = @header_format.call(header_name)
-            acc[formatted_name] = @request_env[header_name]
+            acc[formatted_name] = @request_env[header_name].to_s
           end
       end
     end

data/lib/castle/payload/prepare.rb

@@ -12,8 +12,7 @@ module Castle
         def call(payload_options, request, options = {})
           context = Castle::Context::Prepare.call(request, payload_options.merge(options))
 
-          payload =
-            Castle::Utils::DeepSymbolizeKeys.call(payload_options || {}).merge(context: context)
+          payload = Castle::Utils::DeepSymbolizeKeys.call(payload_options || {}).merge(context: context)
           payload[:timestamp] ||= Castle::Utils::GetTimestamp.call
 
           warn '[DEPRECATION] use user_traits instead of traits key' if payload.key?(:traits)

data/lib/castle/session.rb

@@ -9,7 +9,7 @@ module Castle
 
     class << self
       def call(&block)
-        return unless block_given?
+        return unless block
 
         Castle::Core::GetConnection.call.start(&block)
       end

data/lib/castle/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Castle
-  VERSION = '7.2.0'
+  VERSION = '8.0.0'
 end

data/lib/castle.rb

@@ -67,9 +67,7 @@
 module Castle
   class << self
     def configure(config_hash = nil)
-      (config_hash || {}).each do |config_name, config_value|
-        config.send("#{config_name}=", config_value)
-      end
+      (config_hash || {}).each { |config_name, config_value| config.send(:"#{config_name}=", config_value) }
 
       yield(config) if block_given?
     end

data/spec/integration/rails/rails_spec.rb

@@ -24,11 +24,12 @@ RSpec.describe HomeController, type: :request do
             'Accept' =>
               'text/xml,application/xml,application/xhtml+xml,text/html;q=0.9,text/plain;q=0.8,image/png,*/*;q=0.5',
             'Authorization' => true,
-            'Content-Length' => '0',
             'Cookie' => true,
+            'Content-Length' => '0',
             'Host' => 'www.example.com',
             'X-Forwarded-For' => '5.5.5.5, 1.2.3.4',
-            'Remote-Addr' => '127.0.0.1'
+            'Remote-Addr' => '127.0.0.1',
+            'Version' => 'HTTP/1.0'
           },
           'ip' => '1.2.3.4',
           'library' => {
@@ -40,7 +41,12 @@ RSpec.describe HomeController, type: :request do
     end
     let(:now) { Time.now }
     let(:headers) do
-      { 'HTTP_AUTHORIZATION' => 'Basic 123', 'HTTP_X_FORWARDED_FOR' => '5.5.5.5, 1.2.3.4' }
+      {
+        'HTTP_AUTHORIZATION' => 'Basic 123',
+        'HTTP_X_FORWARDED_FOR' => '5.5.5.5, 1.2.3.4',
+        'HTTP_VERSION' => 'HTTP/1.0',
+        'HTTP_CONTENT_LENGTH' => '0'
+      }
     end
 
     before do

data/spec/integration/rails/support/application.rb

@@ -3,8 +3,8 @@
 require 'action_controller/railtie'
 
 class TestApp < Rails::Application
-  secrets.secret_token = 'secret_token'
-  secrets.secret_key_base = 'secret_key_base'
+  credentials.secret_token = 'secret_token'
+  credentials.secret_key_base = 'secret_key_base'
 
   config.logger = Logger.new($stdout)
   Rails.logger = config.logger

data/spec/integration/rails/support/home_controller.rb

@@ -4,16 +4,7 @@ class HomeController < ActionController::Base
   # prepare context and calling track with client example
   def index1
     request_context = ::Castle::Context::Prepare.call(request)
-    payload = {
-      event: '$login.succeeded',
-      user_id: '123',
-      properties: {
-        key: 'value'
-      },
-      user_traits: {
-        key: 'value'
-      }
-    }
+    payload = { event: '$login.succeeded', user_id: '123', properties: { key: 'value' }, user_traits: { key: 'value' } }
     client = ::Castle::Client.new(context: request_context)
     client.track(payload)
 
@@ -24,16 +15,7 @@ class HomeController < ActionController::Base
   def index2
     payload =
       ::Castle::Payload::Prepare.call(
-        {
-          event: '$login.succeeded',
-          user_id: '123',
-          properties: {
-            key: 'value'
-          },
-          user_traits: {
-            key: 'value'
-          }
-        },
+        { event: '$login.succeeded', user_id: '123', properties: { key: 'value' }, user_traits: { key: 'value' } },
         request
       )
     client = ::Castle::Client.new
@@ -46,18 +28,10 @@ class HomeController < ActionController::Base
   def index3
     payload =
       ::Castle::Payload::Prepare.call(
-        {
-          event: '$login.succeeded',
-          user_id: '123',
-          properties: {
-            key: 'value'
-          },
-          user_traits: {
-            key: 'value'
-          }
-        },
+        { event: '$login.succeeded', user_id: '123', properties: { key: 'value' }, user_traits: { key: 'value' } },
         request
       )
+
     Castle::API::Track.call(payload)
 
     render inline: 'hello'

data/spec/lib/castle/api/approve_device_spec.rb

@@ -2,9 +2,7 @@
 
 describe Castle::API::ApproveDevice do
   before do
-    stub_request(:any, /api.castle.io/)
-      .with(basic_auth: ['', 'secret'])
-      .to_return(status: 200, body: '{}', headers: {})
+    stub_request(:any, /api.castle.io/).with(basic_auth: ['', 'secret']).to_return(status: 200, body: '{}', headers: {})
   end
 
   describe '.call' do
@@ -14,8 +12,6 @@ describe Castle::API::ApproveDevice do
 
     before { retrieve }
 
-    it do
-      assert_requested :put, "https://api.castle.io/v1/devices/#{device_token}/approve", times: 1
-    end
+    it { assert_requested :put, "https://api.castle.io/v1/devices/#{device_token}/approve", times: 1 }
   end
 end

data/spec/lib/castle/api/authenticate_spec.rb

@@ -29,15 +29,16 @@ describe Castle::API::Authenticate do
   after { Timecop.return }
 
   describe '.call' do
-    let(:request_body) do
-      { event: '$login.succeeded', context: context, user_id: '1234', sent_at: time_auto }
-    end
+    let(:request_body) { { event: '$login.succeeded', context: context, user_id: '1234', sent_at: time_auto } }
 
     context 'when used with symbol keys' do
       before do
-        stub_request(:any, /api.castle.io/)
-          .with(basic_auth: ['', 'secret'])
-          .to_return(status: 200, body: response_body, headers: {})
+        stub_request(:any, /api.castle.io/).with(basic_auth: ['', 'secret']).to_return(
+          status: 200,
+          body: response_body,
+          headers: {
+          }
+        )
         call_subject
       end
 
@@ -50,17 +51,9 @@ describe Castle::API::Authenticate do
       end
 
       context 'when passed timestamp in options and no defined timestamp' do
-        let(:options) do
-          { event: '$login.succeeded', user_id: '1234', timestamp: time_user, context: context }
-        end
+        let(:options) { { event: '$login.succeeded', user_id: '1234', timestamp: time_user, context: context } }
         let(:request_body) do
-          {
-            event: '$login.succeeded',
-            user_id: '1234',
-            context: context,
-            timestamp: time_user,
-            sent_at: time_auto
-          }
+          { event: '$login.succeeded', user_id: '1234', context: context, timestamp: time_user, sent_at: time_auto }
         end
 
         it do
@@ -78,15 +71,16 @@ describe Castle::API::Authenticate do
 
       context 'when denied without any risk policy' do
         let(:response_body) { deny_response_without_rp.to_json }
-        let(:deny_response_without_rp) do
-          { action: 'deny', user_id: '12345', device_token: 'abcdefg1234' }
-        end
+        let(:deny_response_without_rp) { { action: 'deny', user_id: '12345', device_token: 'abcdefg1234' } }
         let(:deny_without_rp_failover_result) { deny_response_without_rp.merge(failover_appendix) }
 
         before do
-          stub_request(:any, /api.castle.io/)
-            .with(basic_auth: ['', 'secret'])
-            .to_return(status: 200, body: deny_response_without_rp.to_json, headers: {})
+          stub_request(:any, /api.castle.io/).with(basic_auth: ['', 'secret']).to_return(
+            status: 200,
+            body: deny_response_without_rp.to_json,
+            headers: {
+            }
+          )
           call_subject
         end
 
@@ -117,9 +111,12 @@ describe Castle::API::Authenticate do
         let(:deny_with_rp_failover_result) { deny_response_with_rp.merge(failover_appendix) }
 
         before do
-          stub_request(:any, /api.castle.io/)
-            .with(basic_auth: ['', 'secret'])
-            .to_return(status: 200, body: deny_response_with_rp.to_json, headers: {})
+          stub_request(:any, /api.castle.io/).with(basic_auth: ['', 'secret']).to_return(
+            status: 200,
+            body: deny_response_with_rp.to_json,
+            headers: {
+            }
+          )
           call_subject
         end
 

data/spec/lib/castle/api/end_impersonation_spec.rb

@@ -22,9 +22,12 @@ describe Castle::API::EndImpersonation do
   before do
     Timecop.freeze(time_now)
     stub_const('Castle::VERSION', '2.2.0')
-    stub_request(:any, /api.castle.io/)
-      .with(basic_auth: ['', 'secret'])
-      .to_return(status: 200, body: response_body, headers: {})
+    stub_request(:any, /api.castle.io/).with(basic_auth: ['', 'secret']).to_return(
+      status: 200,
+      body: response_body,
+      headers: {
+      }
+    )
   end
 
   after { Timecop.return }
@@ -32,19 +35,10 @@ describe Castle::API::EndImpersonation do
   describe 'call' do
     let(:impersonator) { 'test@castle.io' }
     let(:request_body) do
-      {
-        user_id: '1234',
-        sent_at: time_auto,
-        properties: {
-          impersonator: impersonator
-        },
-        context: context
-      }
+      { user_id: '1234', sent_at: time_auto, properties: { impersonator: impersonator }, context: context }
     end
     let(:response_body) { { success: true }.to_json }
-    let(:options) do
-      { user_id: '1234', properties: { impersonator: impersonator }, context: context }
-    end
+    let(:options) { { user_id: '1234', properties: { impersonator: impersonator }, context: context } }
 
     context 'when used with symbol keys' do
       before { call }

data/spec/lib/castle/api/get_device_spec.rb

@@ -2,9 +2,7 @@
 
 describe Castle::API::GetDevice do
   before do
-    stub_request(:any, /api.castle.io/)
-      .with(basic_auth: ['', 'secret'])
-      .to_return(status: 200, body: '{}', headers: {})
+    stub_request(:any, /api.castle.io/).with(basic_auth: ['', 'secret']).to_return(status: 200, body: '{}', headers: {})
   end
 
   describe '.call' do

data/spec/lib/castle/api/get_devices_for_user_spec.rb

@@ -2,9 +2,7 @@
 
 describe Castle::API::GetDevicesForUser do
   before do
-    stub_request(:any, /api.castle.io/)
-      .with(basic_auth: ['', 'secret'])
-      .to_return(status: 200, body: '{}', headers: {})
+    stub_request(:any, /api.castle.io/).with(basic_auth: ['', 'secret']).to_return(status: 200, body: '{}', headers: {})
   end
 
   describe '.call' do

data/spec/lib/castle/api/report_device_spec.rb

@@ -2,9 +2,7 @@
 
 describe Castle::API::ReportDevice do
   before do
-    stub_request(:any, /api.castle.io/)
-      .with(basic_auth: ['', 'secret'])
-      .to_return(status: 200, body: '{}', headers: {})
+    stub_request(:any, /api.castle.io/).with(basic_auth: ['', 'secret']).to_return(status: 200, body: '{}', headers: {})
   end
 
   describe '.call' do
@@ -14,8 +12,6 @@ describe Castle::API::ReportDevice do
 
     before { retrieve }
 
-    it do
-      assert_requested :put, "https://api.castle.io/v1/devices/#{device_token}/report", times: 1
-    end
+    it { assert_requested :put, "https://api.castle.io/v1/devices/#{device_token}/report", times: 1 }
   end
 end