fix(io): exception not caught in DicomWeb services
Description
The following discussion from FW4SPL/fw4spl!291 should be addressed:
-
@fbridault started a discussion: (+6 comments) If I don't put the correct pacs configuration, I got a crash:
Qt has caught an exception thrown from an event handler. Throwing exceptions from an event handler is not supported in Qt. You must not let any exception whatsoever propagate through Qt code. If that is not possible, in Qt 5 you must at least reimplement QCoreApplication::notify() and catch all exceptions there. QObject::disconnect: Unexpected null parameter terminate called after throwing an instance of 'fwNetworkIO::exceptions::ConnectionRefused' what(): ConnectionRefusedError
Steps to reproduce
- Launch dicomxplorer
- Launch the dicomweb reader activity
- Clear out the pacs configuration
- Search for a patient
- Observe the error message in the console
Proposal
Optional section to give some functional or technical hints
Functional specifications
If there is any change to the workflow, UX/UI design, screenshots, etc..., please describe it here
Technical specifications
Details of the implementation of the fix
Test plan
Describe how you will verify that the implementation fulfils the specifications