runserver: Debugging now defaults to whatever the app's configuration says

or True if that's None. Also, print a warning when debugging is on.

Author: smurfix

docs/index.rst

@@ -404,9 +404,12 @@ See the :ref:`api` below for details on the various prompt functions.
 Default commands
 ----------------
 
+runserver
++++++++++
+
 **Flask-Script** has a couple of ready commands you can add and customise: ``Server`` and ``Shell``.
 
-The ``Server`` command runs the **Flask** development server. It takes an optional ``port`` argument (default **5000**)::
+The ``Server`` command runs the **Flask** development server.
 
     from flask.ext.script import Server, Manager
     from myapp import create_app
@@ -427,6 +430,26 @@ The ``Server`` command has a number of command-line arguments - run ``python man
 
 Needless to say the development server is not intended for production use.
 
+*New in version 2.0.5*
+
+The most common use-case for ``runserver`` is to run a debug server for
+investigating problems. Therefore the default, if it is *not* set in the
+configuration file, is to enable debugging and auto-reloading.
+
+Unfortunately, Flask currently (as of May 2014) defaults to set the DEBUG
+configuration parameter to ``False``. Until this is changed, you can
+safely add ``DEFAULT=None`` to your Flask configuration. Flask-Script's
+``runserver`` will then turn on debugging, but everything else will treat
+it as being turned off.
+
+To prevent misunderstandings -- after all, debug mode is a serious security
+hole --, a warning is printed when Flask-Script treats a ``None`` default
+value as if it were set to ``True``. You can turn on debugging explicitly
+to get rid of this warning.
+
+shell
++++++
+
 The ``Shell`` command starts a Python shell. You can pass in a ``make_context`` argument, which must be a ``callable`` returning a ``dict``. By default, this is just a dict returning the your Flask application instance::
 
     from flask.ext.script import Shell, Manager

flask_script/commands.py

@@ -2,6 +2,7 @@
 from __future__ import absolute_import
 
 import os
+import sys
 import code
 import warnings
 import string
@@ -407,6 +408,10 @@ def __call__(self, app, host, port, use_debugger, use_reloader,
 
         if use_debugger is None:
             use_debugger = app.debug
+            if use_debugger is None:
+                use_debugger = True
+                if sys.stderr.isatty():
+                    print("Debugging is on. DANGER: Do not allow random users to connect to this server.", file=sys.stderr)
         if use_reloader is None:
             use_reloader = app.debug
         app.run(host=host,

setup.py

@@ -27,7 +27,7 @@
 
 setup(
     name='Flask-Script',
-    version='2.0.4',
+    version='2.0.5',
     url='http://github.com/smurfix/flask-script',
 	download_url = 'https://github.com/smurfix/flask-script/tarball/v2.0.3',
     license='BSD',