use default path when --log path invalidated (#820)

* add log path check and use default path when --log path invalide

* add file type check

lyrebird/log.py

@@ -53,7 +53,7 @@ def make_stream_handler():
 
 def make_file_handler(_log_path=None):
     # Set default value
-    if not _log_path:
+    if not check_path(_log_path):
         _log_path = DEFAULT_LOG_PATH
 
     # Add pid and thread name in log file
@@ -67,6 +67,16 @@ def make_file_handler(_log_path=None):
     return file_handler
 
 
+def check_path(path):
+    if not path:
+        return False
+    if not os.path.exists(os.path.dirname(path)):
+        return False
+    if os.path.isdir(path) or path.split('.')[-1] != 'log':
+        return False
+    return True
+
+
 def init(config, log_queue = None):
     global LOGGER_INITED
     if LOGGER_INITED:
@@ -141,6 +151,9 @@ def run(self, msg_queue, config, log_queue, *args, **kwargs):
             _logger = logging.getLogger(_logger_name)
             _logger.addHandler(file_handler)
             _logger.setLevel(logger_level)
+
+        if log_path and not check_path(log_path):
+            lyrebird_logger.warning(f'Illegal log path: {log_path}, log file path have changed to the default path: {DEFAULT_LOG_PATH}')
 
         while True:
             try:

lyrebird/version.py

@@ -1,3 +1,3 @@
-IVERSION = (2, 23, 4)
+IVERSION = (2, 23, 5)
 VERSION = ".".join(str(i) for i in IVERSION)
 LYREBIRD = "Lyrebird " + VERSION