From 60adcbed919afd5c85442775eca822fec43d816d Mon Sep 17 00:00:00 2001
From: Richard van der Hoff <1389908+richvdh@users.noreply.github.com>
Date: Tue, 31 Mar 2020 15:18:41 +0100
Subject: Fix "'NoneType' has no attribute start|stop" logcontext errors
 (#7181)

Fixes #7179.
---
 synapse/logging/context.py | 5 +++++
 1 file changed, 5 insertions(+)

(limited to 'synapse/logging')

diff --git a/synapse/logging/context.py b/synapse/logging/context.py
index a8eafb1c7c..3254d6a8df 100644
--- a/synapse/logging/context.py
+++ b/synapse/logging/context.py
@@ -539,6 +539,11 @@ def set_current_context(context: LoggingContextOrSentinel) -> LoggingContextOrSe
     Returns:
         The context that was previously active
     """
+    # everything blows up if we allow current_context to be set to None, so sanity-check
+    # that now.
+    if context is None:
+        raise TypeError("'context' argument may not be None")
+
     current = current_context()
 
     if current is not context:
-- 
cgit 1.5.1