Stop sharing unsafely a context dict across threads.

Panels that need to share data with other panels shall do it through the
record_stats / get_stats API. Statistics are automatically pushed to the
template context.

Fix #450.

2 files changed, 5 insertions, 15 deletions

diff --git a/debug_toolbar/panels/__init__.py b/debug_toolbar/panels/__init__.py
index 1b2a767..178ea6f 100644
--- a/debug_toolbar/panels/__init__.py
+++ b/debug_toolbar/panels/__init__.py
@@ -9,16 +9,10 @@ class Panel(object):
     """
     Base class for panels.
     """
-
-    # We'll maintain a local context instance so we can expose our template
-    # context variables to panels which need them. (But see issue #450.)
-    context = {}
-
-    # Private panel methods
-
-    def __init__(self, toolbar, context={}):
+    def __init__(self, toolbar):
         self.toolbar = toolbar
-        self.context.update(context)
+
+    # Private panel properties
 
     @property
     def panel_id(self):
@@ -81,9 +75,7 @@ class Panel(object):
         template's context.
         """
         if self.has_content:
-            context = self.context.copy()
-            context.update(self.get_stats())
-            return render_to_string(self.template, context)
+            return render_to_string(self.template, self.get_stats())
 
     # URLs for panel-specific views
 
diff --git a/debug_toolbar/panels/timer.py b/debug_toolbar/panels/timer.py
index 6df4d4d..6b36bf9 100644
--- a/debug_toolbar/panels/timer.py
+++ b/debug_toolbar/panels/timer.py
@@ -45,9 +45,7 @@ class TimerPanel(Panel):
             (_('Elapsed time'), _('%(total_time)0.3f msec') % stats),
             (_('Context switches'), _('%(vcsw)d voluntary, %(ivcsw)d involuntary') % stats),
         )
-        context = self.context.copy()
-        context.update({'rows': rows})
-        return render_to_string(self.template, context)
+        return render_to_string(self.template, {'rows': rows})
 
     def process_request(self, request):
         self._start_time = time.time()