fortnox-api 0.8.2 → 0.9.0

data/fortnox-api.gemspec

@@ -1,7 +1,6 @@
-
 # frozen_string_literal: true
 
-lib = File.expand_path('../lib', __FILE__)
+lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require_relative 'lib/fortnox/api/version'
 
@@ -11,16 +10,18 @@ Gem::Specification.new do |spec|
   spec.authors       = ['Jonas Schubert Erlandsson', 'Hannes Elvemyr', 'Felix Holmgren', 'Mike Eirih']
   spec.email         = ['info@accodeing.com']
   spec.summary       = 'Gem to use Fortnox REST API in Ruby.'
-  spec.description   = 'This gem uses the HTTParty library to abstract away the REST calls. It gives you access to a '\
-                       'number of objects that behave a lot like ActiveRecord instances, giving you access to methods '\
-                       'like `all`, `find`, `find_by_...` and so on. And each individual instance can be easily'\
-                       'persistable to Fortnox again using the `save` method.'
+  spec.description   = 'This gem uses the HTTParty library to abstract away the REST calls. It gives you access to a ' \
+                       'number of objects that behave a lot like ActiveRecord instances, giving you access to ' \
+                       'methods like `all`, `find`, `find_by_...` and so on. And each individual instance can be' \
+                       'easily persistable to Fortnox again using the `save` method.'
   spec.homepage      = 'http://github.com/accodeing/fortnox-api'
   spec.license       = 'LGPL-3.0'
+  spec.metadata = {
+    'rubygems_mfa_required' => 'true'
+  }
 
   spec.files         = `git ls-files -z`.split("\x0")
   spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
-  spec.test_files    = spec.files.grep(%r{^(spec)/})
   spec.require_paths = ['lib']
 
   spec.required_ruby_version = '>= 2.7'
@@ -29,13 +30,14 @@ Gem::Specification.new do |spec|
   spec.add_dependency 'dry-container', '~> 0.10'
   spec.add_dependency 'dry-struct', '~> 1.6'
   spec.add_dependency 'dry-types', '~> 1.7'
-  spec.add_dependency 'httparty', '~> 0.17'
+  spec.add_dependency 'httparty', '~> 0.21'
   spec.add_dependency 'jwt', '~> 2.3'
 
   spec.add_development_dependency 'bundler', '~> 2.4'
   spec.add_development_dependency 'dotenv', '~> 2.8'
   spec.add_development_dependency 'guard', '~> 2.18'
   spec.add_development_dependency 'guard-rspec', '~> 4.7'
+  spec.add_development_dependency 'guard-rubocop', '~> 1.5'
   spec.add_development_dependency 'pry', '~> 0'
   spec.add_development_dependency 'rake', '~> 13.0'
   spec.add_development_dependency 'rspec', '~> 3.12'
@@ -43,7 +45,6 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rubocop', '~> 1.39.0'
   spec.add_development_dependency 'rubocop-rspec', '~> 2.16.0'
   spec.add_development_dependency 'simplecov', '~> 0.22'
-  spec.add_development_dependency 'timecop', '~> 0.9.6'
   spec.add_development_dependency 'vcr', '~> 6.1'
   spec.add_development_dependency 'webmock', '~> 3.18'
 end

data/lib/fortnox/api/mappers/base/from_json.rb

@@ -4,7 +4,7 @@ module Fortnox
   module API
     module Mapper
       module FromJSON
-        class MissingModelOrMapperException < StandardError
+        class MissingModelOrMapperException < Fortnox::API::Exception
         end
 
         def wrapped_json_collection_to_entities_hash(json_collection_hash)
@@ -45,10 +45,11 @@ module Fortnox
             # Raise exception during test run if this happens so that we can
             # add it before a new release.
 
