files.com 1.0.485 → 1.0.486

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9c9a0745484268da357be364cb333a92140455e2c3a6e010f62239dd9c2b6875
-  data.tar.gz: ae0cb3ca584be567d50722f8311dc7886de107cf15280db79cc13292a5c36302
+  metadata.gz: 322c6e72224efa5614973b2af891e64fa439ae20abf0c7660b53ff4c2b54055c
+  data.tar.gz: 3e32e5563c2c663634074e5639674fe5b734c4ba8de49829e454cab4ca9c596a
 SHA512:
-  metadata.gz: 272b0b8443e770a14e4a3ea6592cdc454f2925a46861f18590d4b04a5887b25ddbe781b80b8859bb3c262c7894417829d3541db508079cbe53989e646133c954
-  data.tar.gz: b1c1c0e78d428403aa8ccfcedd738cb3f16d3a9a2562dd843530d85a5653006a1f5dceba938038a6986c0d8cba9bd0b692e815a51a8b4e78e39a26abf64ab52c
+  metadata.gz: 75490c41da14ebe46d5dca8c79653a9b8158dbe14297d3888ac1bde6cca684e36aa8c6be063360f9b05aae2dd8940104bb5efa3a9d23469f7e23a425fc85fe35
+  data.tar.gz: d6546b9dbbc10027955c872401c395f7a722b5528b8c2c4ef214a80c4747fe47b6bedbeae72babed3bc8f34d5296f109dff1574651f058f28090528ebda6aa7b

data/_VERSION

@@ -1 +1 @@
-1.0.485
+1.0.486

data/lib/files.com/api_client.rb

@@ -235,13 +235,10 @@ module Files
         end
 
         case e
-        when Faraday::ClientError, Faraday::ServerError
-          if e.response
-            handle_error_response(e.response, error_context)
-          else
-            handle_network_error(e, error_context, num_retries, base_url)
-          end
-
+        when Faraday::ClientError
+          handle_error_response(e.response, error_context)
+        when Faraday::ServerError
+          handle_network_error(e, error_context, num_retries, base_url)
         else
           raise
         end
@@ -302,10 +299,10 @@ module Files
       end
     end
 
-    private def handle_network_error(error, context, num_retries, base_url = nil)
+    private def handle_network_error(error, _context, num_retries, base_url = nil)
       base_url ||= Files.base_url
 
-      Util.log_error("Network error", error_message: error.message, request_id: context.request_id)
+      Util.log_error("Network error", error_message: error.message)
       message = "Could not connect to Files.com at URL #{base_url}. Please check your internet connection and try again. If this problem persists, you should check Files.com's service status at https://status.files.com, or contact your primary account representative."
       message += " Request was retried #{num_retries} times." if num_retries > 0
       message += "\n\n(Network error: #{error.message})"

data/lib/files.com/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Files
-  VERSION = "1.0.485"
+  VERSION = "1.0.486"
 end

data/spec/lib/api_client_spec.rb

@@ -1,4 +1,5 @@
 require "spec_helper"
+require "pry"
 
 RSpec.describe Files::ApiClient do
   let(:subject) { described_class.new }
@@ -8,22 +9,22 @@ RSpec.describe Files::ApiClient do
 
     shared_examples 'a server error handler' do
       before do
-        allow(subject).to receive(:log_request).and_raise(Faraday::ServerError.new('', mock_response))
+        allow(subject).to receive(:log_request).and_raise(Faraday::ServerError.new('the server responded with status 502', mock_response))
       end
 
-      it 'retries with sleeps and raises general api error' do
+      it 'retries with sleeps and raises general api connection error' do
         expect(subject).to receive(:sleep).with(0.5).ordered
         expect(subject).to receive(:sleep).with(be_between(0.5, 1)).ordered
         expect(subject).to receive(:sleep).with(be_between(1, 2)).ordered
         expect {
           subject.execute_request_with_rescues(1, context) { 'empty block' }
-        }.to raise_error(Files::APIError, error_message)
+        }.to raise_error(Files::APIConnectionError, error_message)
       end
     end
 
     context 'when response is a valid json' do
       it_behaves_like 'a server error handler' do
-        let(:error_message) { "Server Error" }
+        let(:error_message) { "Could not connect to Files.com at URL 1. Please check your internet connection and try again. If this problem persists, you should check Files.com's service status at https://status.files.com, or contact your primary account representative. Request was retried 3 times.\n\n(Network error: the server responded with status 502)" }
         let(:mock_response) { { status: 502, headers: {}, body: { error: error_message }.to_json } }
       end
     end
@@ -37,7 +38,7 @@ RSpec.describe Files::ApiClient do
             body: "<html><head><title>502 Bad Gateway</title></head><body><center><h1>502 Bad Gateway</h1></center><hr><center>files.com</center></body></html>"
           }
         }
-        let(:error_message) { "Unexpected response object from API: \"#{mock_response[:body]}\" (HTTP response code was 502)" }
+        let(:error_message) { "Could not connect to Files.com at URL 1. Please check your internet connection and try again. If this problem persists, you should check Files.com's service status at https://status.files.com, or contact your primary account representative. Request was retried 3 times.\n\n(Network error: the server responded with status 502)" }
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: files.com
 version: !ruby/object:Gem::Version
-  version: 1.0.485
+  version: 1.0.486
 platform: ruby
 authors:
 - files.com
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-11-10 00:00:00.000000000 Z
+date: 2023-11-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: addressable