Revert "Except and log if an event could not be parsed correctly."

This reverts commit dd0483736d6d2f194908eff73f36f50de1dd0197.

Breaks ability for skipping lines which are not,
affected by any filter (if-clause) of a parser.

Signed-off-by: Stefan Schantl <stefan.schantl@ipfire.org>

diff --git a/guardian b/guardian
index 691ddc66e1f0013add0fd5245f7864d70931d049..7b4c23fae7b86c7e2d5b1ea7a726a23b03baf2f9 100644 (file)
--- a/guardian
+++ b/guardian
@@ -259,12 +259,6 @@ sub Worker ($) {
                                # Put the required action into the queue.
                                $queue->enqueue(@action);
                        }
-                       # If no action is returned by the Parser, the message
-                       # could not be parser corretly.
-                       else {
-                               # Log failed parse attempt.
-                               $logger->Log("err", "Error parsing event: \[$parser - @message\]");
-                       }
                } else {
                        # Sleep for 10ms until the next round of the loop will start.
                        sleep(0.01);

diff --git a/modules/Parser.pm b/modules/Parser.pm
index 9ec058693ee8866aa82ce0979a2eba2f2f5a5d93..9c3fc87130204d8bbb674a4afdb7eecb2c0041d2 100644 (file)
--- a/modules/Parser.pm
+++ b/modules/Parser.pm
@@ -30,16 +30,8 @@ sub Parser ($$) {
        # Call responsible message parser.
        my $action = $logfile_parsers{$parser}->(@message);
 
-       # If the parser successfully parsed the message, an action
-       # has been returned.
-       if ($action) {
-               # Return which action should be performed.
-               return "count $action";
-       }
-
-       # If the parser was not able to parse the the given message
-       # in the right way, return Nothing.
-       return;
+       # Return which action should be performed.
+       return "count $action";
 }
 
 #