setsuzoku 0.10.9 → 0.11.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6c8b08a243d276dfb01612b33007ee093766cccadfadce6ef915b7c4bf9b429d
-  data.tar.gz: c06e45bdb58caa6d110dadb67b77e962022b753e94bc694cf473cf658d75c56f
+  metadata.gz: 576427e8c71dfaf7306d5c7de069440bc47f4d8d9de185f3eb230a8304ea8162
+  data.tar.gz: 24fc28270185bca4db25ac804b783e28ec2e6c505c17e545e35625cc2695ca8e
 SHA512:
-  metadata.gz: 83448fd5e96fc2b25d0b0532dbdea1671be0f3282fcae35ea8e2d8cf1f0e0ed7d1294411fc5d69e50dc8fe66ad40ee16c4df5fbeb74c6d1b948c7781de5d528c
-  data.tar.gz: 05d695d1ebae47ba2e82f116926c0bdad0acce7310b4d738c3d0287d07254392fb095ef87270ab826bad7aecc2f889b676e819d598d7ebedbfaaba29ddd8a5d3
+  metadata.gz: 66c24bc8e924754a8b47fff0180ebd8ea73c7f42d1ddfe9694c3600850870514b6357a87f86c6b820b657e5393a081ab68d635534a2555dcc10419243540c2d7
+  data.tar.gz: f55aae400065ab41b8f5d98e66154f80ca044c8de60cd552b3e29fd3d8d9895d09a930bf9a3e7253d2d5ce835062a75ec89f26cde7275419bda7f0a389dc11f9

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    setsuzoku (0.10.9)
+    setsuzoku (0.11.5)
       activesupport (~> 5.0)
       faraday (~> 0.11)
       httparty (~> 0.16.4)
@@ -57,7 +57,7 @@ GEM
     safe_yaml (1.0.5)
     sorbet (0.5.5675)
       sorbet-static (= 0.5.5675)
-    sorbet-runtime (0.5.5794)
+    sorbet-runtime (0.5.5862)
     sorbet-static (0.5.5675-universal-darwin-14)
     thread_safe (0.3.6)
     tzinfo (1.2.7)

data/lib/setsuzoku/api_response.rb

@@ -4,7 +4,7 @@
 module Setsuzoku
   # The generic API response object that all API responses should return.
   class ApiResponse < T::Struct
-    prop :data, T.nilable(T::Hash[Symbol, T.untyped]), default: {}
+    prop :data, T.nilable(T.untyped), default: {}
     prop :success, T::Boolean, default: false
     prop :error, T.nilable(String), default: nil
   end

data/lib/setsuzoku/api_strategy.rb

@@ -80,7 +80,7 @@ module Setsuzoku
       self.external_api_handler.call_external_api_wrapper(plugin: self.plugin, request: request, action_details: action_details) do
         begin
           # raise if the token is invalid and needs refreshed, but don't raise if we are trying to get a refresh token
-          raise Setsuzoku::Exception::InvalidAuthCredentials.new unless (request.action == :refresh_token || self.auth_strategy.auth_credential_valid?)
+          raise Setsuzoku::Exception::InvalidAuthCredentials.new unless (strategy == :auth || self.auth_strategy.auth_credential_valid?)
           raw_response = self.perform_external_call(request: request, action_details: action_details, **options)
           formatted_response = self.parse_response(response: raw_response, response_type: action_details[:actions][request.action][:response_type])
           success = true
@@ -118,7 +118,7 @@ module Setsuzoku
     # @param options [Hash] the parsing options. Generally the response_type. e.g. :xml, :json
     #
     # @return [Hash] the parsed hash of the response object.
-    sig { abstract.params(response: T.untyped, options: T.untyped).returns(T::Hash[Symbol, T.untyped]) }
+    sig { abstract.params(response: T.untyped, options: T.untyped).returns(T.untyped) }
     def parse_response(response:, **options); end
   end
 end

data/lib/setsuzoku/rspec/dynamic_spec_helper.rb

