Merge pull request #57 from PW-Sat2/fixes

Major Fixes

Author: begreeen

.gitignore

@@ -74,6 +74,8 @@ target/
 # pyenv
 .python-version
 
+.vscode
+
 # celery beat schedule file
 celerybeat-schedule
 

app/from_file_to_gui_queue.py

@@ -24,7 +24,7 @@ def load_from_file(self):
             thread.start()
             thread.join()
         except:
-            self.logger.log(logging.DEBUG, "Load file exception")
+            self.logger.log(logging.ERROR, "Load file exception")
 
 
 class FromFileToGuiQueueThread(Thread):
@@ -42,5 +42,5 @@ def run(self):
             for packet in packets:
                 self.gui_queue.append(packet)
         except TypeError:
-            self.logger.log(logging.DEBUG, "Empty packets list")
+            self.logger.log(logging.ERROR, "Empty packets list")
         self.logger.log(logging.DEBUG, "Finished FromFileToGuiQueueThread")

app/gui_connection_state.py

@@ -24,5 +24,8 @@ def check(self):
     def run(self):
         self.check()
         while not self.stop_event.wait(30):
-            self.check()
+            try:
+                self.check()
+            except Exception as e:
+                self.logger.error("Major Exception in Check Auth hread", exc_info=e)
         self.logger.log(logging.DEBUG, "Finished CheckAuthThread")

app/load_credentials_file.py

@@ -46,5 +46,5 @@ def load_with_dialog(config):
                     logger.log(logging.DEBUG, "Wrong file")
                     return WrongOrEmptyCredentials
         except:
-            logger.log(logging.DEBUG, "Load file exception")
+            logger.log(logging.ERROR, "Load file exception")
             return UnknownError

app/queue_to_signal.py

@@ -21,4 +21,6 @@ def run(self):
                 self.item_ready.emit(widget_item)
             except IndexError:
                 time.sleep(1)
+            except Exception as e:
+                self.logger.error("Major Exception in GetFromQueueToSignalThread", exc_info=e)
         self.logger.log(logging.DEBUG, "Finished GetFromQueueToSignalThread")

app/receive_distribute.py

@@ -28,8 +28,11 @@ def connect_and_get_packet(self):
 
     def run(self):
         while not self.stop_event.wait(0):
-            packet = self.connect_and_get_packet()
-            if packet is not None:
-                self.gui_queue.append(packet)
-                self.file_queue.put(packet)
+            try:
+                packet = self.connect_and_get_packet()
+                if packet is not None:
+                    self.gui_queue.append(packet)
+                    self.file_queue.put(packet)
+            except Exception as e:
+                self.logger.error("Major Exception in Receive Distribute Thread", exc_info=e)
         self.logger.log(logging.DEBUG, "Finished ReceiveDistribute")

app/save_frames_file.py

@@ -43,4 +43,6 @@ def run(self):
                 writer.save(data)
             except Queue.Empty:
                 pass
+            except Exception as e:
+                self.logger.error("Major Exception in Save Frames Thread", exc_info=e)
         self.logger.log(logging.DEBUG, "Finished ReceiveDistribute")

app/setup_log.py

@@ -37,7 +37,9 @@ def setup_log(debug):
 
     if debug:
         root_logger_std.setLevel(logging.DEBUG)
+        root_logger_file.setLevel(logging.DEBUG)
     else:
         root_logger_std.setLevel(logging.INFO)
+        root_logger_file.setLevel(logging.INFO)
 
-    root_logger_file.setLevel(logging.DEBUG)
+    

app/update.py

@@ -35,7 +35,7 @@ def download_version_file(self):
                 got_version_file = True
             except Exception as e:
                 got_version_file = False
-                self.logger.log(logging.DEBUG, "Error in getting version file: {0}".format(e))
+                self.logger.log(logging.ERROR, "Error in getting version file: {0}".format(e))
                 time.sleep(10)
         self.logger.log(logging.DEBUG, "Got version file")
 
@@ -72,7 +72,10 @@ def msgbtn(self, response):
             self.logger.log(logging.DEBUG, "Web browser opened")
 
     def run(self):
-        self.download_version_file()
-        self.load_new_version_desc()
-        self.is_new_version_available()
+        try:
+            self.download_version_file()
+            self.load_new_version_desc()
+            self.is_new_version_available()
+        except Exception as e:
+            self.logger.error("Major Exception in Updater", exc_info=e)
         self.logger.log(logging.DEBUG, "Finished Updater")

app/upload_cloud.py

@@ -34,14 +34,17 @@ def run(self):
                         self.cloud_tx_queue.appendleft(data)
                         if data.upload_status is False:
                             self.error_queue.appendleft(data)
-                            self.logger.log(logging.DEBUG, "added to error queue because of credentials")
-                    except requests.ConnectionError, requests.ReadTimeout:
+                            self.logger.log(logging.WARNING, "added to error queue because of credentials")
+                    except (requests.ConnectionError, requests.ReadTimeout):
                         data.upload_status = False
                         self.error_queue.appendleft(data)
                         self.cloud_tx_queue.appendleft(data)
-                        self.logger.log(logging.DEBUG, "added to error queue because of internet")
+                        self.logger.log(logging.WARNING, "added to error queue because of internet")
                 except IndexError:
                     time.sleep(1)
+                except Exception as e:
+                    self.logger.error("Major Exception in Upload Thread", exc_info=e)
+
         self.logger.log(logging.DEBUG, "Finished UploadCloud")
 
 
@@ -70,11 +73,13 @@ def run(self):
                 self.cloud_tx_queue.append(data)
                 if data.upload_status is False:
                     self.error_queue.appendleft(data)
-                    print "again added to error queue"
+                    self.logger.log(logging.WARNING, "again added to error queue")
             except IndexError:
                 time.sleep(1)
-            except requests.ConnectionError:
+            except (requests.ConnectionError, requests.ReadTimeout):
                 self.error_queue.appendleft(data)
-                print "again added to error queue"
+                self.logger.log(logging.WARNING, "again added to error queue:")
+            except Exception as e:
+                self.logger.error("Major Exception in UploadCloudError Thread", exc_info=e)
 
         self.logger.log(logging.DEBUG, "Finished UploadCloudError")