mihari 5.3.1 → 5.3.2

data/lib/mihari/emitters/webhook.rb

@@ -57,7 +57,7 @@ module Mihari
 
       #
       # @param [Array<Mihari::Artifact>] artifacts
-      # @param [Mihari::Structs::Rule] rule
+      # @param [Mihari::Services::Rule] rule
       # @param [Hash] **options
       #
       def initialize(artifacts:, rule:, **options)

data/lib/mihari/mixins/configurable.rb

@@ -49,6 +49,11 @@ module Mihari
 
       private
 
+      #
+      # Check whether API key is set or not
+      #
+      # @return [Boolean]
+      #
       def api_key?
         value = method(:api_key).call
         !value.nil?

data/lib/mihari/mixins/falsepositive.rb

@@ -22,7 +22,7 @@ module Mihari
       memoize :normalize_falsepositive
 
       #
-      # Check whetehr a value is valid format as a disallowed data value
+      # Check whether a value is valid format as a disallowed data value
       #
       # @param [String] value Data value
       #

data/lib/mihari/mixins/retriable.rb

@@ -21,8 +21,6 @@ module Mihari
       # @param [Integer] interval
       # @param [Array<StandardError>] on
       #
-      # @return [nil]
-      #
       def retry_on_error(times: 3, interval: 5, on: DEFAULT_ON)
         try = 0
         begin

data/lib/mihari/{structs → services}/rule.rb

@@ -8,7 +8,7 @@ require "securerandom"
 require "yaml"
 
 module Mihari
-  module Structs
+  module Services
     class Rule
       include Mixins::FalsePositive
 
@@ -49,16 +49,16 @@ module Mihari
       end
 
       def validate!
-        raise RuleValidationError if errors?
-      rescue RuleValidationError => e
+        return unless errors?
+
         Mihari.logger.error "Failed to parse the input as a rule:"
         Mihari.logger.error JSON.pretty_generate(errors.to_h)
 
-        raise e
+        raise RuleValidationError, errors
       end
 
       def [](key)
-        data[key.to_sym]
+        data key.to_sym
       end
 
       #
@@ -141,7 +141,7 @@ module Mihari
       #
       # @return [Mihari::Rule]
       #
-      def model
+      def to_model
         rule = Mihari::Rule.find(id)
 
         rule.title = title
@@ -161,8 +161,8 @@ module Mihari
       #
       # @return [Mihari::Analyzers::Rule]
       #
-      def analyzer
-        Mihari::Analyzers::Rule.new(self)
+      def to_analyzer
+        Mihari::Analyzers::Rule.new self
       end
 
       class << self
@@ -171,10 +171,10 @@ module Mihari
         #
         # @param [String] yaml
         #
-        # @return [Mihari::Structs::Rule]
+        # @return [Mihari::Services::Rule]
         #
         def from_yaml(yaml)
-          Structs::Rule.new YAML.safe_load(ERB.new(yaml).result, permitted_classes: [Date, Symbol])
+          Services::Rule.new YAML.safe_load(ERB.new(yaml).result, permitted_classes: [Date, Symbol])
         rescue Psych::SyntaxError => e
           raise YAMLSyntaxError, e.message
         end
@@ -182,10 +182,10 @@ module Mihari
         #
         # @param [Mihari::Rule] model
         #
-        # @return [Mihari::Structs::Rule]
+        # @return [Mihari::Services::Rule]
         #
         def from_model(model)
-          Structs::Rule.new(model.data)
+          Services::Rule.new model.data
         end
 
         #
@@ -193,7 +193,7 @@ module Mihari
         #
         # @param [String] path
         #
-        # @return [Mihari::Structs::Rule, nil]
+        # @return [Mihari::Services::Rule, nil]
         #
         def from_path(path)
           return nil unless Pathname(path).exist?
@@ -206,18 +206,18 @@ module Mihari
         #
         # @param [String] id
         #
-        # @return [Mihari::Structs::Rule, nil]
+        # @return [Mihari::Services::Rule, nil]
         #
         def from_id(id)
           return nil unless Mihari::Rule.exists?(id)
 
-          Structs::Rule.from_model Mihari::Rule.find(id)
+          Services::Rule.from_model Mihari::Rule.find(id)
         end
 
         #
         # @param [String] path_or_id Path to YAML file or YAML string or ID of a rule in the database
         #
-        # @return [Mihari::Structs::Rule]
+        # @return [Mihari::Services::Rule]
         #
         def from_path_or_id(path_or_id)
           rule = from_path(path_or_id)

data/lib/mihari/version.rb

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

data/lib/mihari/web/endpoints/rules.rb

@@ -83,12 +83,13 @@ module Mihari
           id = params["id"].to_s
 
           begin
-            rule = Mihari::Structs::Rule.from_model(Mihari::Rule.find(id))
+            rule = Mihari::Services::Rule.from_model(Mihari::Rule.find(id))
           rescue ActiveRecord::RecordNotFound
             error!({ message: "ID:#{id} is not found" }, 404)
           end
 
-          rule.analyzer.run
+          analyzer = rule.to_analyzer
+          analyzer.run
 
           status 201
           present({ message: "ID:#{id} is ran successfully" }, with: Entities::Message)
@@ -105,7 +106,7 @@ module Mihari
           yaml = params[:yaml]
 
           begin
-            rule = Structs::Rule.from_yaml(yaml)
+            rule = Services::Rule.from_yaml(yaml)
           rescue YAMLSyntaxError => e
             error!({ message: e.message }, 400)
           end
@@ -128,13 +129,13 @@ module Mihari
           end
 
           begin
-            rule.model.save
+            rule.to_model.save
           rescue ActiveRecord::RecordNotUnique
             error!({ message: "ID:#{rule.id} is already registered" }, 400)
           end
 
           status 201
-          present rule.model, with: Entities::Rule
+          present rule.to_model, with: Entities::Rule
         end
 
         desc "Update a rule", {
@@ -156,7 +157,7 @@ module Mihari
           end
 
           begin
-            rule = Structs::Rule.from_yaml(yaml)
+            rule = Services::Rule.from_yaml(yaml)
           rescue YAMLSyntaxError => e
             error!({ message: e.message }, 400)
           end
@@ -171,13 +172,13 @@ module Mihari
           end
 
           begin
-            rule.model.save
+            rule.to_model.save
           rescue ActiveRecord::RecordNotUnique
             error!({ message: "ID:#{id} is already registered" }, 400)
           end
 
           status 201
-          present rule.model, with: Entities::Rule
+          present rule.to_model, with: Entities::Rule
         end
 
         desc "Delete a rule", {