@@ -2,6 +2,8 @@
 
 require 'active_support/core_ext/class/subclasses'
 require 'active_support/inflector/methods'
+require 'faraday'
+
 module Setsuzoku
   module DynamicSpecHelper
 
@@ -150,20 +152,21 @@ module Setsuzoku
         stub.each do |action_name, props|
           props.each do |propz|
             plugin.api_strategy.current_action = action_name
-            default_headers = plugin.api_strategy.request_options(propz[:request_format])[:headers] || { 'Authorization' => 'Bearer stubbed_token', 'Content-Type' => "application/#{propz[:request_format]}" }
+            default_headers = plugin.api_strategy.request_options(propz[:request_format])[:headers] || { 'Content-Type' => "application/#{propz[:request_format]}" }
             if plugin.auth_strategy.is_a?(Setsuzoku::Service::WebService::AuthStrategies::BasicAuthStrategy)
               basic_auth = plugin.api_strategy.request_options(propz[:request_format])[:basic_auth]
               auth_header = { 'Authorization' => "Basic #{Base64.encode64("#{basic_auth[:username]}:#{basic_auth[:password]}").gsub("\n", '')}" }
-              default_headers.merge!(auth_header)
+            elsif plugin.auth_strategy.is_a?(Setsuzoku::Service::WebService::AuthStrategies::OAuthStrategy)
+              auth_header = { 'Authorization' => "Bearer #{plugin.api_strategy.request_options(propz[:request_format])[:token]}" }
             end
+            default_headers.merge!(auth_header) if auth_header
             headers = propz[:headers] ? [propz[:headers]] : [default_headers]
             stub_directory ||= "#{plugin.class.plugin_namespace}/#{provider}"
+
             headers.each do |header|
-              header = {
-                  'Accept'=>'*/*',
-                  'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3',
-                  'User-Agent'=>'Ruby'
-              }.merge(header.deep_stringify_keys)
+              header = Faraday.new.headers
+                        .merge({'Accept'=>'*/*','Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3'})
+                        .merge(header.deep_stringify_keys)
 
               if propz[:stub_data]
                 # we have specified specific stub data for this action
@@ -265,9 +268,11 @@ module Setsuzoku
           url, body = plugin.api_strategy.replace_dynamic_vars(full_url: url, req_params: req_params)
         end
 
-        case format
-        when :json
+        if propz[:request_format] == :file
+          body = body.to_json if body.is_a?(Hash)
+        elsif format == :json
           #faraday sorts hash keys for some reason
+          body = body.to_json if body.is_a?(Hash)
           body = JSON.parse(body).sort.to_h
         end
         [url, body]

data/lib/setsuzoku/service/web_service/api_strategies/rest_strategy.rb

@@ -33,7 +33,7 @@ module Setsuzoku
           #   media_type - 'json'
           #
           # @return [Hash] the parsed response object.
-          sig { override.params(request: T.untyped, action_details: T::Hash[T.untyped, T.untyped], options: T.untyped).returns(Faraday::Response) }
+          sig { override.params(request: RestAPIRequest, action_details: T::Hash[T.untyped, T.untyped], options: T.untyped).returns(Faraday::Response) }
           def perform_external_call(request:, action_details:, **options)
             request_properties = self.get_request_properties(action_name: request.action, action_details: action_details, req_params: request.body)
             request_options = self.request_options(request_properties[:request_format], action_details[:actions][request.action])
@@ -43,11 +43,14 @@ module Setsuzoku
             @faraday = Faraday.new(url: request_properties[:formatted_full_url], request: { params_encoder: Faraday::FlatParamsEncoder }) do |faraday|
               faraday.request(:multipart) if options[:attachments].present?
               faraday.request(:url_encoded)
-              # faraday.basic_auth(request_options[:basic_auth][:username], request_options[:basic_auth][:password]) if request_options[:basic_auth]
-              faraday.request(:basic_auth, request_options[:basic_auth][:username], request_options[:basic_auth][:password]) if request_options[:basic_auth]
-              # faraday.response request_properties[:response_format] if request_properties[:response_format]
-              faraday.authorization(:Bearer, request_properties[:token]) if request_properties[:token]
-              # request_options[:headers][:Authorization].prepend('Bearer ') if request_options[:headers][:Authorization]
+              #TODO: change these to faraday = self.auth_strategy.set_authorization!(request: faraday)
+              unless request.without_headers
+                if request_options[:token]
+                  faraday.authorization(:Bearer, request_options[:token])
+                elsif request_options[:basic_auth]
+                  faraday.request(:basic_auth, request_options[:basic_auth][:username], request_options[:basic_auth][:password])
+                end
+              end
               faraday.adapter Faraday.default_adapter
             end
 
@@ -60,7 +63,11 @@ module Setsuzoku
                 req.body = payload
               end
             else
-              @faraday.headers = request_options[:headers] if request_options[:headers]
+              if request.without_headers
+                @faraday.headers = {}
+              elsif request_options[:headers]
+                @faraday.headers = (@faraday.headers || {}).merge(request_options[:headers])
+              end
               resp = @faraday.send(request_properties[:request_method], request_properties[:formatted_full_url], full_request)
             end
             resp
@@ -81,7 +88,6 @@ module Setsuzoku
             request_method = request_method.downcase.to_sym
             request_format = action[:request_type]
             response_format = action[:response_type]
-            token = action[:token]
             stub_data = action[:stub_data] if for_stub
             full_url = url + endpoint
             formatted_full_url, req_params = self.replace_dynamic_vars(full_url: full_url, req_params: req_params)
@@ -93,7 +99,6 @@ module Setsuzoku
                 formatted_full_url: formatted_full_url,
                 req_params: req_params,
                 stub_data: stub_data,
-                token: token
             }
           end
 

