Merge pull request #4732 from hypothesis/fix-double-reporting

Fix double-reporting of DB session warning to Sentry
hypothesis · Jan 8, 2018 · bcae58f · bcae58f
2 parents 40213d9 + 133ca81
commit bcae58f
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/h/db/__init__.py b/h/db/__init__.py
@@ -108,11 +108,11 @@ def _session(request):
     def close_the_sqlalchemy_session(request):
         changes = tracker.uncommitted_changes() if tracker else []
         if changes:
-            msg = 'closing a session with uncommitted changes'
-            request.sentry.captureMessage(msg, stack=True, extra={
+            msg = 'closing a session with uncommitted changes %s'
+            log.warn(msg, changes, extra={
+                'stack': True,
                 'changes': changes,
             })
-            log.warn('{} {}'.format(msg, changes))
         session.close()
 
     return session