misc: remove support for sentry (#35913)

help/fr/settings-env.page

@@ -32,16 +32,6 @@ d'environnement.
   </p>
 </section>
 
-<section>
-  <title>Accès au serveur Sentry</title>
-
-  <p>
-   Également accessible depuis l'écran de paramétrage des options de debug, l'adresse (DSN) du serveur Sentry, pour la notification des erreurs
-   applicatives, peut aussi être placée dans la variable d'environnement
-   <code>SENTRY_DSN</code>.
-  </p>
-</section>
-
 <section>
   <title>Fréquence de vérification des expirations des sauts</title>
 

wcs/qommon/admin/settings.py

@@ -111,11 +111,6 @@ class SettingsDirectory(AccessControlled, Directory):
                             (str('html'), _('Display as HTML')) ])
         form.add(CheckboxWidget, 'logger', title=_('Logger'),
                 value=debug_cfg.get('logger', True))
-        if get_publisher().can_sentry():
-            form.add(StringWidget, 'sentry_dsn', title=_('Sentry DSN'),
-                    value=debug_cfg.get('sentry_dsn', ''))
-        else:
-            form.add_hidden('sentry_dsn', debug_cfg.get('sentry_dsn', ''))
         form.add(CheckboxWidget, 'debug_mode', title = _('Enable debug mode'),
                 value = debug_cfg.get('debug_mode', False))
         form.add(StringWidget, 'mail_redirection', title = _('Mail redirection'),
@@ -137,5 +132,5 @@ class SettingsDirectory(AccessControlled, Directory):
         else:
             cfg_submit(form, 'debug', ('error_email', 'display_exceptions',
                                        'logger', 'debug_mode',
-                                       'mail_redirection', 'sentry_dsn'))
+                                       'mail_redirection'))
             return redirect('.')

wcs/qommon/publisher.py

@@ -40,11 +40,6 @@ import requests
 from StringIO import StringIO
 import xml.etree.ElementTree as ET
 
-try:
-    import raven
-except ImportError:
-    raven = None
-
 from django.conf import settings
 from django.http import Http404
 from django.utils import translation
@@ -262,9 +257,6 @@ class QommonPublisher(Publisher, object):
                         exc_type, exc_value,
                         tb))
 
-        self.notify_sentry(exc_tuple, request=self.get_request(),
-                context=context)
-
         self.log_internal_error(error_summary, plain_error_msg, record=True)
 
     def log_internal_error(self, error_summary, plain_error_msg, record=False):
@@ -278,30 +270,6 @@ class QommonPublisher(Publisher, object):
             # this could happen on file descriptor exhaustion
             pass
 
-    def can_sentry(self):
-        return (raven is not None)
-
-    def notify_sentry(self, exc_tuple, request=None, context=None):
-        if not self.can_sentry():
-            return
-
-        debug_cfg = self.cfg.get('debug', {})
-        sentry_dsn = debug_cfg.get('sentry_dsn') or os.environ.get('SENTRY_DSN')
-        if not sentry_dsn:
-            return
-
-        from raven.transport.requests import RequestsHTTPTransport
-        client = raven.Client(sentry_dsn, transport=RequestsHTTPTransport)
-        extra = {}
-        tags = {}
-        if context:
-            extra['context'] = context
-        if request:
-            extra['request'] = request.dump()
-            tags['url'] = request.get_url()
-
-        client.captureException(exc_tuple, extra=extra, tags=tags)
-
     def finish_successful_request(self):
         if not self.get_request().ignore_session:
             self.session_manager.finish_successful_request()
@@ -318,8 +286,6 @@ class QommonPublisher(Publisher, object):
 
         (exc_type, exc_value, tb) = sys.exc_info()
 
-        self.notify_sentry((exc_type, exc_value, tb), request)
-
         if exc_type is NotImplementedError:
             get_response().set_header('Content-Type', 'text/html') # set back content-type
             return template.error_page(