data/lib/setsuzoku/service/web_service/api_strategy.rb

@@ -72,13 +72,15 @@ module Setsuzoku
         # @param options [Hash] the parsing options. Generally the response_type. e.g. :xml, :json
         #
         # @return [Hash] the parsed hash of the response object.
-        sig { override.params(response: Faraday::Response, options: T.untyped).returns(T::Hash[Symbol, T.untyped]) }
+        sig { override.params(response: Faraday::Response, options: T.untyped).returns(T.untyped) }
         def parse_response(response:, **options)
           case options[:response_type]
           when :json
             JSON.parse(response.body).deep_symbolize_keys
           when :xml
             convert_xml_to_hash(response.body)
+          when :html
+            response.body
           else
             JSON.parse(response.body).deep_symbolize_keys
           end

data/lib/setsuzoku/service/web_service/auth_strategies/o_auth_strategy.rb

@@ -31,9 +31,7 @@ module Setsuzoku
           sig { override.returns(T::Hash[Symbol, T.untyped]) }
           def auth_headers
             {
-                headers: {
-                    'Authorization': "Bearer #{self.credential.token}"
-                }
+              token: self.credential.token
             }
           end
 
@@ -112,7 +110,7 @@ module Setsuzoku
           sig { params(body: T::Hash[Symbol, String], action: Symbol).void }
           def get_token!(body, action)
             success = false
-            request = self.api_strategy.request_class.new(action: action, body: body)
+            request = self.api_strategy.request_class.new(action: action, body: body, without_headers: true)
 
             resp = self.api_strategy.call_external_api(request: request, strategy: :auth)
 

data/lib/setsuzoku/version.rb

@@ -2,5 +2,5 @@
 # frozen_string_literal: true
 
 module Setsuzoku
-  VERSION = '0.10.9'
+  VERSION = '0.11.5'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: setsuzoku
 version: !ruby/object:Gem::Version
-  version: 0.10.9
+  version: 0.11.5
 platform: ruby
 authors:
 - Luke Stadtler
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-08-04 00:00:00.000000000 Z
+date: 2020-08-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler