mihari 7.0.4 → 7.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a5859aa404d85ccc45008979195ace2c4104cc86336b65afacbeece226c7def5
-  data.tar.gz: 75f6687159420d8f0d7b5f2f13fbab7f523881ad55a66574a7a9ec612cf720ae
+  metadata.gz: aca048c797a73db1c57f0a5ad5f29066fbfba69451c71df1a26c3ea57853a1e3
+  data.tar.gz: 5ce2d298f325cc9d0ae50a98a6c18a64ec11cc06386504535c8ca647c7c68ad0
 SHA512:
-  metadata.gz: 9db820d33af592d9b2953a01e745ce05c3a297a6cbefb409e147fd90ab5d02070722abbba18b3f282bf9abcc1cc85f50d2579ab2774ea26074d74b6aea8d6073
-  data.tar.gz: 8218aaf062e8ecad9cf01f58188de497ffba135ce42269e7ff87b82d49cf96c0a46c0aa273e752da1f97b288ffead8569fa00c8d6452701273e9694bcf4dba74
+  metadata.gz: ac1d6ab49351a2c9c60e703a5fe5fe0f94c84c4294271f297d529a781d3a87a37b515d090a78676bcaf3da225f03617f8900966d108e31557e86319a2407b00b
+  data.tar.gz: f5aad0f8648783f4d3c3cc42a087f646358e5e93586cb394a2644e8f6c32a296b3668ea7758348c0dd55d4868527b4aa6a3d774ee113d5a4a14c39ae3ecd2857

data/lib/mihari/analyzers/base.rb

@@ -92,7 +92,15 @@ module Mihari
 
         return res.recover { [] } if ignore_error?
 
-        res.to_result
+        result = res.to_result
+        return result if result.success?
+
+        # Wrap failure with AnalyzerError to explicitly name a failed analyzer
+        Failure AnalyzerError.new(
+          result.failure.message,
+          self.class.class_key,
+          cause: result.failure
+        )
       end
 
       class << self

data/lib/mihari/cli/application.rb

@@ -41,17 +41,17 @@ module Mihari
         def safe_execute
           yield
         rescue StandardError => e
-          err = unwrap_error(e)
+          error = unwrap_error(e)
 
-          raise err if options["debug"]
+          raise error if options["debug"]
 
-          case err
-          when ValidationError
-            warn JSON.pretty_generate(err.errors.to_h)
-          when StandardError
-            Sentry.capture_exception(err) if Sentry.initialized?
-            warn err
-          end
+          data = Entities::ErrorMessage.represent(
+            message: error.message,
+            detail: error.respond_to?(:detail) ? error.detail : nil
+          )
+          warn JSON.pretty_generate(data.as_json)
+
+          Sentry.capture_exception(error) if Sentry.initialized? && !error.is_a?(ValidationError)
 
           exit 1
         end

data/lib/mihari/clients/fofa.rb

@@ -52,7 +52,10 @@ module Mihari
       def search(query, page:, size: PAGE_SIZE)
         qbase64 = Base64.urlsafe_encode64(query)
         params = { qbase64: qbase64, size: size, page: page, email: email, key: api_key }.compact
-        Structs::Fofa::Response.from_dynamic! get_json("/api/v1/search/all", params: params)
+        res = Structs::Fofa::Response.from_dynamic!(get_json("/api/v1/search/all", params: params))
+        raise ResponseError, res.errmsg if res.error
+
+        res
       end
 
       #

data/lib/mihari/concerns/error_unwrappable.rb

@@ -17,9 +17,10 @@ module Mihari
         receiver = err.receiver
         case receiver
         when Dry::Monads::Try::Error
-          receiver.exception
+          # Error may be wrapped like Matryoshka
+          unwrap_error receiver.exception
         when Dry::Monads::Failure
-          receiver.failure
+          unwrap_error receiver.failure
         else
           err
         end

data/lib/mihari/concerns/retriable.rb

@@ -8,35 +8,48 @@ module Mihari
     module Retriable
       extend ActiveSupport::Concern
 
-      DEFAULT_ON = [
-        Errno::ECONNRESET,
-        Errno::ECONNABORTED,
-        Errno::EPIPE,
+      RETRIABLE_ERRORS = [
         OpenSSL::SSL::SSLError,
         Timeout::Error,
-        RetryableError,
-        NetworkError,
-        TimeoutError,
-        StatusCodeError
+        ::HTTP::ConnectionError,
+        ::HTTP::ResponseError,
+        ::HTTP::TimeoutError
       ].freeze
 
+      DEFAULT_CONDITION = lambda do |error|
+        return true if RETRIABLE_ERRORS.any? { |klass| error.is_a? klass }
+
+        case error
+        when StatusCodeError
+          error.status_code != 404
+        else
+          false
+        end
+      end
+
       #
       # Retry on error
       #
       # @param [Integer] times
       # @param [Integer] interval
       # @param [Boolean] exponential_backoff
-      # @param [Array<StandardError>] on
+      # @param [Proc] condition
       #
-      def retry_on_error(times: 3, interval: 5, exponential_backoff: true, on: DEFAULT_ON)
+      # @param [Object] on
+      def retry_on_error(times: 3, interval: 5, exponential_backoff: true, condition: DEFAULT_CONDITION)
         try = 0
         begin
           try += 1
           yield
-        rescue *on => e
+        rescue StandardError => e
+          # Raise error if it's not a retriable error
+          raise e unless condition.call(e)
+
           sleep_seconds = exponential_backoff ? interval * (2**(try - 1)) : interval
           sleep sleep_seconds
           retry if try < times
+
+          # Raise error if retry times exceed a given times
           raise e
         end
       end

data/lib/mihari/errors.rb

@@ -1,27 +1,45 @@
 # frozen_string_literal: true
 
+require "http"
+
 module Mihari
   class Error < StandardError; end
 
   class ValueError < Error; end
 
-  class RetryableError < Error; end
-
   class ConfigurationError < Error; end
 
-  class IntegrityError < Error; end
+  class ResponseError < Error; end
 
-  # errors for HTTP interactions
-  class HTTPError < Error; end
+  class AnalyzerError < Error
+    # @return [StandardException, nil]
+    attr_reader :cause
 
-  class NetworkError < HTTPError; end
+    #
+    # @param [String] msg
+    # @param [String] analyzer
+    # @param [StandardException, nil] cause
+    #
+    def initialize(msg, analyzer, cause: nil)
+      super("#{msg} (from #{analyzer})")
+
+      @cause = cause
+      set_backtrace(cause.backtrace) if cause
+    end
 
-  class TimeoutError < HTTPError; end
+    def detail
+      return nil unless cause.respond_to?(:detail)
+
+      cause.detail
+    end
+  end
+
+  class IntegrityError < Error; end
 
   #
   # HTTP status code error
   #
-  class StatusCodeError < HTTPError
+  class StatusCodeError < ::HTTP::Error
     # @return [Integer]
     attr_reader :status_code
 
@@ -39,6 +57,10 @@ module Mihari
       @status_code = status_code
       @body = body
     end
+
+    def detail
+      { status_code: status_code, body: body }
+    end
   end
 
   #
@@ -56,5 +78,9 @@ module Mihari
 
       @errors = errors
     end
+
+    def detail
+      errors.to_h
+    end
   end
 end

data/lib/mihari/http.rb

@@ -18,11 +18,6 @@ module Mihari
         )
       end
 
-      def on_error(_request, error)
-        raise TimeoutError, error if error.is_a?(::HTTP::TimeoutError)
-        raise NetworkError, error if error.is_a?(::HTTP::Error)
-      end
-
       ::HTTP::Options.register_feature(:better_error, self)
     end
 

data/lib/mihari/rule.rb

@@ -83,6 +83,20 @@ module Mihari
       data[:data_types]
     end
 
+    #
+    # @return [Date, nil]
+    #
+    def created_on
+      data[:created_on]
+    end
+
+    #
+    # @return [Date, nil]
+    #
+    def updated_on
+      data[:updated_on]
+    end
+
     #
     # @return [Array<Mihari::Models::Tag>]
     #
@@ -279,8 +293,8 @@ module Mihari
       # data is serialized as JSON so dates (created_on & updated_on) are stringified in there
       # thus dates & (hash) keys have to be stringified when comparing
       data.deep_dup.tap do |data|
-        data[:created_on] = data[:created_on].to_s
-        data[:updated_on] = data[:updated_on].to_s
+        data[:created_on] = created_on.to_s unless created_on.nil?
+        data[:updated_on] = updated_on.to_s unless updated_on.nil?
       end.deep_stringify_keys
     end
 

data/lib/mihari/structs/fofa.rb

@@ -8,6 +8,10 @@ module Mihari
         #   @return [Boolean]
         attribute :error, Types::Bool
 
+        # @!attribute [r] errmsg
+        #   @return [String, nil]
+        attribute? :errmsg, Types::String.optional
+
         # @!attribute [r] size
         #   @return [Integer, nil]
         attribute? :size, Types::Int.optional
@@ -35,6 +39,7 @@ module Mihari
           def from_dynamic!(d)
             new(
               error: d.fetch("error"),
+              errmsg: d["errmsg"],
               size: d["size"],
               page: d["page"],
               mode: d["mode"],

data/lib/mihari/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Mihari
-  VERSION = "7.0.4"
+  VERSION = "7.0.5"
 end