Produits Entr'ouvert » w.c.s.

Il y a des exceptions qui passent entre les mailles du filet, dans quixote/server/scgi_server.py il y a ce code pour la sérialisation d'une réponse dans la connexion SCGI:

        response = self.publisher.process(input, env)
        try:
            response.write(output)
            input.close()
            output.close()
            conn.close()
        except IOError, err:
            self.publisher.log("IOError while sending response " 
                               "ignored: %s" % err)

Si response.write(output) génère une exception (par exemple en essayant de renvoyer un header qui a une valeur unicode, on ne la verra pas. Le patch suivant rajoute encore un niveau pour rattraper les exceptions pour surveiller un peu ça.

--- a/wcs/qommon/scgi_server.py
+++ b/wcs/qommon/scgi_server.py
@@ -28,7 +28,7 @@ import quixote.server.scgi_server
 import sessions
 import cron
 import quixote.server.simple_server as simple_server
-
+import traceback

 class QommonHandler(quixote.server.scgi_server.QuixoteHandler):
     connection_limit = -1
@@ -36,7 +36,22 @@ class QommonHandler(quixote.server.scgi_server.QuixoteHandler):

     def handle_connection(self, conn):
         self.number_of_connection_handled = self.number_of_connection_handled + 1
-        quixote.server.scgi_server.QuixoteHandler.handle_connection(self, conn)
+        try:
+            quixote.server.scgi_server.QuixoteHandler.handle_connection(self, conn)
+        except Exception:
+            (exc_type, exc_value, tb) = sys.exc_info()
+            error_summary = traceback.format_exception_only(exc_type, exc_value)
+            error_summary = error_summary[0][0:-1] # de-listify and strip newline
+            plain_error_msg = self.publisher._generate_plaintext_error(None,
+                    None, exc_type, exc_value, tb)
+            try:
+                self.publisher.logger.log_internal_error(error_summary, plain_error_msg)
+            except socket.error:
+                # will happen if there is no mail server available and exceptions
+                # were configured to be mailed.
+                pass
+            raise
+
         # input, output and conn are closed, long running jobs could be done
         # here.
         self.publisher.response.process_after_jobs()

J'ai mis le patch sur la prod (auquo), sans le commiter pour l'instant. On va voir ce que ça donne.

Je pense que ne pas perdre des exception c'est assez important quand même, en général ça fait gagner du temps :) Maintenant on peut s'accorder qu'il serait bien de factoriser le code qui transforme les exceptions en mail plutôt que de le recopier.