-            message = "for #{key} (#{mapper_name}, #{Fortnox::API::Mapper::DefaultTemplates.canonical_name_sym}) with"\
-                      " #{collection}"
+            message = "for #{key} (#{mapper_name}, #{Fortnox::API::Mapper::DefaultTemplates.canonical_name_sym}) " \
+                      "with #{collection}"
 
             raise MissingModelOrMapperException, message if ENV['RUBY_ENV']
+
             Fortnox::API.logger.warn("Missing Model or Mapper implementation for #{key} with attributes: #{collection}")
             convert_hash_keys_from_json_format(collection, {})
           end

data/lib/fortnox/api/mappers/base/to_json.rb

@@ -35,13 +35,12 @@ module Fortnox
           end
 
           def default_key_to_json_transform(key)
-            key.to_s.split('_').map(&:capitalize).join('')
+            key.to_s.split('_').map(&:capitalize).join
           end
 
           def sanitise(hash, keys_to_filter)
             hash.reject do |key, value|
-              next false if keys_to_filter.include?(key)
-              value.nil?
+              keys_to_filter.include?(key) || value.nil?
             end
           end
         end

data/lib/fortnox/api/models/base.rb

@@ -26,8 +26,8 @@ module Fortnox
             obj = preserve_meta_properties(hash) do
               super(hash)
             end
-          rescue Dry::Struct::Error => e
-            raise Fortnox::API::AttributeError, e
+          rescue Dry::Struct::Error => exception
+            raise Fortnox::API::AttributeError, exception
           end
 
           IceNine.deep_freeze(obj)
@@ -52,30 +52,30 @@ module Fortnox
           end
         end
 
-        def to_hash(recursive = false)
+        def to_hash(recursive = false) # rubocop:disable Style/OptionalBooleanParameter
           return super() if recursive
 
           self.class.schema.each_with_object({}) do |key, result|
-              # Only output attributes that have a value set
-              result[key.name] = self[key.name] if self.send("#{key.name}?")
+            # Only output attributes that have a value set
+            result[key.name] = self[key.name] if send("#{key.name}?")
           end
         end
 
         def update(hash)
           old_attributes = to_hash
-          new_attributes = old_attributes.merge(hash)
+          new_attributes = old_attributes.merge(hash).compact
 
           return self if new_attributes == old_attributes
 
-          new_hash = new_attributes.delete_if { |_, value| value.nil? }
-          new_hash[:new] = @new
-          new_hash[:parent] = self
-          self.class.new(new_hash)
+          new_attributes[:new] = @new
+          new_attributes[:parent] = self
+          self.class.new(new_attributes)
         end
 
         # Generic comparison, by value, use .eql? or .equal? for object identity.
         def ==(other)
           return false unless other.is_a? self.class
+
           to_hash == other.to_hash
         end
 
@@ -115,6 +115,8 @@ module Fortnox
           obj
         end
 
+        private_class_method :preserve_meta_properties
+
         private
 
         def private_attributes

data/lib/fortnox/api/models/customer.rb

@@ -30,31 +30,31 @@ module Fortnox
         # Url Direct URL to the record
         attribute :url, Types::Nullable::String.is(:read_only)
 
-        # Address1	First address of the customer. 1024 characters
+        # Address1  First address of the customer. 1024 characters
         attribute :address1, Types::Sized::String[1024]
 
-        # Address2	Second address of the customer. 1024 characters
+        # Address2  Second address of the customer. 1024 characters
         attribute :address2, Types::Sized::String[1024]
 
-        # City	City of the customer. 1024 characters
+        # City  City of the customer. 1024 characters
         attribute :city, Types::Sized::String[1024]
 
-        # Country	Country of the customer. Read-only.
+        # Country  Country of the customer. Read-only.
         attribute :country, Types::Nullable::String.is(:read_only)
 
-        # Comments	Comments. 1024 characters.
+        # Comments  Comments. 1024 characters.
         attribute :comments, Types::Sized::String[1024]
 
