Merge pull request #203 from Nitrokey/fix-dev-remove

Fix/catch device removal error from fido2 lib
Nitrokey · Jan 25, 2024 · 344a9a4 · 344a9a4
2 parents 90ce8c8 + c6015af
commit 344a9a4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/nitrokeyapp/gui.py b/nitrokeyapp/gui.py
@@ -247,7 +247,7 @@ def detect_removed_devices(self) -> None:
         if self.devices:
             try:
                 nk3_list = [device.uuid() for device in Nitrokey3Device.list()]
-            except OSError as e:
+            except Exception as e:
                 logger.info(f"detect removed failed: {e}")
                 return
             logger.info(f"list nk3: {nk3_list}")

diff --git a/nitrokeyapp/secrets_tab/worker.py b/nitrokeyapp/secrets_tab/worker.py
@@ -70,7 +70,7 @@ def run(self) -> None:
                     # TODO: catch a more specific exception
                     pass
         except Exception as e:
-            logger.info("check device job failed: ", repr(e))
+            logger.info(f"check device job failed: {e}")
             compatible = False
 
         self.device_checked.emit(compatible)