summary refs log tree commit diff
path: root/synapse
diff options
context:
space:
mode:
authorErik Johnston <erik@matrix.org>2019-02-13 16:14:37 +0000
committerErik Johnston <erik@matrix.org>2019-02-13 16:15:11 +0000
commit6cb415b63fd58ab253f8519f725a581a0e15044e (patch)
tree3643762ec18d393659b2fdd4492c4f52987f89b3 /synapse
parentNewsfile (diff)
downloadsynapse-6cb415b63fd58ab253f8519f725a581a0e15044e.tar.xz
Fixup comments and add warning
Diffstat (limited to 'synapse')
-rw-r--r--synapse/app/_base.py6
-rw-r--r--synapse/config/metrics.py9
2 files changed, 10 insertions, 5 deletions
diff --git a/synapse/app/_base.py b/synapse/app/_base.py
index 482f0e22ea..0284151d0f 100644
--- a/synapse/app/_base.py
+++ b/synapse/app/_base.py
@@ -269,7 +269,7 @@ def start(hs, listeners=None):
         hs.start_listening(listeners)
         hs.get_datastore().start_profiling()
 
-        setup_sentry_io(hs)
+        setup_sentry(hs)
     except Exception:
         traceback.print_exc(file=sys.stderr)
         reactor = hs.get_reactor()
@@ -278,8 +278,8 @@ def start(hs, listeners=None):
         sys.exit(1)
 
 
-def setup_sentry_io(hs):
-    """Enable sentry.io integration, if enabled in configuration
+def setup_sentry(hs):
+    """Enable sentry integration, if enabled in configuration
 
     Args:
         hs (synapse.server.HomeServer)
diff --git a/synapse/config/metrics.py b/synapse/config/metrics.py
index 7bab8b0b2b..185b895a24 100644
--- a/synapse/config/metrics.py
+++ b/synapse/config/metrics.py
@@ -16,7 +16,7 @@
 from ._base import Config, ConfigError
 
 MISSING_SENTRY = (
-    """Missing sentry_sdk library. This is required for enable sentry.io
+    """Missing sentry_sdk library. This is required for enable sentry
     integration.
 
     Install by running:
@@ -48,7 +48,12 @@ class MetricsConfig(Config):
         # Enable collection and rendering of performance metrics
         enable_metrics: False
 
-        # Enable sentry.io integration
+        # Enable sentry integration
+        # NOTE: While attempts are made to ensure that the logs don't contain
+        # any sensitive information, this cannot be guaranteed. By enabling
+        # this option the sentry server may therefore receive sensitive
+        # information, and it in turn may then diseminate sensitive information
+        # through insecure notification channels if so configured.
         #sentry:
         #    dsn: "..."
         """