fix: Move from appsignal logger to rails logger (#5)

.github/workflows/reviewdog.yml

@@ -6,8 +6,9 @@ jobs:
       fail-fast: false
       matrix:
         ruby:
-          - 3.0
+          - 3.1
           - 3.2
+          - 3.3
 
     runs-on: ubuntu-latest
     steps:

appsignal-sourcemap.gemspec

@@ -16,12 +16,14 @@ Gem::Specification.new do |spec|
 
   spec.metadata["allowed_push_host"] = "https://rubygems.pkg.github.com/Drieam"
 
-  spec.required_ruby_version = ">= 2.7"
+  spec.required_ruby_version = ">= 3.1"
 
   spec.files = Dir["lib/**/*", "README.md"]
 
+  spec.add_dependency "rails", ">= 6.1", "< 8.0"
   spec.add_dependency "appsignal", "~> 3.0"
   spec.add_dependency "parallel", "~> 1.0"
 
   spec.add_development_dependency "standard"
+  spec.metadata["rubygems_mfa_required"] = "true"
 end

lib/appsignal/sourcemap/supervisor.rb

@@ -15,27 +15,25 @@ def self.start
       def start
         return if invalid_preconditions
 
-        Appsignal.logger.info("Starting sourcemaps upload")
+        Rails.logger.info("Starting sourcemaps upload")
 
         Parallel.each(source_map_paths, in_threads: PARALLEL_THREADS) do |source_map_path|
           Uploader.upload(source_map_path)
         end
 
-        Appsignal.logger.info("Finished sourcemaps upload")
+        Rails.logger.info("Finished sourcemaps upload")
       end
 
       private
 
       def invalid_preconditions
         unless Appsignal.config.valid?
-          return Appsignal.logger.error("Skipping sourcemaps upload since Appsignal config is invalid")
+          return Rails.logger.error("Skipping sourcemaps upload since Appsignal config is invalid")
         end
         if asset_host.blank?
-          return Appsignal.logger.error("Skipping sourcemaps upload since Rails asset_host is not set")
-        end
-        if source_map_paths.empty?
-          return Appsignal.logger.info("Skipping sourcemaps upload since no javascript maps are found")
+          return Rails.logger.error("Skipping sourcemaps upload since Rails asset_host is not set")
         end
+        return Rails.logger.info("Skipping sourcemaps upload since no javascript maps are found") if source_map_paths.empty?
 
         false
       end

lib/appsignal/sourcemap/uploader.rb

@@ -13,24 +13,22 @@ def initialize(sourcemap_path)
         @sourcemap_path = sourcemap_path
       end
 
-      def upload # rubocop:disable Metrics/AbcSize
-        Appsignal.logger.debug "Starting sourcemap upload '#{@sourcemap_path}' with parameters: #{request_form_data}"
+      def upload
+        Rails.logger.debug("Starting sourcemap upload '#{@sourcemap_path}' with parameters: #{request_form_data}")
 
         response = Net::HTTP.start(UPLOAD_URI.hostname, UPLOAD_URI.port, use_ssl: true) do |http|
           http.request(request)
         end
 
         if response.is_a?(Net::HTTPSuccess)
-          Appsignal.logger.debug("Finished sourcemap upload '#{@sourcemap_path}'")
+          Rails.logger.debug("Finished sourcemap upload '#{@sourcemap_path}'")
           File.delete(sourcemap_full_path)
           return
         end
 
-        Appsignal.logger.error <<~MESSAGE
+        Rails.logger.error <<~MESSAGE
           Uploading sourcemap #{@sourcemap_path} failed with message '#{response.message}'.
-
             Response: #{response.body}
-
         MESSAGE
       end
 

lib/appsignal/sourcemap/version.rb

@@ -2,6 +2,6 @@
 
 module Appsignal
   module Sourcemap
-    VERSION = "1.0.0.develop"
+    VERSION = "1.1.1.develop"
   end
 end