castle-rb 4.0.0 → 4.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 40234604d033086c79d951cbec5a52902df4ccda433eff1749c20cfeaca114d6
-  data.tar.gz: e46b4d91acbcf6c370de3a0804d3f9e75ad86bc181b40e2727e92d5d63579953
+  metadata.gz: d0d6aa4996eb091692c0a7dfcf9658b2b26fb011cfa94103604a79e60d401ac0
+  data.tar.gz: 8e7eb4c21cf3edd7a3845fc946857dd25ad1031d75dfa8cf8cb8b911b928eebe
 SHA512:
-  metadata.gz: d70f1b03fb44e3d23afcb03ca83165f5b521a53269f34d2ed63740cc181e110da3692db50d6f701477a24430262d08859a71f7faeae8867c8962242d7421545d
-  data.tar.gz: b4a32a603c55df97450e59f936677d5b56387b8d4bf145d4355ca7f5ec606c557d00da72a92bb3f2b641d93a3d3f6e21b9f9412fff35f1840c43db66789b3b6c
+  metadata.gz: 9bdf75588225fd282a36d1e37c1ee3eb69ecfab3f3e50a2c27d695f8c7c7ad1a2bccdb8b097dbe9ff361c17481c9449808d9c26adf56924eb249a18ffb2e9a71
+  data.tar.gz: '089b2507cd099a863b9592c8a194f5971f2126a7ff2892d0f211ace1a9b06261872a9a13d98a72b617f59815d70d87a4c49ca9e348c06031554a3a8cbe905140'

data/lib/castle.rb

@@ -29,15 +29,15 @@
   castle/configuration
   castle/failover_auth_response
   castle/client
-  castle/header_filter
-  castle/header_formatter
+  castle/headers_filter
+  castle/headers_formatter
   castle/secure_mode
   castle/extractors/client_id
   castle/extractors/headers
   castle/extractors/ip
   castle/api/response
   castle/api/request
-  castle/api/request/build
+  castle/api/session
   castle/review
   castle/api
 ].each(&method(:require))
@@ -54,7 +54,7 @@ module Castle
     end
 
     def config
-      @config ||= Castle::Configuration.new
+      Configuration.instance
     end
 
     def api_secret=(api_secret)

data/lib/castle/api.rb

@@ -17,16 +17,16 @@ module Castle
     private_constant :HANDLED_ERRORS
 
     class << self
+      # @param command [String]
+      # @param headers [Hash]
       def request(command, headers = {})
         raise Castle::ConfigurationError, 'configuration is not valid' unless Castle.config.valid?
 
         begin
