Only try to read-in the providers settings file, in case it exists. Otherwise the script produces an error message, about the missing file, each time it gets executed.
Because of the fcron job this would be twice a day in most cases.
Signed-off-by: Stefan Schantl stefan.schantl@ipfire.org --- src/scripts/update-ids-ruleset | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/scripts/update-ids-ruleset b/src/scripts/update-ids-ruleset index e9a082e62..553c1a1e1 100644 --- a/src/scripts/update-ids-ruleset +++ b/src/scripts/update-ids-ruleset @@ -95,8 +95,8 @@ if ($return) { # The script has requested a lock, so set locket to "1". $locked = "1";
-# Grab the configured providers. -&General::readhasharray("$IDS::providers_settings_file", %providers); +# Grab the configured providers, if the providers settings file exists. +&General::readhasharray("$IDS::providers_settings_file", %providers) if (-f "$IDS::providers_settings_file");
# Loop through the array of available providers. foreach my $id (keys %providers) {
Reviewed-by: Michael Tremer michael.tremer@ipfire.org
On 13 May 2022, at 18:10, Stefan Schantl stefan.schantl@ipfire.org wrote:
Only try to read-in the providers settings file, in case it exists. Otherwise the script produces an error message, about the missing file, each time it gets executed.
Because of the fcron job this would be twice a day in most cases.
Signed-off-by: Stefan Schantl stefan.schantl@ipfire.org
src/scripts/update-ids-ruleset | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/scripts/update-ids-ruleset b/src/scripts/update-ids-ruleset index e9a082e62..553c1a1e1 100644 --- a/src/scripts/update-ids-ruleset +++ b/src/scripts/update-ids-ruleset @@ -95,8 +95,8 @@ if ($return) { # The script has requested a lock, so set locket to "1". $locked = "1";
-# Grab the configured providers. -&General::readhasharray("$IDS::providers_settings_file", %providers); +# Grab the configured providers, if the providers settings file exists. +&General::readhasharray("$IDS::providers_settings_file", %providers) if (-f "$IDS::providers_settings_file");
# Loop through the array of available providers. foreach my $id (keys %providers) { -- 2.30.2
-
Michael Tremer -
Stefan Schantl