feat: allow overriding response timeout of redis

Author: czerwonk

server/src/main.rs

@@ -106,7 +106,8 @@ async fn connect_to_redis(args: &Args) -> anyhow::Result<ConnectionManager> {
     let client = redis::Client::open(args.redis_dsn.clone())?;
     let config = ConnectionManagerConfig::default()
         .set_connection_timeout(Some(args.redis_connection_timeout))
-        .set_max_delay(args.redis_reconnection_max_delay);
+        .set_max_delay(args.redis_reconnection_max_delay)
+        .set_response_timeout(args.redis_response_timeout);
     let con = timeout(
         args.redis_connection_timeout,
         redis::aio::ConnectionManager::new_with_config(client, config),

server/src/options.rs

@@ -86,6 +86,15 @@ pub struct Args {
     )]
     pub redis_reconnection_max_delay: Duration,
 
+    /// The timeout for redis requets
+    #[arg(
+        long,
+        value_name = "REDIS_RESPONSE_TIMEOUT",
+        env = "HAKANAI_REDIS_RESPONSE_TIMEOUT",
+        value_parser = humantime::parse_duration
+    )]
+    pub redis_response_timeout: Option<Duration>,
+
     #[arg(
         long,
         value_name = "STATS_TTL",
@@ -328,6 +337,7 @@ mod tests {
             one_time_token_ttl: Duration::from_secs(604800),
             redis_connection_timeout: Duration::from_secs(10),
             redis_reconnection_max_delay: Duration::from_millis(10),
+            redis_response_timeout: None,
         }
     }