RFC: add logger that logs into browser console

CONTRIBUTING.md

@@ -195,6 +195,19 @@ Check the [OS dependencies](https://superset.incubator.apache.org/installation.h
     superset runserver -d
 
 
+### Logging to the browser console
+
+When debugging your application, you can have the server logs sent directly to the browser console:
+
+    superset runserver -d --console-log
+
+You can log anything to the browser console, including objects:
+
+    from superset import app
+    app.logger.error('An exception occurred!')
+    app.logger.info(form_data)
+
+
 ## Setting up the node / npm javascript environment
 
 `superset/assets` contains all npm-managed, front end assets.

dev-reqs.txt

@@ -10,5 +10,6 @@ pylint
 pyyaml
 redis
 statsd
+console_log
 # Also install everything we need to build Sphinx docs
 -r dev-reqs-for-docs.txt

setup.py

@@ -93,6 +93,7 @@ def get_git_sha():
     ],
     extras_require={
         'cors': ['Flask-Cors>=2.0.0'],
+        'console_log': ['console_log==0.2.10'],
     },
     tests_require=[
         'coverage',

superset/__init__.py

@@ -77,7 +77,9 @@ def get_js_manifest():
 if conf.get('SILENCE_FAB'):
     logging.getLogger('flask_appbuilder').setLevel(logging.ERROR)
 
-if not app.debug:
+if app.debug:
+    app.logger.setLevel(logging.DEBUG)
+else:
     # In production mode, add log handler to sys.stderr.
     app.logger.addHandler(logging.StreamHandler())
     app.logger.setLevel(logging.INFO)

superset/cli.py

@@ -14,6 +14,7 @@
 from flask_migrate import MigrateCommand
 from flask_script import Manager
 from pathlib2 import Path
+import werkzeug.serving
 import yaml
 
 from superset import app, db, dict_import_export_util, security, utils
@@ -31,11 +32,45 @@ def init():
     security.sync_role_definitions()
 
 
+def debug_run(app, port, use_reloader):
+    return app.run(
+        host='0.0.0.0',
+        port=int(port),
+        threaded=True,
+        debug=True,
+        use_reloader=use_reloader)
+
+
+def console_log_run(app, port, use_reloader):
+    from console_log import ConsoleLog
+    from gevent import pywsgi
+    from geventwebsocket.handler import WebSocketHandler
+
+    app.wsgi_app = ConsoleLog(app.wsgi_app, app.logger)
+
+    def run():
+        server = pywsgi.WSGIServer(
+            ('0.0.0.0', int(port)),
+            app,
+            handler_class=WebSocketHandler)
+        server.serve_forever()
+
+    if use_reloader:
+        from gevent import monkey
+        monkey.patch_all()
+        run = werkzeug.serving.run_with_reloader(run)
+
+    run()
+
+
 @manager.option(
     '-d', '--debug', action='store_true',
     help='Start the web server in debug mode')
 @manager.option(
-    '-n', '--no-reload', action='store_false', dest='no_reload',
+    '--console-log', action='store_true',
+    help='Create logger that logs to the browser console (implies -d)')
+@manager.option(
+    '-n', '--no-reload', action='store_false', dest='use_reloader',
     default=config.get('FLASK_USE_RELOAD'),
     help="Don't use the reloader in debug mode")
 @manager.option(
@@ -56,9 +91,9 @@ def init():
     help='Path to a UNIX socket as an alternative to address:port, e.g. '
          '/var/run/superset.sock. '
          'Will override the address and port values.')
-def runserver(debug, no_reload, address, port, timeout, workers, socket):
+def runserver(debug, console_log, use_reloader, address, port, timeout, workers, socket):
     """Starts a Superset web server."""
-    debug = debug or config.get('DEBUG')
+    debug = debug or config.get('DEBUG') or console_log
     if debug:
         print(Fore.BLUE + '-=' * 20)
         print(
@@ -67,12 +102,10 @@ def runserver(debug, no_reload, address, port, timeout, workers, socket):
             Fore.YELLOW + ' mode')
         print(Fore.BLUE + '-=' * 20)
         print(Style.RESET_ALL)
-        app.run(
-            host='0.0.0.0',
-            port=int(port),
-            threaded=True,
-            debug=True,
-            use_reloader=no_reload)
+        if console_log:
+            console_log_run(app, port, use_reloader)
+        else:
+            debug_run(app, port, use_reloader)
     else:
         addr_str = ' unix:{socket} ' if socket else' {address}:{port} '
         cmd = (

superset/views/core.py

@@ -747,8 +747,10 @@ def shortner(self):
         obj = models.Url(url=url)
         db.session.add(obj)
         db.session.commit()
-        return('http://{request.headers[Host]}/{directory}?r={obj.id}'.format(
-            request=request, directory=directory, obj=obj))
+        return Response(
+            'http://{request.headers[Host]}/{directory}?r={obj.id}'.format(
+                request=request, directory=directory, obj=obj),
+            mimetype='text/plain')
 
     @expose('/msg/')
     def msg(self):