-        # Currency	Currency of the customer, 3 letters
+        # Currency  Currency of the customer, 3 letters
         attribute :currency, Types::Currency
 
-        # CostCenter	Cost center of the customer, Cost center in Fortnox
+        # CostCenter  Cost center of the customer, Cost center in Fortnox
         attribute :cost_center, Types::Nullable::String
 
-        # CountryCode	Country code of the customer, 2 letters
+        # CountryCode  Country code of the customer, 2 letters
         attribute :country_code, Types::CountryCode
 
-        # CustomerNumber	Customer number. 1024 characters
+        # CustomerNumber  Customer number. 1024 characters
         attribute :customer_number, Types::Sized::String[1024]
 
         # DefaultDeliveryTypes The properties for this object is listed in the table for "Default Delivery Types".
@@ -63,146 +63,146 @@ module Fortnox
         # The properties for this object is listed in the table for "Default Templates".
         attribute :default_templates, Types::DefaultTemplates
 
-        # DeliveryAddress1	First delivery address of the customer. 1024 characters
+        # DeliveryAddress1  First delivery address of the customer. 1024 characters
         attribute :delivery_address1, Types::Sized::String[1024]
 
-        # DeliveryAddress2	Second delivery address of the customer. 1024 characters
+        # DeliveryAddress2  Second delivery address of the customer. 1024 characters
         attribute :delivery_address2, Types::Sized::String[1024]
 
-        # DeliveryCity	Delivery city of the customer. 1024 characters
+        # DeliveryCity  Delivery city of the customer. 1024 characters
         attribute :delivery_city, Types::Sized::String[1024]
 
-        # DeliveryCountry	Delivery country of the customer. Read-only.
+        # DeliveryCountry  Delivery country of the customer. Read-only.
         attribute :delivery_country, Types::Nullable::String.is(:read_only)
 
-        # DeliveryCountryCode	Delivery country code of the customer, 2 letters
+        # DeliveryCountryCode  Delivery country code of the customer, 2 letters
         attribute :delivery_country_code, Types::CountryCode
 
-        # DeliveryFax	Delivery fax number of the customer. 1024 characters
+        # DeliveryFax  Delivery fax number of the customer. 1024 characters
         attribute :delivery_fax, Types::Sized::String[1024]
 
-        # DeliveryName	Delivery name of the customer. 1024 characters
+        # DeliveryName  Delivery name of the customer. 1024 characters
         attribute :delivery_name, Types::Sized::String[1024]
 
-        # DeliveryPhone1	First delivery phone number of the customer. 1024 characters
+        # DeliveryPhone1  First delivery phone number of the customer. 1024 characters
         attribute :delivery_phone1, Types::Sized::String[1024]
 
-        # DeliveryPhone2	Second delivery phone number of the customer. 1024 characters
+        # DeliveryPhone2  Second delivery phone number of the customer. 1024 characters
         attribute :delivery_phone2, Types::Sized::String[1024]
 
-        # DeliveryZipCode	Delivery zip code of the customer. 1024 characters.
+        # DeliveryZipCode  Delivery zip code of the customer. 1024 characters.
         attribute :delivery_zip_code, Types::Sized::String[1024]
 
-        # Email	Email address of the customer. 1024 characters
+        # Email  Email address of the customer. 1024 characters
         attribute :email, Types::Email
 
-        # EmailInvoice	Invoice email address of the customer. 1024 characters
+        # EmailInvoice  Invoice email address of the customer. 1024 characters
         attribute :email_invoice, Types::Email
 
-        # EmailInvoiceBCC	Invoice BCC email address of the customer. 1024 characters
+        # EmailInvoiceBCC  Invoice BCC email address of the customer. 1024 characters
         attribute :email_invoice_bcc, Types::Email
 
-        # EmailInvoiceCC	Invoice CC email address of the customer. 1024 characters
+        # EmailInvoiceCC  Invoice CC email address of the customer. 1024 characters
         attribute :email_invoice_cc, Types::Email
 
