From bfb6fdee924cc60dfdba8ede609a206ca6982454 Mon Sep 17 00:00:00 2001
From: Frederick Muriuki Muriithi
Date: Mon, 5 Jun 2023 06:41:43 +0300
Subject: Logging: Don't allow getting user info to stop application starting

In certain scenarios (probably due to host contamination) the effective UID of
the running application is not the expected "genenetwork" user, rather, it is
a user on the host system, which leads to the error:

    KeyError: 'getpwuid(): uid not found: 1000'

This commit prevents the application from failing in such a case, but still
logs out such weirdness.
---
 gn3/auth/db.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'gn3/auth/db.py')

diff --git a/gn3/auth/db.py b/gn3/auth/db.py
index 7c20bf7..2ba6619 100644
--- a/gn3/auth/db.py
+++ b/gn3/auth/db.py
@@ -48,7 +48,7 @@ class DbCursor(Protocol):
 @contextlib.contextmanager
 def connection(db_path: str, row_factory: Callable = sqlite3.Row) -> Iterator[DbConnection]:
     """Create the connection to the auth database."""
-    logging.debug("SQLite3 DB Path: '%s'." % (db_path,))
+    logging.debug("SQLite3 DB Path: '%s'.", db_path)
     conn = sqlite3.connect(db_path)
     conn.row_factory = row_factory
     conn.set_trace_callback(logging.debug)
-- 
cgit v1.2.3