forest_liana 9.6.0 → 9.6.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bdddfd71dbc4f9c5ddab069162d5b0ba65c057c6f9e25cda811259e72e1daaa8
-  data.tar.gz: 8213f378007db1ac2663b3143c0e8cc5ca35a63e603c8fb81b2c4bdbbcb7d270
+  metadata.gz: 654d11ad9ebc49989ce8f011c901cd93a8ec42d524242e4fb81a28c6ddf82028
+  data.tar.gz: 653d6d504a67d2bfbd34a73bbe4d091a8f076f3eaf72161da83cd72f0ce0dd23
 SHA512:
-  metadata.gz: 1f805f7d8604f9db01bb69a41d6a70d0b30d1baa4897e12cf4d16d2805c915d272cf197111d8102c7d1f543efbd34490399d8d86144840c615b1a02a2cc03e36
-  data.tar.gz: 257e946211ad7b126c99ef5f75e45f2f5e8e073b9c4996294ac1d3fcce1da7fbaa4db06d52284ead53072b3c7585d72a4cc1d7b2a7a3a61f5dfe0c37325c7ca6
+  metadata.gz: ecfe2712fe3abab2c261ee042b40964747d7d8f1b7289538e4fe1072f4f999bae6c2cbc7b3c165e3b170d7fd548e5041d85d29f10f9eb9ec04262ed6cf9383cc
+  data.tar.gz: 21589dd2b1f38b9d12d32fd92b41af0f449dad28b3139ca6692fe6729482eb0a16474dd83763c0702902c5a2c4514f6b33c4fee201442199688a8f7302dfe136

data/app/serializers/forest_liana/serializer_factory.rb

@@ -127,8 +127,8 @@ module ForestLiana
                   ret[:href] = "/forest/#{ForestLiana.name_for(object.class)}/#{object.id}/relationships/#{attribute_name}"
                 end
               end
-            rescue TypeError, ActiveRecord::StatementInvalid, NoMethodError
-              puts "Cannot load the association #{attribute_name} on #{object.class.name} #{object.id}."
+            rescue TypeError, ActiveRecord::StatementInvalid, NoMethodError => exception
+              FOREST_LOGGER.warn "Cannot load the association #{attribute_name} on #{object.class.name} #{object.id}.\n#{exception&.backtrace&.join("\n\t")}"
             end
           end
 

data/app/services/forest_liana/ability/exceptions/access_denied.rb

@@ -2,12 +2,13 @@ module ForestLiana
   module Ability
     module Exceptions
       class AccessDenied < ForestLiana::Errors::ExpectedError
-        def initialize
+        def initialize (backtrace = nil)
           super(
             403,
             :forbidden,
             'You don\'t have permission to access this resource',
-            'AccessDenied'
+            'AccessDenied',
+            backtrace
           )
         end
       end

data/app/services/forest_liana/ability/exceptions/action_condition_error.rb

@@ -2,12 +2,13 @@ module ForestLiana
   module Ability
     module Exceptions
       class ActionConditionError < ForestLiana::Errors::ExpectedError
-        def initialize
+        def initialize (backtrace = nil)
           super(
             409,
                 :conflict,
                 'The conditions to trigger this action cannot be verified. Please contact an administrator.',
-                'InvalidActionConditionError'
+                'InvalidActionConditionError',
+            backtrace
           )
         end
       end

data/app/services/forest_liana/ability/exceptions/require_approval.rb

@@ -3,13 +3,14 @@ module ForestLiana
     module Exceptions
       class RequireApproval < ForestLiana::Errors::ExpectedError
         attr_reader :data
-        def initialize(data)
+        def initialize(data, backtrace = nil)
           @data = data
           super(
             403,
                 :forbidden,
                 'This action requires to be approved.',
                 'CustomActionRequiresApprovalError',
+            backtrace,
           )
         end
       end

data/app/services/forest_liana/ability/exceptions/trigger_forbidden.rb

@@ -2,12 +2,13 @@ module ForestLiana
   module Ability
     module Exceptions
       class TriggerForbidden < ForestLiana::Errors::ExpectedError