-        # EmailOffer	Offer email address of the customer. 1024 characters
+        # EmailOffer  Offer email address of the customer. 1024 characters
         attribute :email_offer, Types::Email
 
-        # EmailOfferBCC	Offer BCC email address of the customer. 1024 characters
+        # EmailOfferBCC  Offer BCC email address of the customer. 1024 characters
         attribute :email_offer_bcc, Types::Email
 
-        # EmailOfferCC	Offer CC email address of the customer. 1024 characters
+        # EmailOfferCC  Offer CC email address of the customer. 1024 characters
         attribute :email_offer_cc, Types::Email
 
-        # EmailOrder	Order email address of the customer. 1024 characters
+        # EmailOrder  Order email address of the customer. 1024 characters
         attribute :email_order, Types::Email
 
-        # EmailOrderBCC	Order BCC email address of the customer. 1024 characters
+        # EmailOrderBCC  Order BCC email address of the customer. 1024 characters
         attribute :email_order_bcc, Types::Email
 
-        # EmailOrderCC	Order CC email address of the customer. 1024 characters
+        # EmailOrderCC  Order CC email address of the customer. 1024 characters
         attribute :email_order_cc, Types::Email
 
-        # Fax	Fax number of the customer. 1024 characters
+        # Fax  Fax number of the customer. 1024 characters
         attribute :fax, Types::Sized::String[1024]
 
         # InvoiceAdministrationFee  Invoice administration fee of the customer, 12 digits (incl. decimals).
         attribute :invoice_administration_fee,
                   Types::Sized::Float[0.0, 99_999_999_999.9]
 
-        # InvoiceDiscount	Invoice discount of the customer, 12 digits (incl. decimals)
+        # InvoiceDiscount  Invoice discount of the customer, 12 digits (incl. decimals)
         attribute :invoice_discount, Types::Sized::Float[0.0, 99_999_999_999.9]
 
-        # InvoiceFreight	Invoice freight fee of the customer, 12 digits (incl. decimals)
+        # InvoiceFreight  Invoice freight fee of the customer, 12 digits (incl. decimals)
         attribute :invoice_freight, Types::Sized::Float[0.0, 99_999_999_999.9]
 
-        # InvoiceRemark	Invoice remark of the customer. 1024 characters
+        # InvoiceRemark  Invoice remark of the customer. 1024 characters
         attribute :invoice_remark, Types::Sized::String[1024]
 
-        # Name	Name of the customer, 1024 characters
+        # Name  Name of the customer, 1024 characters
         attribute :name, Types::Sized::String[1024].is(:required)
 
-        # OrganisationNumber	Organisation number of the customer. 30 characters
+        # OrganisationNumber  Organisation number of the customer. 30 characters
         attribute :organisation_number, Types::Sized::String[30]
 
-        # OurReference	Our reference of the customer. 50 characters
+        # OurReference  Our reference of the customer. 50 characters
         attribute :our_reference, Types::Sized::String[50]
 
-        # Phone1	First phone number of the customer. 1024 characters
+        # Phone1  First phone number of the customer. 1024 characters
         attribute :phone1, Types::Sized::String[1024]
 
-        # Phone2	Second phone number of the customer. 1024 characters
+        # Phone2  Second phone number of the customer. 1024 characters
         attribute :phone2, Types::Sized::String[1024]
 
-        # PriceList	Price list of the customer, Price list in Fortnox
+        # PriceList  Price list of the customer, Price list in Fortnox
         attribute :price_list, Types::Nullable::String
 
-        # Project	Project of the customer, Project in Fortnox
+        # Project  Project of the customer, Project in Fortnox
         attribute :project, Types::Nullable::String
 
