Logging improvements, fix default not log generating

Author: dvingerh

pyinstalive/download.py

@@ -280,7 +280,7 @@ def update_stream_data(self, from_thread=False):
             stream_status = stream_heartbeat.get("broadcast_status")
             if stream_heartbeat.get("status") == "fail":
                 logger.separator()
-                logger.error('The livestream host has lost connection with Instagram.')
+                logger.error('The livestream host has possibly lost connection with Instagram.')
                 self.download_stop = True
                 self.downloader_object.stop()
                 return False

pyinstalive/helpers.py

@@ -52,7 +52,7 @@ def new_config():
                 config_file = open(globals.config.config_path, "w")
                 config_file.write(Constants.CONFIG_TEMPLATE.format(os.getcwd()).strip())
                 config_file.close()
-                logger.info("A new configuration file has been created.")
+                logger.info("A new configuration file has successfully been created.")
                 logger.separator()
                 return
             except Exception as e:
@@ -66,7 +66,7 @@ def new_config():
                 logger.separator()
     except Exception as e:
         logger.error("An error occurred: {}".format(str(e)))
-        logger.warn("If configuration file exists, manually create one using the following template:")
+        logger.warn("If no configuration file exists, manually create one using the following template:")
         logger.whiteline()
         logger.plain(Constants.CONFIG_TEMPLATE)
         logger.whiteline()

pyinstalive/startup.py

@@ -2,7 +2,7 @@
 import configparser
 import os
 import logging
-
+import sys
 
 from . import globals
 from . import logger
@@ -27,8 +27,9 @@ def validate_settings():
                 logger.warn("Falling back to default path: {:s}".format(globals.config.config_path), pre_config=True)
                 logger.separator(pre_config=True)
         elif not os.path.isfile(globals.config.config_path):
+            logger.banner(log_only=True)
             helpers.new_config()
-            validate_succeeded = False
+            sys.exit(0)
 
         if validate_succeeded:
             globals.config.config_path = os.path.realpath(globals.config.config_path)
@@ -131,21 +132,20 @@ def __init__(self, prog):
     parser.add_argument('-na', '--no-assemble', dest='no_assemble', action='store_true', help="Do not assemble the downloaded livestream segments into a video file. Overrides the configuration file setting.")
 
     globals.args, unknown_args = parser.parse_known_args()
+    validate_success = validate_settings()
 
     if unknown_args:
         logger.warn("The following unknown argument(s) were provided and will be ignored.", pre_config=True)
-        logger.warn('    ' + ' '.join(unknown_args), pre_config=True)
+        logger.warn('    ' + ' '.join(unknown_args))
         logger.separator(pre_config=True)
 
     if not any(vars(globals.args).values()):
         logger.error("No known arguments were provided.", pre_config=True)
         logger.separator(pre_config=True)
-        return
+        validate_success = False
 
-    validate_success = validate_settings()
     if globals.config.log_to_file:
         logger._log_to_file(None, pre_config=True)
-
 
     if validate_success:
         if globals.args.download or globals.args.download_following:
@@ -175,3 +175,4 @@ def __init__(self, prog):
             elif globals.args.info:
                 helpers.show_info()
             logger.separator()
+