-        def initialize
+        def initialize(backtrace = nil)
           super(
             403,
                 :forbidden,
                 'You don\'t have the permission to trigger this action',
-                'CustomActionTriggerForbiddenError'
+                'CustomActionTriggerForbiddenError',
+            backtrace,
           )
         end
       end

data/app/services/forest_liana/ability/exceptions/unknown_collection.rb

@@ -2,12 +2,13 @@ module ForestLiana
   module Ability
     module Exceptions
       class UnknownCollection < ForestLiana::Errors::ExpectedError
-        def initialize(collection_name)
+        def initialize(collection_name, backtrace = nil)
           super(
             409,
             :conflict,
             "The collection #{collection_name} doesn't exist",
-            'collection not found'
+            'collection not found',
+            backtrace
           )
         end
       end

data/app/services/forest_liana/ability/permission/smart_action_checker.rb

@@ -65,8 +65,8 @@ module ForestLiana
             end
 
             records.select(@collection.table_name + '.id').count == attributes[:ids].count
-          rescue
-            raise ForestLiana::Ability::Exceptions::ActionConditionError.new
+          rescue => exception
+            raise ForestLiana::Ability::Exceptions::ActionConditionError.new(exception.backtrace)
           end
         end
 

data/app/services/forest_liana/ability/permission.rb

@@ -27,8 +27,8 @@ module ForestLiana
           is_allowed
         rescue ForestLiana::Errors::ExpectedError => exception
           raise exception
-        rescue
-          raise ForestLiana::Ability::Exceptions::UnknownCollection.new(collection_name)
+        rescue => exception
+          raise ForestLiana::Ability::Exceptions::UnknownCollection.new(collection_name, exception.backtrace)
         end
       end
 
@@ -45,8 +45,8 @@ module ForestLiana
           smart_action_approval.can_execute?
         rescue ForestLiana::Errors::ExpectedError => exception
           raise exception
-        rescue
-          raise ForestLiana::Ability::Exceptions::UnknownCollection.new(collection_name)
+        rescue => exception
+          raise ForestLiana::Ability::Exceptions::UnknownCollection.new(collection_name, exception.backtrace)
         end
       end
 

data/app/services/forest_liana/forest_api_requester.rb

@@ -9,8 +9,10 @@ module ForestLiana
           headers: base_headers.merge(headers),
           query: query,
         }).response
-      rescue
-        raise "Cannot reach Forest API at #{forest_api_url}#{route}, it seems to be down right now."
+      rescue => error
+        FOREST_LOGGER.error "Cannot reach Forest API at #{forest_api_url}#{route}, it seems to be down right now."
+        FOREST_REPORTER.report error
+        raise error
       end
     end
 
@@ -28,8 +30,10 @@ module ForestLiana
           query: query,
           body: body.to_json,
         }).response
-      rescue
-        raise "Cannot reach Forest API at #{post_route}, it seems to be down right now."
+      rescue => error
+        FOREST_LOGGER.error "Cannot reach Forest API at #{post_route}, it seems to be down right now."
+        FOREST_REPORTER.report error
+        raise error
       end
     end
 

data/config/initializers/errors.rb

@@ -44,11 +44,13 @@ module ForestLiana
     class ExpectedError < StandardError
       attr_reader :error_code, :status, :message, :name
 
-      def initialize(error_code, status, message, name = nil)
+      def initialize(error_code, status, message, name = nil, backtrace = nil)
         @error_code = error_code
         @status = status
         @message = message
         @name = name
+
+        set_backtrace(backtrace) unless backtrace.nil?
       end
 
       def display_error

data/lib/forest_liana/version.rb

@@ -1,3 +1,3 @@
 module ForestLiana
-  VERSION = "9.6.0"
+  VERSION = "9.6.4"
 end

data/spec/config/initializers/errors_spec.rb