-        # SalesAccount	Sales account of the customer, 4 digits
-        attribute :sales_account, Types::AccountNumber
+        # SalesAccount  Sales account of the customer, 4 characters
+        attribute :sales_account, Types::SalesAccount
 
-        # ShowPriceVATIncluded	Show prices with VAT included or not
+        # ShowPriceVATIncluded  Show prices with VAT included or not
         attribute :show_price_vat_included, Types::Nullable::Boolean
 
-        # TermsOfDeliveryCode	Terms of delivery code of the customer
+        # TermsOfDeliveryCode  Terms of delivery code of the customer
         attribute :terms_of_delivery, Types::Nullable::String
 
-        # TermsOfPaymentCode	Terms of payment code of the customer
+        # TermsOfPaymentCode  Terms of payment code of the customer
         attribute :terms_of_payment, Types::Nullable::String
 
-        # Type	Customer type, PRIVATE / COMPANY
+        # Type  Customer type, PRIVATE / COMPANY
         attribute :type, Types::CustomerType
 
-        # VATNumber	VAT number of the customer
+        # VATNumber  VAT number of the customer
         attribute :vat_number, Types::Nullable::String
 
-        # VATType	VAT type of the customer, SEVAT / SEREVERSEDVAT / EUREVERSEDVAT / EUVAT / EXPORT
+        # VATType  VAT type of the customer, SEVAT / SEREVERSEDVAT / EUREVERSEDVAT / EUVAT / EXPORT
         attribute :vat_type, Types::VATType
 
-        # VisitAddress	Visit address of the customer. 128 characters
+        # VisitAddress  Visit address of the customer. 128 characters
         attribute :visiting_address, Types::Sized::String[128]
 
-        # VisitCity	Visit city of the customer. 128 characters
+        # VisitCity  Visit city of the customer. 128 characters
         attribute :visiting_city, Types::Sized::String[128]
 
-        # VisitCountry	Visit country of the customer, read-only
+        # VisitCountry  Visit country of the customer, read-only
         attribute :visiting_country, Types::Nullable::String.is(:read_only)
 
         # VisitingCountryCode Code of the visiting country for the customer, 2 letters
         attribute :visiting_country_code, Types::CountryCode
 
-        # VisitZipCode	Visit zip code of the customer. 10 characters
+        # VisitZipCode  Visit zip code of the customer. 10 characters
         attribute :visiting_zip_code, Types::Sized::String[10]
 
-        # WayOfDeliveryCode	Way of delivery code of the customer
+        # WayOfDeliveryCode  Way of delivery code of the customer
         attribute :way_of_delivery, Types::Nullable::String
 
-        # YourReference	Your reference of the customer. 50 characters
+        # YourReference  Your reference of the customer. 50 characters
         attribute :your_reference, Types::Sized::String[50]
 
-        # ZipCode	Zip code of the customer. 10 characters
+        # ZipCode  Zip code of the customer. 10 characters
         attribute :zip_code, Types::Sized::String[10]
       end
     end

data/lib/fortnox/api/models/label.rb

@@ -8,10 +8,10 @@ module Fortnox
       class Label < Model::Base
         STUB = {}.freeze
 
-        # Id	integer, read-only. The ID of the label.
+        # Id  integer, read-only. The ID of the label.
         attribute :id, Types::Required::Integer.is(:read_only)
 
-        # Description	string, 25 characters, required. Description of the label
+        # Description  string, 25 characters, required. Description of the label
         attribute :description, Types::Sized::String[25].is(:read_only)
       end
     end

data/lib/fortnox/api/repositories/authentication.rb

