From 92571d961f8ec0ce72c0c40433e2487032643060 Mon Sep 17 00:00:00 2001
From: Jonas Platte <jplatte+git@posteo.de>
Date: Thu, 3 Feb 2022 19:54:29 +0100
Subject: [PATCH] Remove mutation from default_config and set default log_level
 to off

---
 Cargo.lock  |  1 +
 Cargo.toml  |  1 +
 src/main.rs | 37 ++++++++++++++++++-------------------
 3 files changed, 20 insertions(+), 19 deletions(-)

diff --git a/Cargo.lock b/Cargo.lock
index 6dbb6586..85487715 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -317,6 +317,7 @@ dependencies = [
  "image",
  "jsonwebtoken",
  "lru-cache",
+ "maplit",
  "num_cpus",
  "opentelemetry",
  "opentelemetry-jaeger",
diff --git a/Cargo.toml b/Cargo.toml
index 05782e7c..fe60f6e9 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -84,6 +84,7 @@ hmac = "0.11.0"
 sha-1 = "0.9.8"
 # used for conduit's CLI and admin room command parsing
 clap = { version = "3.0.10", default-features = false, features = ["std", "derive"] }
+maplit = "1.0.2"
 
 [target.'cfg(not(target_env = "msvc"))'.dependencies]
 tikv-jemalloc-ctl = { version = "0.4.2", features = ['use_std'] }
diff --git a/src/main.rs b/src/main.rs
index 5fda5737..b3e85c95 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -9,6 +9,7 @@
 
 use std::sync::Arc;
 
+use maplit::hashset;
 use opentelemetry::trace::{FutureExt, Tracer};
 use rocket::{
     catch, catchers,
@@ -292,28 +293,26 @@ fn bad_json_catcher() -> Result<()> {
 }
 
 fn default_config() -> rocket::Config {
-    let mut config = rocket::Config::release_default();
+    use rocket::config::{LogLevel, Shutdown, Sig};
 
-    {
-        let mut shutdown = &mut config.shutdown;
+    rocket::Config {
+        // Disable rocket's logging to get only tracing-subscriber's log output
+        log_level: LogLevel::Off,
+        shutdown: Shutdown {
+            // Once shutdown is triggered, this is the amount of seconds before rocket
+            // will forcefully start shutting down connections, this gives enough time to /sync
+            // requests and the like (which havent gotten the memo, somehow) to still complete gracefully.
+            grace: 35,
 
-        #[cfg(unix)]
-        {
-            use rocket::config::Sig;
+            // After the grace period, rocket starts shutting down connections, and waits at least this
+            // many seconds before forcefully shutting all of them down.
+            mercy: 10,
 
-            shutdown.signals.insert(Sig::Term);
-            shutdown.signals.insert(Sig::Int);
-        }
+            #[cfg(unix)]
+            signals: hashset![Sig::Term, Sig::Int],
 
-        // Once shutdown is triggered, this is the amount of seconds before rocket
-        // will forcefully start shutting down connections, this gives enough time to /sync
-        // requests and the like (which havent gotten the memo, somehow) to still complete gracefully.
-        shutdown.grace = 35;
-
-        // After the grace period, rocket starts shutting down connections, and waits at least this
-        // many seconds before forcefully shutting all of them down.
-        shutdown.mercy = 10;
+            ..Shutdown::default()
+        },
+        ..rocket::Config::release_default()
     }
-
-    config
 }