From da869f1efe3f623a52dd5a432f984f7780ea6149 Mon Sep 17 00:00:00 2001
From: Duda Dornelles <ddornell@thoughtworks.com>
Date: Thu, 6 Nov 2014 14:56:57 -0200
Subject: improving log and logging to file on functional tests

---
 service/pixelated/config/app_factory.py |  6 ++----
 service/pixelated/runserver.py          | 30 +++++++++++++++++-------------
 2 files changed, 19 insertions(+), 17 deletions(-)

(limited to 'service/pixelated')

diff --git a/service/pixelated/config/app_factory.py b/service/pixelated/config/app_factory.py
index c479a913..8b256550 100644
--- a/service/pixelated/config/app_factory.py
+++ b/service/pixelated/config/app_factory.py
@@ -33,7 +33,9 @@ from leap.common.events import (
     register,
     events_pb2 as proto
 )
+from twisted.web.iweb import IAccessLogFormatter
 from twisted.web.server import Site
+from zope.interface import provider
 
 
 def init_index_and_remove_dupes(querier, search_engine, mail_service):
@@ -130,10 +132,6 @@ def init_app(app):
 
 
 def create_app(app):
-    from twisted.python import log
-    import sys
-    log.startLogging(sys.stdout)
-
     reactor.listenTCP(3333, Site(app.resource()), interface='localhost')
     reactor.callWhenRunning(lambda: init_app(app))
     reactor.run()
diff --git a/service/pixelated/runserver.py b/service/pixelated/runserver.py
index 832008ad..a2e37ec0 100644
--- a/service/pixelated/runserver.py
+++ b/service/pixelated/runserver.py
@@ -19,10 +19,13 @@ import sys
 import logging
 import json
 from klein import Klein
+from twisted.python.log import ILogObserver
 
 klein_app = Klein()
 
 import ConfigParser
+from twisted.python import log
+import sys
 from leap.common.events import server as events_server
 from pixelated.config import app_factory
 import pixelated.config.args as input_args
@@ -71,19 +74,20 @@ def fetch_credentials_from_dispatcher(filename):
 
 def setup_debugger(enabled):
     debug_enabled = enabled or os.environ.get('DEBUG', False)
-    logging.basicConfig(level=logging.DEBUG,
-                        format='%(asctime)s %(name)-12s %(levelname)-8s %(message)s',
-                        datefmt='%m-%d %H:%M',
-                        filename='/tmp/leap.log',
-                        filemode='w')  # define a Handler which writes INFO messages or higher to the sys.stderr
-    console = logging.StreamHandler()
-    console.setLevel(logging.DEBUG)
-    # set a format which is simpler for console use
-    formatter = logging.Formatter('%(name)-12s: %(levelname)-8s %(message)s')
-    # tell the handler to use this format
-    console.setFormatter(formatter)
-    # add the handler to the root logger
-    logging.getLogger('').addHandler(console)
+    log.startLogging(sys.stdout)
+
+    if debug_enabled:
+        logging.basicConfig(level=logging.DEBUG,
+                            format='%(asctime)s %(name)-12s %(levelname)-8s %(message)s',
+                            datefmt='%m-%d %H:%M',
+                            filename='/tmp/leap.log',
+                            filemode='w')  # define a Handler which writes INFO messages or higher to the sys.stderr
+
+        console = logging.StreamHandler()
+        console.setLevel(logging.DEBUG)
+        formatter = logging.Formatter('%(name)-12s: %(levelname)-8s %(message)s')
+        console.setFormatter(formatter)
+        logging.getLogger('').addHandler(console)
 
     return debug_enabled
 
-- 
cgit v1.2.3