implemeted debug mode. fixes #50

2 files changed, 17 insertions, 21 deletions

diff --git a/onionshare_gui/onionshare_gui.py b/onionshare_gui/onionshare_gui.py
index 5c427275..fda1df64 100644
--- a/onionshare_gui/onionshare_gui.py
+++ b/onionshare_gui/onionshare_gui.py
@@ -98,7 +98,7 @@ def main():
     filename = args.filename
     local_only = args.local_only
     stay_open = bool(args.stay_open)
-    debug = args.debug
+    debug = bool(args.debug)
 
     onionshare.set_stay_open(stay_open)
 
@@ -126,14 +126,15 @@ def main():
     webapp.onionshare = onionshare
     webapp.onionshare_port = onionshare_port
     webapp.filename = filename
+    webapp.qtapp = app
+    webapp.clipboard = app.clipboard()
+    webapp.stay_open = stay_open
     if not local_only:
         webapp.onion_host = onion_host
     else:
         webapp.onion_host = local_host
-    webapp.qtapp = app
-    webapp.clipboard = app.clipboard()
-    webapp.stay_open = stay_open
-    webapp.debug = debug
+    if debug:
+        webapp.debug_mode()
 
     # run the web app in a new thread
     webapp_port = onionshare.choose_port()
diff --git a/onionshare_gui/webapp.py b/onionshare_gui/webapp.py
index 11f62cd8..60583402 100644
--- a/onionshare_gui/webapp.py
+++ b/onionshare_gui/webapp.py
@@ -8,28 +8,23 @@ onion_host = None
 qtapp = None
 clipboard = None
 stay_open = None
-debug = None
 
 url = None
 
-# figure out this platform's temp dir
-if platform.system() == 'Windows':
-    temp_dir = os.environ['Temp'].replace('\\', '/')
-else:
-    temp_dir = '/tmp/'
+app = Flask(__name__, template_folder='./templates')
 
-# suppress output in windows
-if platform.system() == 'Windows':
-    sys.stdout = open('{0}/onionshare.stdout.log'.format(temp_dir), 'w')
-    sys.stderr = open('{0}/onionshare.stderr.log'.format(temp_dir), 'w')
+def debug_mode():
+    import logging
+    global app
 
-# log web errors to file
-import logging
-log_handler = logging.FileHandler('{0}/onionshare.web.log'.format(temp_dir))
-log_handler.setLevel(logging.WARNING)
+    if platform.system() == 'Windows':
+        temp_dir = os.environ['Temp'].replace('\\', '/')
+    else:
+        temp_dir = '/tmp/'
 
-app = Flask(__name__, template_folder='./templates')
-app.logger.addHandler(log_handler)
+    log_handler = logging.FileHandler('{0}/onionshare.web.log'.format(temp_dir))
+    log_handler.setLevel(logging.WARNING)
+    app.logger.addHandler(log_handler)
 
 @app.route("/")
 def index():