@@ -0,0 +1,61 @@
+# frozen_string_literal: true
+
+require_relative 'base'
+
+module Fortnox
+  module API
+    module Repository
+      class Authentication
+        def renew_tokens(refresh_token:, client_id:, client_secret:)
+          body = {
+            grant_type: 'refresh_token',
+            refresh_token: refresh_token
+          }
+
+          response = HTTParty.post(Fortnox::API.config.token_url,
+                                   headers: headers(client_id, client_secret),
+                                   body: body)
+
+          validate_response(response)
+
+          parsed_response_to_hash(response.parsed_response)
+        end
+
+        private
+
+        def parsed_response_to_hash(parsed_response)
+          {
+            access_token: parsed_response['access_token'],
+            refresh_token: parsed_response['refresh_token'],
+            expires_in: parsed_response['expires_in'],
+            token_type: parsed_response['token_type'],
+            scope: parsed_response['scope']
+          }
+        end
+
+        def headers(client_id, client_secret)
+          credentials = Base64.encode64("#{client_id}:#{client_secret}")
+
+          {
+            'Content-type' => 'application/x-www-form-urlencoded',
+            Authorization: "Basic #{credentials}"
+          }
+        end
+
+        def validate_response(response)
+          return if response.code == 200
+
+          case response.code
+          when 400
+            raise Fortnox::API::RemoteServerError, "Bad request. Error: \"#{response.body}\""
+          when 401
+            raise Fortnox::API::RemoteServerError, "Unauthorized request. Error: \"#{response.body}\""
+          else
+            raise Exception, "Unable to renew access token. Response code: #{response.code}. " \
+                             "Response message: #{response.message}. Response body: #{response.body}"
+          end
+        end
+      end
+    end
+  end
+end

data/lib/fortnox/api/repositories/base/savers.rb

@@ -8,6 +8,7 @@ module Fortnox
           return true if entity.saved?
 
           return save_new(entity) if entity.new?
+
           update_existing(entity)
         end
 
@@ -33,8 +34,9 @@ module Fortnox
 
         def get_changes_on(entity)
           hash = @mapper.entity_to_hash(entity, @keys_filtered_on_save)
-          parent_hash = @mapper.entity_to_hash(entity.parent, @keys_filtered_on_save)
+          return hash unless entity.parent.saved?
 
+          parent_hash = @mapper.entity_to_hash(entity.parent, @keys_filtered_on_save)
           @mapper.wrap_entity_json_hash(@mapper.diff(hash, parent_hash))
         end
 

data/lib/fortnox/api/repositories/base.rb

@@ -1,6 +1,8 @@
 # frozen_string_literal: true
 
 require 'httparty'
+require 'jwt'
+require 'base64'
 
 require_relative 'base/loaders'
 require_relative 'base/savers'
@@ -27,53 +29,47 @@ module Fortnox
         attr_accessor :headers
         attr_reader :mapper, :keys_filtered_on_save
 
-        def self.set_headers(headers = {}) # rubocop:disable Naming/AccessorMethodName
+        def self.set_headers(headers = {})
           self.headers.merge!(headers)
         end
 
         HTTP_METHODS.each do |method|
           define_method method do |path, options = {}, &block|
             provided_headers = options[:headers] || {}
-            provided_headers['Client-Secret'] = client_secret
-            provided_headers['Access-Token'] = next_access_token
+            provided_headers['Authorization'] = "Bearer #{access_token}"
             options[:headers] = provided_headers
             options[:base_uri] ||= base_url
+
+            debug_log_request(path, options)
+
             execute do |remote|
               remote.send(method, path, options, &block)
             end
           end
         end
 
-        def initialize(keys_filtered_on_save: [:url], token_store: :default)
+        def initialize(keys_filtered_on_save: [:url])
           @keys_filtered_on_save = keys_filtered_on_save
-          @token_store = token_store
           @mapper = Registry[Mapper::Base.canonical_name_sym(self.class::MODEL)].new
-        end
+          return unless access_token.nil?
 
-        def next_access_token
-          @access_tokens ||= CircularQueue.new(*access_tokens)
-          @access_tokens.next
+          raise MissingAccessToken,
+                'No Access Token provided! You need to provide an Access Token: ' \
+                'Fortnox::API.access_token = token'
         end
 
