Don't passthrough_errors unless instructed.

Fix pallets#2005 Revert pallets#1679 and pallets#1996
untitaker · Sep 6, 2016 · 9c2e2e9 · 9c2e2e9
1 parent ccd02bf
commit 9c2e2e9
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 6 deletions.
diff --git a/flask/app.py b/flask/app.py
@@ -838,8 +838,6 @@ def run(self, host=None, port=None, debug=None, **options):
             self.debug = bool(debug)
         options.setdefault('use_reloader', self.debug)
         options.setdefault('use_debugger', self.debug)
-        options.setdefault('passthrough_errors',
-                           self.config['PROPAGATE_EXCEPTIONS'])
         try:
             run_simple(host, port, self, **options)
         finally:

diff --git a/flask/cli.py b/flask/cli.py
@@ -429,8 +429,7 @@ def run_command(info, host, port, reload, debugger, eager_loading,
             print(' * Forcing debug mode %s' % (debug and 'on' or 'off'))
 
     run_simple(host, port, app, use_reloader=reload,
-               use_debugger=debugger, threaded=with_threads,
-               passthrough_errors=app.config['PROPAGATE_EXCEPTIONS'])
+               use_debugger=debugger, threaded=with_threads)
 
 
 @click.command('shell', short_help='Runs a shell in the app context.')

diff --git a/tests/test_basic.py b/tests/test_basic.py
@@ -1268,8 +1268,6 @@ def run_simple_mock(*args, **kwargs):
     monkeypatch.setattr(werkzeug.serving, 'run_simple', run_simple_mock)
     app.config['PROPAGATE_EXCEPTIONS'] = propagate_exceptions
     app.run(debug=debug, use_debugger=use_debugger, use_reloader=use_reloader)
-    # make sure werkzeug passes errors through if PROPAGATE_EXCEPTIONS
-    assert rv['passthrough_errors'] == propagate_exceptions
 
 
 def test_max_content_length():