From 6f679bc1727d7c5448116e4bf1c60388afa5dceb Mon Sep 17 00:00:00 2001
From: Klaas van Schelven <klaas@vanschelven.com>
Date: Fri, 13 Sep 2024 09:52:06 +0200
Subject: [PATCH] Settings 'TODOs'; reflect current thinking on them

---
 bugsink/settings/default.py | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/bugsink/settings/default.py b/bugsink/settings/default.py
index ef039fe..fe9885f 100644
--- a/bugsink/settings/default.py
+++ b/bugsink/settings/default.py
@@ -50,7 +50,7 @@ INSTALLED_APPS = [
     'django.contrib.staticfiles',
 
     'tailwind',  # As currently set up, this is also needed in production (templatetags)
-    'admin_auto_filters',  # TODO: decide whether 'admin.py' is useful in production too.
+    'admin_auto_filters',
 
     'users',
     'theme',
@@ -84,7 +84,10 @@ MIDDLEWARE = [
     'django.contrib.messages.middleware.MessageMiddleware',
     'django.middleware.clickjacking.XFrameOptionsMiddleware',
 
-    'bugsink.middleware.PerformanceStatsMiddleware',  # TODO decide whether this is useful in production too.
+    # NOTE: _most_ useful while building Bugsink; in the recommended production setup the timings/counts of this
+    # middleware are not logged to a visible location; and this feature is undocumented. However, it _could_ prove
+    # useful in such contexts too, so I'm not going to put it behind a conditional.
+    'bugsink.middleware.PerformanceStatsMiddleware',
 ]
 
 ROOT_URLCONF = 'bugsink.urls'