appsignal 3.5.0 → 3.5.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fc20e180510d89079b968f752fe4378590ef2edc3c881af7390e4eca883829c3
-  data.tar.gz: e475459d746362fe4a1d66ea260596dd376dd78de5ce0c6192784a61afcaa17b
+  metadata.gz: b47730be2c60175630bd45d3605b48ef70c5b9f6caaf1b304e8d073b72f8f495
+  data.tar.gz: '039bc2bf8ce611fe2778dfa2aa20c5259bc8e3bdb4e86689861957a42eaa53a2'
 SHA512:
-  metadata.gz: e794952ff70e7b8170bb173ee594cfa609a28cc713e04959a3bc97d274e7c547519f54404dfa2a1ac9d71c0ee47cb999bbbc21f19a66e2cef3d39d88f0ac30f7
-  data.tar.gz: 62c66a4814ee5663d7db3a1c2c1e844e5024c152425998685c12cd48e52fad3d6a63027457c3f599d2e506ca5b924734d1d48ca7c7d2b348ab80d2d831260ec3
+  metadata.gz: bebb4c065d529b36897c9c561c24e52b41e9a3b9ac3314ef8ede5bdb99830f293a45fd40513433dd8e66c3a1a50c9604404fad2a0b7a0f9065dbc0ddf713506e
+  data.tar.gz: e5659b4b13ba75d680e106b16d7ff30dba55c15728f5dbb3191180abf3aec38ebab16fbe2b3889cf0829a97abe00ca4b0cea45b1f0e2db1827ba140fde8fb809

data/CHANGELOG.md

@@ -1,5 +1,18 @@
 # AppSignal for Ruby gem Changelog
 
+## 3.5.2
+
+### Fixed
+
+- [a5963f65](https://github.com/appsignal/appsignal-ruby/commit/a5963f65cd06cdc0f6482be34917c365affc87dd) patch - Fix Ruby Logger 1.6.0 compatibility
+
+## 3.5.1
+
+### Fixed
+
+- [2e93182b](https://github.com/appsignal/appsignal-ruby/commit/2e93182b6ae83b16fe9885558cd8f0bfce6a9a5f) patch - Fix an error in the diagnose report when reading a file's contents results in an "Invalid seek" error. This could happen when the log path is configured to `/dev/stdout`, which is not supported.
+- [ae0b779b](https://github.com/appsignal/appsignal-ruby/commit/ae0b779b3ec00cc46291bc0373d748d720231e74) patch - Fix logger compatibility with Ruby 3.3
+
 ## 3.5.0
 
 ### Added

data/lib/appsignal/cli/diagnose/paths.rb

@@ -71,10 +71,14 @@ module Appsignal
               :group => Utils.group_for_gid(path_gid)
             }
             if info[:type] == "file"
-              info[:content] = Utils.read_file_content(
-                path,
-                BYTES_TO_READ_FOR_FILES
-              ).split("\n")
+              begin
+                info[:content] = Utils.read_file_content(
+                  path,
+                  BYTES_TO_READ_FOR_FILES
+                ).split("\n")
+              rescue => error
+                info[:read_error] = "#{error.class}: #{error.message}"
+              end
             end
           end
         end

data/lib/appsignal/cli/diagnose.rb

@@ -631,6 +631,11 @@ module Appsignal
           else
             print_empty_line
           end
+
+          return unless path.key?(:read_error)
+
+          puts "    Read error: #{path[:read_error]}"
+          print_empty_line
         end
 
         def print_empty_line

data/lib/appsignal/logger.rb

@@ -17,6 +17,8 @@ module Appsignal
       FATAL => 7
     }.freeze
 
+    attr_reader :level
+
     # Create a new logger instance
     #
     # @param group Name of the group for this logger.

data/lib/appsignal/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Appsignal
-  VERSION = "3.5.0"
+  VERSION = "3.5.2"
 end

data/spec/lib/appsignal/cli/diagnose/utils_spec.rb

@@ -71,5 +71,16 @@ describe Appsignal::CLI::Diagnose::Utils do
         is_expected.to eq(file_contents)
       end
     end
+
+    context "when reading the file raises an illegal seek error" do
+      let(:file_contents) { "line 1\n" }
+      before do
+        expect(File).to receive(:binread).and_raise(Errno::ESPIPE)
+      end
+
+      it "returns the error as the content" do
+        expect { subject }.to raise_error(Errno::ESPIPE)
+      end
+    end
   end
 end

data/spec/lib/appsignal/cli/diagnose_spec.rb

@@ -1424,6 +1424,26 @@ describe Appsignal::CLI::Diagnose, :api_stub => true, :send_report => :yes_cli_i
             )
           end
         end
+
+        context "when reading the file returns a illegal seek error" do
+          before do
+            File.write(file_path, "Some content")
+            allow(File).to receive(:binread).and_call_original
+            expect(File).to receive(:binread).with(file_path, anything,
+              anything).and_raise(Errno::ESPIPE)
+            run
+          end
+
+          it "outputs file does not exists" do
+            expect(output).to include %(Read error: Errno::ESPIPE: Illegal seek)
+          end
+
+          it "transmits file data in report" do
+            expect(received_report["paths"][filename]).to include(
+              "read_error" => "Errno::ESPIPE: Illegal seek"
+            )
+          end
+        end
       end
 
       describe "mkmf.log" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 3.5.0
+  version: 3.5.2
 platform: ruby
 authors:
 - Robert Beekman
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-12-11 00:00:00.000000000 Z
+date: 2023-12-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -446,7 +446,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.7
+rubygems_version: 3.4.11
 signing_key:
 specification_version: 4
 summary: Logs performance and exception data from your app to appsignal.com