@@ -0,0 +1,34 @@
+module ForestLiana
+  describe Errors do
+    describe 'ExpectedError' do
+      describe 'when initializing' do
+        describe 'when backtrace is added' do
+          it 'should add the backtrace to the errors if passed' do
+            err = nil
+
+            begin
+              raise "This is an exception"
+            rescue => error
+              err = ForestLiana::Errors::ExpectedError.new(300, 300, error.message, nil, error.backtrace )
+            end
+
+            expect(err.backtrace).to be_truthy
+          end
+        end
+        describe 'when backtrace is not added' do
+          it 'should not break nor add any backtrace' do
+            err = nil
+
+            begin
+              raise "This is an exception"
+            rescue => error
+              err = ForestLiana::Errors::ExpectedError.new(300, 300, error.message, nil)
+            end
+
+            expect(err.backtrace).to be_falsy
+          end
+        end
+      end
+    end
+  end
+end

data/spec/services/forest_liana/forest_api_requester_spec.rb

@@ -0,0 +1,44 @@
+module ForestLiana
+  describe ForestApiRequester do
+    describe 'when an error occurs' do
+      before do
+        allow(HTTParty).to receive(:get).and_raise(StandardError, 'Custom error message')
+        allow(HTTParty).to receive(:post).and_raise(StandardError, 'Custom error message')
+      end
+
+      describe 'Get' do
+        it 'should keep the original error and raise it with backtrace' do
+          err = nil
+
+          begin
+            ForestApiRequester.get('/incorrect_url')
+          rescue => error
+            err = error
+          end
+
+          expect(error)
+            .to be_instance_of(StandardError)
+            .and have_attributes(:message => 'Custom error message')
+            .and have_attributes(:backtrace => be_truthy)
+        end
+      end
+
+      describe 'Post' do
+        it 'should keep the original error and raise it with backtrace' do
+          err = nil
+
+          begin
+            ForestApiRequester.post('/incorrect_url')
+          rescue => error
+            err = error
+          end
+
+          expect(error)
+            .to be_instance_of(StandardError)
+            .and have_attributes(:message => 'Custom error message')
+            .and have_attributes(:backtrace => be_truthy)
+        end
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: forest_liana
 version: !ruby/object:Gem::Version
-  version: 9.6.0
+  version: 9.6.4
 platform: ruby
 authors:
 - Sandro Munda
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-12-10 00:00:00.000000000 Z
+date: 2024-12-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -329,6 +329,7 @@ files:
 - lib/tasks/clear_oidc_data.rake
 - lib/tasks/display_apimap.rake
 - lib/tasks/send_apimap.rake
+- spec/config/initializers/errors_spec.rb
 - spec/config/initializers/logger_spec.rb
 - spec/dummy/README.rdoc
 - spec/dummy/Rakefile
@@ -420,6 +421,7 @@ files:
 - spec/services/forest_liana/ability/permission_spec.rb
 - spec/services/forest_liana/apimap_sorter_spec.rb
 - spec/services/forest_liana/filters_parser_spec.rb
+- spec/services/forest_liana/forest_api_requester_spec.rb
 - spec/services/forest_liana/has_many_getter_spec.rb
 - spec/services/forest_liana/ip_whitelist_checker_spec.rb
 - spec/services/forest_liana/line_stat_getter_spec.rb
@@ -632,6 +634,7 @@ test_files:
 - test/services/forest_liana/operator_date_interval_parser_test.rb
 - test/services/forest_liana/schema_adapter_test.rb
 - test/test_helper.rb
+- spec/config/initializers/errors_spec.rb
 - spec/config/initializers/logger_spec.rb
 - spec/dummy/README.rdoc
 - spec/dummy/Rakefile
@@ -723,6 +726,7 @@ test_files:
 - spec/services/forest_liana/ability/permission_spec.rb
 - spec/services/forest_liana/apimap_sorter_spec.rb
 - spec/services/forest_liana/filters_parser_spec.rb
+- spec/services/forest_liana/forest_api_requester_spec.rb
 - spec/services/forest_liana/has_many_getter_spec.rb
 - spec/services/forest_liana/ip_whitelist_checker_spec.rb
 - spec/services/forest_liana/line_stat_getter_spec.rb