Initialize Redis in searx/webapp.py

settings.yml:
* The default URL was unix:///usr/local/searxng-redis/run/redis.sock?db=0
* The default URL is now "false"

The default URL makes the log difficult to deal with:
if the admin didn't install a Redis instance, the logs record a false error.

It worked before because SearXNG initialized the Redis connection when the limiter started.

In this commit, SearXNG initializes Redis in searx/webapp.py
so various components can use Redis without taking care of the initialization step.

1 files changed, 3 insertions, 3 deletions

diff --git a/searx/search/checker/scheduler.lua b/searx/search/checker/scheduler.lua
index b3c6023fe..c5ebb05bc 100644
--- a/searx/search/checker/scheduler.lua
+++ b/searx/search/checker/scheduler.lua
@@ -18,8 +18,8 @@ if (next_call_ts == false or next_call_ts == nil) then
     -- the scheduler has never run on this Redis instance, so:
     -- 1/ the scheduler does not run now
     -- 2/ the next call is a random time between start_after_from and start_after_to
-    local delay = start_after_from + math.random(start_after_to - start_after_from)
-    redis.call('SET', redis_key, now + delay)
+    local initial_delay = math.random(start_after_from, start_after_to)
+    redis.call('SET', redis_key, now + initial_delay)
     return { false, delay }
 end
 
@@ -30,7 +30,7 @@ local call_now = next_call_ts <= now
 if call_now then
     -- the checker runs now, define the timestamp of the next call:
     -- this is a random delay between every_from and every_to
-    local periodic_delay = every_from + math.random(every_to - every_from)
+    local periodic_delay = math.random(every_from, every_to)
     next_call_ts = redis.call('INCRBY', redis_key, periodic_delay)
 end
 return { call_now, next_call_ts - now }