bugfixes

src/webdrop_bridge/core/drag_interceptor.py

@@ -3,8 +3,8 @@
 from pathlib import Path
 from typing import List, Optional
 
-from PySide6.QtCore import Qt, Signal
+from PySide6.QtCore import QMimeData, Qt, QUrl, Signal
-from PySide6.QtGui import QDrag, QMimeData
+from PySide6.QtGui import QDrag
 from PySide6.QtWidgets import QWidget
 
 from webdrop_bridge.core.validator import PathValidator, ValidationError
@@ -102,5 +102,3 @@ class DragInterceptor(QWidget):
             return False
 
 
-# Import QUrl here to avoid circular import at module level
-from PySide6.QtCore import QUrl  # noqa: E402, F401

src/webdrop_bridge/main.py

@@ -25,8 +25,8 @@ def main() -> int:
 
         # Set up logging
         log_file = None
-        if config.log_file_path:
+        if config.log_file:
-            log_file = Path(config.log_file_path)
+            log_file = config.log_file
 
         setup_logging(
             name="webdrop_bridge",

src/webdrop_bridge/ui/main_window.py

@@ -151,13 +151,13 @@ class MainWindow(QMainWindow):
 
         # Back button
         back_action = self.web_view.pageAction(
-            self.web_view.WebAction.Back
+            self.web_view.WebAction.Back  # type: ignore
         )
         toolbar.addAction(back_action)
 
         # Forward button
         forward_action = self.web_view.pageAction(
-            self.web_view.WebAction.Forward
+            self.web_view.WebAction.Forward  # type: ignore
         )
         toolbar.addAction(forward_action)
 
@@ -170,7 +170,7 @@ class MainWindow(QMainWindow):
 
         # Refresh button
         refresh_action = self.web_view.pageAction(
-            self.web_view.WebAction.Reload
+            self.web_view.WebAction.Reload  # type: ignore
         )
         toolbar.addAction(refresh_action)
 

src/webdrop_bridge/ui/restricted_web_view.py

@@ -39,7 +39,7 @@ class RestrictedWebEngineView(QWebEngineView):
         Args:
             request: Navigation request to process
         """
-        url = request.url
+        url = request.url()
 
         # If no restrictions, allow all URLs
         if not self.allowed_urls:

tests/unit/test_logging.py

@@ -1,6 +1,7 @@
 """Unit tests for logging module."""
 
 import logging
+import logging.handlers
 from pathlib import Path
 from tempfile import TemporaryDirectory