-          Castle::API::Response.call(
-            Castle::API::Request.call(
-              command,
-              Castle.config.api_secret,
-              headers
-            )
+          Castle::API::Request.call(
+            command,
+            Castle.config.api_secret,
+            headers
           )
         rescue *HANDLED_ERRORS => e
           # @note We need to initialize the error, as the original error is a cause for this
@@ -35,6 +35,12 @@ module Castle
           raise Castle::RequestError.new(e) # rubocop:disable Style/RaiseArgs
         end
       end
+
+      # @param command [String]
+      # @param headers [Hash]
+      def call(command, headers = {})
+        Castle::API::Response.call(request(command, headers))
+      end
     end
   end
 end

data/lib/castle/api/request.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
 module Castle
-  # this class is responsible for making requests to api
   module API
+    # this class is responsible for making requests to api
     module Request
       # Default headers that we add to passed ones
       DEFAULT_HEADERS = {
@@ -13,8 +13,8 @@ module Castle
 
       class << self
         def call(command, api_secret, headers)
-          http.request(
-            Castle::API::Request::Build.call(
+          Castle::API::Session.get.request(
+            build(
               command,
               headers.merge(DEFAULT_HEADERS),
               api_secret
@@ -22,14 +22,19 @@ module Castle
           )
         end
 
-        def http
-          http = Net::HTTP.new(Castle.config.host, Castle.config.port)
-          http.read_timeout = Castle.config.request_timeout / 1000.0
-          if Castle.config.port == 443
-            http.use_ssl = true
-            http.verify_mode = OpenSSL::SSL::VERIFY_PEER
+        def build(command, headers, api_secret)
+          request_obj = Net::HTTP.const_get(
+            command.method.to_s.capitalize
+          ).new("#{Castle.config.url_prefix}/#{command.path}", headers)
+
+          unless command.method == :get
+            request_obj.body = ::Castle::Utils.replace_invalid_characters(
+              command.data
+            ).to_json
           end
-          http
+
+          request_obj.basic_auth('', api_secret)
+          request_obj
         end
       end
     end

data/lib/castle/api/session.rb

@@ -0,0 +1,39 @@
+# frozen_string_literal: true
+
+require 'singleton'
+
+module Castle
+  module API
+    # this class keeps http config object
+    # and provides start/finish methods for persistent connection usage
+    # when there is a need of sending multiple requests at once
+    class Session
+      include Singleton
+
+      attr_accessor :http
+
+      def initialize
+        reset
+      end
+
+      def reset
+        @http = Net::HTTP.new(Castle.config.host, Castle.config.port)
+        @http.read_timeout = Castle.config.request_timeout / 1000.0
+
+        if Castle.config.port == 443
+          @http.use_ssl = true
+          @http.verify_mode = OpenSSL::SSL::VERIFY_PEER
+        end
+
+        @http
+      end
+
+      class << self
+        # @return [Net::HTTP]
+        def get
+          instance.http
+        end
+      end
+    end
+  end
+end

data/lib/castle/client.rb

@@ -44,7 +44,7 @@ module Castle
       add_timestamp_if_necessary(options)
       command = Castle::Commands::Authenticate.new(@context).build(options)
       begin
-        Castle::API.request(command).merge(failover: false, failover_reason: nil)
+        Castle::API.call(command).merge(failover: false, failover_reason: nil)
       rescue Castle::RequestError, Castle::InternalServerError => e
         self.class.failover_response_or_raise(
           FailoverAuthResponse.new(options[:user_id], reason: e.to_s), e
@@ -60,7 +60,7 @@ module Castle
       add_timestamp_if_necessary(options)
 
       command = Castle::Commands::Identify.new(@context).build(options)
-      Castle::API.request(command)
+      Castle::API.call(command)
     end
 
     def track(options = {})
@@ -71,14 +71,14 @@ module Castle
       add_timestamp_if_necessary(options)
 
       command = Castle::Commands::Track.new(@context).build(options)
-      Castle::API.request(command)
+      Castle::API.call(command)
     end
 
     def impersonate(options = {})
       options = Castle::Utils.deep_symbolize_keys(options || {})
       add_timestamp_if_necessary(options)
       command = Castle::Commands::Impersonate.new(@context).build(options)
-      Castle::API.request(command).tap do |response|
+      Castle::API.call(command).tap do |response|
         raise Castle::ImpersonationFailed unless response[:success]
       end
     end

data/lib/castle/configuration.rb

@@ -3,9 +3,11 @@
 module Castle
   # manages configuration variables
   class Configuration
+    include Singleton
+
     HOST = 'api.castle.io'
     PORT = 443
-    URL_PREFIX = 'v1'
+    URL_PREFIX = '/v1'
     FAILOVER_STRATEGY = :allow
     REQUEST_TIMEOUT = 500 # in milliseconds
     FAILOVER_STRATEGIES = %i[allow deny challenge throw].freeze
@@ -44,8 +46,12 @@ module Castle
     attr_reader :api_secret, :whitelisted, :blacklisted, :failover_strategy, :ip_headers, :trusted_proxies
 
     def initialize
-      @formatter = Castle::HeaderFormatter.new
+      @formatter = Castle::HeadersFormatter
       @request_timeout = REQUEST_TIMEOUT
+      reset
+    end
+
+    def reset
       self.failover_strategy = FAILOVER_STRATEGY
       self.host = HOST
       self.port = PORT

data/lib/castle/context/default.rb

@@ -4,7 +4,7 @@ module Castle
   module Context
     class Default
       def initialize(request, cookies = nil)
-        @pre_headers = HeaderFilter.new(request).call
+        @pre_headers = HeadersFilter.new(request).call
         @cookies = cookies || request.cookies
         @request = request
       end

data/lib/castle/extractors/ip.rb

@@ -45,7 +45,13 @@ module Castle
       # @param ips [Array<String>]
       # @return [Array<String>]
       def remove_proxies(ips)
-        ips.reject { |ip| @proxies.any? { |proxy| proxy.match(ip) } }
+        ips.reject { |ip| proxy?(ip) }
+      end
+
+      # @param ip [String]
+      # @return [Boolean]
+      def proxy?(ip)
+        @proxies.any? { |proxy| proxy.match(ip) }
       end
 
       # @param header [String]

data/lib/castle/{header_filter.rb → headers_filter.rb}

@@ -2,23 +2,23 @@
 
 module Castle
   # used for preparing valuable headers list
-  class HeaderFilter
+  class HeadersFilter
     # headers filter
     # HTTP_ - this is how Rack prefixes incoming HTTP headers
     # CONTENT_LENGTH - for responses without Content-Length or Transfer-Encoding header
     # REMOTE_ADDR - ip address header returned by web server
-    VALUABLE_HEADERS = /^(
-      HTTP_.*|
-      CONTENT_LENGTH|
-      REMOTE_ADDR
-    )$/x.freeze
+    VALUABLE_HEADERS = /^
+      HTTP(?:_|-).*|
+      CONTENT(?:_|-)LENGTH|
+      REMOTE(?:_|-)ADDR
+    $/xi.freeze
 
     private_constant :VALUABLE_HEADERS
 
     # @param request [Rack::Request]
     def initialize(request)
       @request_env = request.env
-      @formatter = HeaderFormatter.new
+      @formatter = HeadersFormatter
     end
 
     # Serialize HTTP headers

data/lib/castle/headers_formatter.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+module Castle
+  # formats header name
+  class HeadersFormatter
+    class << self
+      # @param header [String]
+      # @return [String]
+      def call(header)
+        format(header.to_s.gsub(/^HTTP(?:_|-)/i, ''))
+      end
+
+      private
+
+      # @param header [String]
+      # @return [String]
+      def format(header)
+        header.split(/_|-/).map(&:capitalize).join('-')
+      end
+    end
+  end
+end

data/lib/castle/version.rb

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

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

@@ -4,57 +4,69 @@ describe Castle::API::Request do
   describe '#call' do
     subject(:call) { described_class.call(command, api_secret, headers) }
 
+    let(:session) { instance_double('Castle::API::Session') }
     let(:http) { instance_double('Net::HTTP') }
-    let(:request_build) { instance_double('Castle::API::Request::Build') }
     let(:command) { Castle::Commands::Track.new({}).build(event: '$login.succeeded') }
     let(:headers) { {} }
     let(:api_secret) { 'secret' }
+    let(:request_build) { {} }
     let(:expected_headers) { { 'Content-Type' => 'application/json' } }
 
     before do
-      allow(described_class).to receive(:http).and_return(http)
-      allow(http).to receive(:request).with(request_build)
-      allow(Castle::API::Request::Build).to receive(:call)
-        .with(command, expected_headers, api_secret)
-        .and_return(request_build)
+      allow(Castle::API::Session).to receive(:instance).and_return(session)
+      allow(session).to receive(:http).and_return(http)
+      allow(http).to receive(:request)
+      allow(described_class).to receive(:build).and_return(request_build)
       call
     end
 
     it do
-      expect(Castle::API::Request::Build).to have_received(:call)
-        .with(command, expected_headers, api_secret)
+      expect(described_class).to have_received(:build).with(command, expected_headers, api_secret)
     end
 
     it { expect(http).to have_received(:request).with(request_build) }
   end
 
-  describe '#http' do
-    subject(:http) { described_class.http }
+  describe '#build' do
+    subject(:build) { described_class.build(command, headers, api_secret) }
 
-    context 'when ssl false' do
-      before do
-        Castle.config.host = 'localhost'
-        Castle.config.port = 3002
-      end
+    let(:headers) { { 'SAMPLE-HEADER' => '1' } }
+    let(:api_secret) { 'secret' }
 
-      after do
-        Castle.config.host = Castle::Configuration::HOST
-        Castle.config.port = Castle::Configuration::PORT
-      end
+    context 'when get' do
+      let(:command) { Castle::Commands::Review.build(review_id) }
+      let(:review_id) { SecureRandom.uuid }
 
-      it { expect(http).to be_instance_of(Net::HTTP) }
-      it { expect(http.address).to eq(Castle.config.host) }
-      it { expect(http.port).to eq(Castle.config.port) }
-      it { expect(http.use_ssl?).to be false }
-      it { expect(http.verify_mode).to be_nil }
+      it { expect(build.body).to be_nil }
+      it { expect(build.method).to eql('GET') }
+      it { expect(build.path).to eql("/v1/#{command.path}") }
+      it { expect(build.to_hash).to have_key('authorization') }
+      it { expect(build.to_hash).to have_key('sample-header') }
+      it { expect(build.to_hash['sample-header']).to eql(['1']) }
     end
 
-    context 'when ssl true' do
-      it { expect(http).to be_instance_of(Net::HTTP) }
-      it { expect(http.address).to eq(Castle.config.host) }
-      it { expect(http.port).to eq(Castle.config.port) }
-      it { expect(http.use_ssl?).to be true }
-      it { expect(http.verify_mode).to eq(OpenSSL::SSL::VERIFY_PEER) }
+    context 'when post' do
+      let(:time) { Time.now.utc.iso8601(3) }
+      let(:command) do
+        Castle::Commands::Track.new({}).build(event: '$login.succeeded', name: "\xC4")
+      end
+      let(:expected_body) do
+        {
+          event: '$login.succeeded',
+          name: '�',
+          context: {},
+          sent_at: time
+        }
+      end
+
+      before { allow(Castle::Utils::Timestamp).to receive(:call).and_return(time) }
+
+      it { expect(build.body).to be_eql(expected_body.to_json) }
+      it { expect(build.method).to eql('POST') }
+      it { expect(build.path).to eql("/v1/#{command.path}") }
+      it { expect(build.to_hash).to have_key('authorization') }
+      it { expect(build.to_hash).to have_key('sample-header') }
+      it { expect(build.to_hash['sample-header']).to eql(['1']) }
     end
   end
 end

data/spec/lib/castle/api_spec.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 describe Castle::API do
-  subject(:request) { described_class.request(command) }
+  subject(:call) { described_class.call(command) }
 
   let(:command) { Castle::Commands::Track.new({}).build(event: '$login.succeeded') }
 
@@ -10,7 +10,7 @@ describe Castle::API do
 
     it do
       expect do
-        request
+        call
       end.to raise_error(Castle::RequestError)
     end
   end
@@ -20,7 +20,7 @@ describe Castle::API do
 
     it do
       expect do
-        request
+        call
       end.to raise_error(Castle::BadRequestError)
     end
   end
@@ -30,7 +30,7 @@ describe Castle::API do
 
     it do
       expect do
-        request
+        call
       end.to raise_error(Castle::ConfigurationError)
     end
   end

data/spec/lib/castle/configuration_spec.rb

@@ -2,7 +2,7 @@
 
 describe Castle::Configuration do
   subject(:config) do
-    described_class.new
+    described_class.instance
   end
 
   describe 'host' do
@@ -38,6 +38,7 @@ describe Castle::Configuration do
         allow(ENV).to receive(:fetch).with(
           'CASTLE_API_SECRET', ''
         ).and_return(secret_key_env)
+        config.reset
       end
 
       it do
@@ -66,7 +67,7 @@ describe Castle::Configuration do
     end
 
     it do
-      expect(config.api_secret).to be_eql('')
+      expect(config.api_secret).to be_eql('secret')
     end
   end
 

data/spec/lib/castle/context/default_spec.rb

@@ -4,14 +4,14 @@ describe Castle::Context::Default do
   subject { described_class.new(request, nil) }
 
   let(:ip) { '1.2.3.4' }
-  let(:cookie_id) { 'abcd' }
+  let(:client_id) { 'abcd' }
 
   let(:env) do
     Rack::MockRequest.env_for('/',
                               'HTTP_X_FORWARDED_FOR' => ip,
                               'HTTP_ACCEPT_LANGUAGE' => 'en',
                               'HTTP_USER_AGENT' => 'test',
-                              'HTTP_COOKIE' => "__cid=#{cookie_id};other=efgh")
+                              'HTTP_COOKIE' => "__cid=#{client_id};other=efgh")
   end
   let(:request) { Rack::Request.new(env) }
   let(:default_context) { subject.call }
@@ -34,6 +34,7 @@ describe Castle::Context::Default do
   it { expect(default_context[:origin]).to be_eql('web') }
   it { expect(default_context[:headers]).to be_eql(result_headers) }
   it { expect(default_context[:ip]).to be_eql(ip) }
+  it { expect(default_context[:client_id]).to be_eql(client_id) }
   it { expect(default_context[:library][:name]).to be_eql('castle-rb') }
   it { expect(default_context[:library][:version]).to be_eql(version) }
   it { expect(default_context[:user_agent]).to be_eql('test') }

data/spec/lib/castle/extractors/client_id_spec.rb

@@ -3,7 +3,7 @@
 describe Castle::Extractors::ClientId do
   subject(:extractor) { described_class.new(formatted_headers, cookies) }
 
-  let(:formatted_headers) { Castle::HeaderFilter.new(request).call }
+  let(:formatted_headers) { Castle::HeadersFilter.new(request).call }
   let(:client_id_cookie) { 'abcd' }
   let(:client_id_header) { 'abcde' }
   let(:cookies) { request.cookies }

data/spec/lib/castle/extractors/headers_spec.rb

@@ -3,7 +3,6 @@
 describe Castle::Extractors::Headers do
   subject(:headers) { described_class.new(formatted_headers).call }
 
-  let(:client_id) { 'abcd' }
   let(:formatted_headers) do
     {
       'Content-Length' => '0',

data/spec/lib/castle/extractors/ip_spec.rb

@@ -4,6 +4,11 @@ describe Castle::Extractors::IP do
   subject(:extractor) { described_class.new(headers) }
 
   describe 'ip' do
+    after do
+      Castle.config.ip_headers = []
+      Castle.config.trusted_proxies = []
+    end
+
     context 'when regular ip' do
       let(:headers) { { 'X-Forwarded-For' => '1.2.3.5' } }
 

data/spec/lib/castle/{header_filter_spec.rb → headers_filter_spec.rb}

@@ -1,16 +1,15 @@
 # frozen_string_literal: true
 
-describe Castle::HeaderFilter do
+describe Castle::HeadersFilter do
   subject(:headers) { described_class.new(request).call }
 
-  let(:client_id) { 'abcd' }
   let(:env) do
     Rack::MockRequest.env_for(
       '/',
       'Action-Dispatch.request.content-Type' => 'application/json',
       'HTTP_AUTHORIZATION' => 'Basic 123456',
-      'HTTP_COOKIE' => "__cid=#{client_id};other=efgh",
-      'HTTP_OK' => 'OK',
+      'HTTP_COOKIE' => '__cid=abcd;other=efgh',
+      'http-ok' => 'OK',
       'HTTP_ACCEPT' => 'application/json',
       'HTTP_X_FORWARDED_FOR' => '1.2.3.4',
       'HTTP_USER_AGENT' => 'Mozilla 1234',
@@ -22,7 +21,7 @@ describe Castle::HeaderFilter do
     {
       'Accept' => 'application/json',
       'Authorization' => 'Basic 123456',
-      'Cookie' => "__cid=#{client_id};other=efgh",
+      'Cookie' => '__cid=abcd;other=efgh',
       'Content-Length' => '0',
       'Ok' => 'OK',
       'User-Agent' => 'Mozilla 1234',

data/spec/lib/castle/{header_formatter_spec.rb → headers_formatter_spec.rb}

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-describe Castle::HeaderFormatter do
-  subject(:formatter) { described_class.new }
+describe Castle::HeadersFormatter do
+  subject(:formatter) { described_class }
 
   it 'removes HTTP_' do
     expect(formatter.call('HTTP_X_TEST')).to be_eql('X-Test')

data/spec/spec_helper.rb

@@ -16,8 +16,7 @@ WebMock.disable_net_connect!(allow_localhost: true)
 
 RSpec.configure do |config|
   config.before do
-    Castle.instance_variable_set(:@configuration, Castle::Configuration.new)
-
+    Castle.config.reset
     Castle.configure do |cfg|
       cfg.api_secret = 'secret'
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: castle-rb
 version: !ruby/object:Gem::Version
-  version: 4.0.0
+  version: 4.1.0
 platform: ruby
 authors:
 - Johan Brissmyr
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-03-20 00:00:00.000000000 Z
+date: 2020-03-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: appraisal
@@ -35,8 +35,8 @@ files:
 - lib/castle.rb
 - lib/castle/api.rb
 - lib/castle/api/request.rb
-- lib/castle/api/request/build.rb
 - lib/castle/api/response.rb
+- lib/castle/api/session.rb
 - lib/castle/client.rb
 - lib/castle/command.rb
 - lib/castle/commands/authenticate.rb
@@ -54,8 +54,8 @@ files:
 - lib/castle/extractors/headers.rb
 - lib/castle/extractors/ip.rb
 - lib/castle/failover_auth_response.rb
-- lib/castle/header_filter.rb
-- lib/castle/header_formatter.rb
+- lib/castle/headers_filter.rb
+- lib/castle/headers_formatter.rb
 - lib/castle/review.rb
 - lib/castle/secure_mode.rb
 - lib/castle/support/hanami.rb
@@ -73,7 +73,6 @@ files:
 - spec/integration/rails/support/all.rb
 - spec/integration/rails/support/application.rb
 - spec/integration/rails/support/home_controller.rb
-- spec/lib/castle/api/request/build_spec.rb
 - spec/lib/castle/api/request_spec.rb
 - spec/lib/castle/api/response_spec.rb
 - spec/lib/castle/api_spec.rb
@@ -92,8 +91,8 @@ files:
 - spec/lib/castle/extractors/client_id_spec.rb
 - spec/lib/castle/extractors/headers_spec.rb
 - spec/lib/castle/extractors/ip_spec.rb
-- spec/lib/castle/header_filter_spec.rb
-- spec/lib/castle/header_formatter_spec.rb
+- spec/lib/castle/headers_filter_spec.rb
+- spec/lib/castle/headers_formatter_spec.rb
 - spec/lib/castle/review_spec.rb
 - spec/lib/castle/secure_mode_spec.rb
 - spec/lib/castle/utils/cloner_spec.rb
@@ -136,6 +135,7 @@ test_files:
 - spec/integration/rails/rails_spec.rb
 - spec/lib/castle_spec.rb
 - spec/lib/castle/review_spec.rb
+- spec/lib/castle/headers_filter_spec.rb
 - spec/lib/castle/client_spec.rb
 - spec/lib/castle/context/default_spec.rb
 - spec/lib/castle/context/merger_spec.rb
@@ -143,15 +143,13 @@ test_files:
 - spec/lib/castle/api_spec.rb
 - spec/lib/castle/configuration_spec.rb
 - spec/lib/castle/version_spec.rb
-- spec/lib/castle/header_formatter_spec.rb
 - spec/lib/castle/utils/cloner_spec.rb
 - spec/lib/castle/utils/timestamp_spec.rb
 - spec/lib/castle/utils/merger_spec.rb
 - spec/lib/castle/command_spec.rb
+- spec/lib/castle/headers_formatter_spec.rb
 - spec/lib/castle/api/request_spec.rb
 - spec/lib/castle/api/response_spec.rb
-- spec/lib/castle/api/request/build_spec.rb
-- spec/lib/castle/header_filter_spec.rb
 - spec/lib/castle/commands/review_spec.rb
 - spec/lib/castle/commands/authenticate_spec.rb
 - spec/lib/castle/commands/track_spec.rb

data/lib/castle/api/request/build.rb

@@ -1,27 +0,0 @@
-# frozen_string_literal: true
-
-module Castle
-  module API
-    # generate api request
-    module Request
-      module Build
-        class << self
-          def call(command, headers, api_secret)
-            request = Net::HTTP.const_get(
-              command.method.to_s.capitalize
-            ).new("/#{Castle.config.url_prefix}/#{command.path}", headers)
-
-            unless command.method == :get
-              request.body = ::Castle::Utils.replace_invalid_characters(
-                command.data
-              ).to_json
-            end
-
-            request.basic_auth('', api_secret)
-            request
-          end
-        end
-      end
-    end
-  end
-end

data/lib/castle/header_formatter.rb

@@ -1,12 +0,0 @@
-# frozen_string_literal: true
-
-module Castle
-  # formats header name
-  class HeaderFormatter
-    # @param header [String]
-    # @return [String]
-    def call(header)
-      header.to_s.gsub(/^HTTP(?:_|-)/i, '').split(/_|-/).map(&:capitalize).join('-')
-    end
-  end
-end

data/spec/lib/castle/api/request/build_spec.rb

@@ -1,46 +0,0 @@
-# frozen_string_literal: true
-
-describe Castle::API::Request::Build do
-  subject(:call) { described_class.call(command, headers, api_secret) }
-
-  let(:headers) { { 'SAMPLE-HEADER' => '1' } }
-  let(:api_secret) { 'secret' }
-
-  describe 'call' do
-    context 'when get' do
-      let(:command) { Castle::Commands::Review.build(review_id) }
-      let(:review_id) { SecureRandom.uuid }
-
-      it { expect(call.body).to be_nil }
-      it { expect(call.method).to eql('GET') }
-      it { expect(call.path).to eql("/v1/#{command.path}") }
-      it { expect(call.to_hash).to have_key('authorization') }
-      it { expect(call.to_hash).to have_key('sample-header') }
-      it { expect(call.to_hash['sample-header']).to eql(['1']) }
-    end
-
-    context 'when post' do
-      let(:time) { Time.now.utc.iso8601(3) }
-      let(:command) do
-        Castle::Commands::Track.new({}).build(event: '$login.succeeded', name: "\xC4")
-      end
-      let(:expected_body) do
-        {
-          event: '$login.succeeded',
-          name: '�',
-          context: {},
-          sent_at: time
-        }
-      end
-
-      before { allow(Castle::Utils::Timestamp).to receive(:call).and_return(time) }
-
-      it { expect(call.body).to be_eql(expected_body.to_json) }
-      it { expect(call.method).to eql('POST') }
-      it { expect(call.path).to eql("/v1/#{command.path}") }
-      it { expect(call.to_hash).to have_key('authorization') }
-      it { expect(call.to_hash).to have_key('sample-header') }
-      it { expect(call.to_hash['sample-header']).to eql(['1']) }
-    end
-  end
-end