-        def check_access_tokens!(tokens)
-          tokens_present = !(tokens.nil? || tokens.empty?)
-          return if tokens_present
-          error_message = "You have not provided any access tokens in token store #{@token_store.inspect}."
-          raise MissingConfiguration, error_message
-        end
+        private
 
-        def access_tokens
-          begin
-            tokens = config.token_store.fetch(@token_store)
-          rescue KeyError
-            token_store_not_found!(@token_store.inspect)
-          end
+        def debug_log_request(path, options)
+          return unless Fortnox::API.debugging
 
-          check_access_tokens!(tokens)
-          tokens
+          Fortnox::API.logger.debug("path: #{path.inspect}")
+          Fortnox::API.logger.debug("options: #{options.inspect}")
         end
 
-        private
+        def access_token
+          Fortnox::API.access_token
+        end
 
         def instantiate(hash)
           hash[:new] = false
@@ -84,23 +80,13 @@ module Fortnox
         def base_url
           base_url = config.base_url
           raise MissingConfiguration, 'You have to provide a base url.' unless base_url
-          base_url
-        end
 
-        def client_secret
-          client_secret = config.client_secret
-          raise MissingConfiguration, 'You have to provide your client secret.' unless client_secret
-          client_secret
+          base_url
         end
 
         def config
           Fortnox::API.config
         end
-
-        def token_store_not_found!(store_name)
-          raise MissingConfiguration,
-                "There is no token store named #{store_name}. Available stores are #{config.token_store.keys}."
-        end
       end
     end
   end

data/lib/fortnox/api/repositories.rb

@@ -7,3 +7,4 @@ require_relative 'repositories/order'
 require_relative 'repositories/project'
 require_relative 'repositories/unit'
 require_relative 'repositories/terms_of_payment'
+require_relative 'repositories/authentication'

data/lib/fortnox/api/request_handling.rb

@@ -5,30 +5,42 @@ module Fortnox
     module RequestHandling
       private
 
-        def raise_api_error(error, response)
-          message = (error['message'] || error['Message'] || 'Okänt fel')
+      def raise_api_error(error, response)
+        message = (error['message'] || error['Message'] || 'Okänt fel')
 
-          message += "\n\n#{response.request.inspect}" if Fortnox::API.debugging
+        message += "\n\n#{response.request.inspect}" if Fortnox::API.debugging
 
-          raise Fortnox::API::RemoteServerError, message
-        end
+        raise Fortnox::API::RemoteServerError, message
+      end
 
-        def validate_response(response)
-          return if response.code == 200
+      def validate_response(response)
+        return if response.code == 200
 
-          api_error = response.parsed_response['ErrorInformation']
-          raise_api_error(api_error, response) if api_error
-        end
+        raise_content_type_error(response) if response.headers['content-type'].start_with?('text/html')
 
-        def validate_and_parse(response)
-          validate_response(response)
-          response.parsed_response
-        end
+        raise Fortnox::API::RemoteServerError, "Unauthorized request. Error: #{response.body}" if response.code == 401
+        raise Fortnox::API::RemoteServerError, "Forbidden request. Error: #{response.body}" if response.code == 403
 
-        def execute
-          response = yield(self.class)
-          validate_and_parse response
-        end
+        api_error = response.parsed_response['ErrorInformation']
+        raise_api_error(api_error, response) if api_error
+      end
+
+      def raise_content_type_error(response)
+        raise Fortnox::API::RemoteServerError,
+              "Fortnox API's response has content type \"text/html\" instead of requested \"application/json\"." \
+              'This could be due to invalid endpoint or when the API is down. ' \
+              "Body: #{response.body}"
+      end
+
+      def validate_and_parse(response)
+        validate_response(response)
+        response.parsed_response
+      end
+
+      def execute
+        response = yield(self.class)
+        validate_and_parse response
+      end
     end
   end
 end