[SinghCod3r]

Overview

This PR introduces Prototype of a dedicated CUPS background worker integrated directly into authconn.Connection. It safely moves the blocking pycups operation getServerPPD() off the GTK main thread, resolving severe UI freezes during driverless PPD validation.

Importantly, this is achieved while completely preserving the existing authconn semantics (authentication, polkit integration, retries, server-error handling, and operation bookkeeping).

Key Changes

• Threaded authconn Worker:
  • Implemented _worker_loop and _execute_in_worker() in authconn.py.
  • Blocking CUPS operations are routed to a dedicated background thread with its own isolated cups.Connection.
  • The GTK main thread continues pumping the GLib event loop while waiting, ensuring the UI (e.g., Cancel buttons, Spinners) remains fully responsive.
• Wizard Lifecycle Race Fix (newprinter.py):
  • Because the UI remains responsive, users can cancel the wizard while getServerPPD() is executing in the background.
  • Added explicit getattr(self, 'device', None) is None safety checks immediately after async validation returns. If the wizard was destroyed/cancelled via on_NPCancel(), the late worker result is safely abandoned, preventing hard AttributeError: 'NoneType' object has no attribute 'driverless' crashes.
• Deterministic Regression Testing:
  • Added test_regression_getServerPPD_cancellation_real_v2_race to test_newprinter.py.
  • This test uses deterministic threading.Event synchronization to orchestrate the exact production race: it blocks the worker, triggers a genuine on_NPCancel() callback in the GTK loop, unblocks the worker, and verifies the late result is safely discarded without exceptions.

Architecture Notes & Known Debt

• Thread Isolation: CUPS connections are strictly owned per-thread to avoid libcups context corruption.
• Polling Mechanism: The GTK thread currently waits for the worker by pumping a local GLib.MainLoop() and polling a shared queue using a 20ms GLib.timeout_add. While functional, a future iteration should upgrade this to a proper asynchronous eventfd or pipe() wakeup mechanism via GLib.io_add_watch.