Rerun CardEventMonitorThread after verify PIN failed

WE2-1136

Signed-off-by: Raul Metsma <raul@metsma.ee>

Author: metsma

src/controller/command-handlers/authenticate.hpp

@@ -35,10 +35,6 @@ class Authenticate : public CertificateReader
     QVariantMap onConfirm(WebEidUI* window,
                           const EidCertificateAndPinInfo& certAndPinInfo) override;
 
-signals:
-    void verifyPinFailed(const electronic_id::VerifyPinFailed::Status status,
-                         const qint8 retriesLeft);
-
 private:
     QString challengeNonce;
 };

src/controller/command-handlers/sign.hpp

@@ -37,8 +37,6 @@ class Sign : public CertificateReader
 
 signals:
     void signingCertificateMismatch();
-    void verifyPinFailed(const electronic_id::VerifyPinFailed::Status status,
-                         const qint8 retriesLeft);
 
 private:
     void

src/controller/commandhandler.hpp

@@ -45,6 +45,7 @@ class CommandHandler : public QObject
     void multipleCertificatesReady(const QUrl& origin,
                                    const std::vector<EidCertificateAndPinInfo>& certAndPinInfos);
     void singleCertificateReady(const QUrl& origin, const EidCertificateAndPinInfo& certAndPinInfo);
+    void verifyPinFailed(electronic_id::VerifyPinFailed::Status status, qint8 retriesLeft);
 
 protected:
     CommandHandler(const CommandWithArguments& cmd) : command(cmd) {}

src/controller/controller.cpp

@@ -93,6 +93,15 @@ void Controller::run()
 
         commandHandler = getCommandHandler(*command);
 
+        // When the command handler run thread retrieves certificates successfully, call
+        // onCertificatesLoaded() that starts card event monitoring while user enters the PIN.
+        connect(commandHandler.get(), &CommandHandler::singleCertificateReady, this,
+                &Controller::onCertificatesLoaded);
+        connect(commandHandler.get(), &CommandHandler::multipleCertificatesReady, this,
+                &Controller::onCertificatesLoaded);
+        connect(commandHandler.get(), &CommandHandler::verifyPinFailed, this,
+                &Controller::onCertificatesLoaded);
+
         startCommandExecution();
 
     } catch (const std::exception& error) {
@@ -160,25 +169,14 @@ void Controller::onCardsAvailable(
     }
 }
 
-void Controller::runCommandHandler(const std::vector<ElectronicID::ptr>& availableEids)
-{
-    try {
-        auto* commandHandlerRunThread =
-            new CommandHandlerRunThread(this, *commandHandler, availableEids);
-        connectRetry(commandHandlerRunThread);
-
-        // When the command handler run thread retrieves certificates successfully, call
-        // onCertificatesLoaded() that starts card event monitoring while user enters the PIN.
-        connect(commandHandler.get(), &CommandHandler::singleCertificateReady, this,
-                &Controller::onCertificatesLoaded);
-        connect(commandHandler.get(), &CommandHandler::multipleCertificatesReady, this,
-                &Controller::onCertificatesLoaded);
-
-        commandHandlerRunThread->start();
-
-    } catch (const std::exception& error) {
-        onCriticalFailure(error.what());
-    }
+void Controller::runCommandHandler(std::vector<ElectronicID::ptr> availableEids) noexcept
+try {
+    auto* commandHandlerRunThread =
+        new CommandHandlerRunThread(this, *commandHandler, std::move(availableEids));
+    connectRetry(commandHandlerRunThread);
+    commandHandlerRunThread->start();
+} catch (const std::exception& error) {
+    onCriticalFailure(error.what());
 }
 
 void Controller::onCertificatesLoaded()
@@ -202,22 +200,20 @@ void Controller::disposeUI()
     }
 }
 
-void Controller::onConfirmCommandHandler(const EidCertificateAndPinInfo& certAndPinInfo)
-{
+void Controller::onConfirmCommandHandler(const EidCertificateAndPinInfo& certAndPinInfo) noexcept
+try {
     emit stopCardEventMonitorThread();
 
-    try {
-        auto* commandHandlerConfirmThread =
-            new CommandHandlerConfirmThread(this, *commandHandler, window, certAndPinInfo);
-        connect(commandHandlerConfirmThread, &CommandHandlerConfirmThread::completed, this,
-                &Controller::onCommandHandlerConfirmCompleted);
-        connectRetry(commandHandlerConfirmThread);
+    auto* commandHandlerConfirmThread =
+        new CommandHandlerConfirmThread(this, *commandHandler, window, certAndPinInfo);
+    connect(commandHandlerConfirmThread, &CommandHandlerConfirmThread::completed, this,
+            &Controller::onCommandHandlerConfirmCompleted);
+    connectRetry(commandHandlerConfirmThread);
 
-        commandHandlerConfirmThread->start();
+    commandHandlerConfirmThread->start();
 
-    } catch (const std::exception& error) {
-        onCriticalFailure(error.what());
-    }
+} catch (const std::exception& error) {
+    onCriticalFailure(error.what());
 }
 
 void Controller::onCommandHandlerConfirmCompleted(const QVariantMap& res)
@@ -258,7 +254,7 @@ void Controller::onRetry()
     }
 }
 
-void Controller::connectRetry(const ControllerChildThread* childThread)
+void Controller::connectRetry(const ControllerChildThread* childThread) const
 {
     REQUIRE_NON_NULL(childThread)
     connect(childThread, &ControllerChildThread::failure, this, &Controller::onCriticalFailure);

src/controller/controller.hpp

@@ -52,7 +52,7 @@ class Controller : public QObject
     void onCertificatesLoaded();
 
     // Called either directly from onDialogOK().
-    void onConfirmCommandHandler(const EidCertificateAndPinInfo& certAndPinInfo);
+    void onConfirmCommandHandler(const EidCertificateAndPinInfo& certAndPinInfo) noexcept;
 
     // Called from CommandHandlerConfirm thread.
     void onCommandHandlerConfirmCompleted(const QVariantMap& result);
@@ -69,8 +69,8 @@ class Controller : public QObject
 
 private:
     void startCommandExecution();
-    void runCommandHandler(const std::vector<electronic_id::ElectronicID::ptr>& availableEids);
-    void connectRetry(const ControllerChildThread* childThread);
+    void runCommandHandler(std::vector<electronic_id::ElectronicID::ptr> availableEids) noexcept;
+    void connectRetry(const ControllerChildThread* childThread) const;
     void createWindow();
     void disposeUI();
     void exit();

src/controller/threads/commandhandlerrunthread.hpp

@@ -30,8 +30,9 @@ class CommandHandlerRunThread : public ControllerChildThread
 
 public:
     CommandHandlerRunThread(QObject* parent, CommandHandler& handler,
-                            const std::vector<electronic_id::ElectronicID::ptr>& eids) :
-        ControllerChildThread(handler.commandType(), parent), commandHandler(handler), eids(eids)
+                            std::vector<electronic_id::ElectronicID::ptr> eids) noexcept :
+        ControllerChildThread(handler.commandType(), parent), commandHandler(handler),
+        eids(std::move(eids))
     {
         // Connect retry signal to retry signal to pass it up from the command handler.
         connect(&commandHandler, &CommandHandler::retry, this, &ControllerChildThread::retry);

src/ui/webeiddialog.cpp

@@ -171,7 +171,8 @@ WebEidDialog::WebEidDialog(QWidget* parent) : WebEidUI(parent), ui(new Private)
                 if (auto* button =
                         qobject_cast<CertificateButton*>(ui->selectionGroup->checkedButton())) {
                     ui->lockedWarning->setHidden(button->certificateInfo().cardActive);
-                    ui->okButton->setEnabled(currentCommand == CommandType::AUTHENTICATE || button->certificateInfo().cardActive);
+                    ui->okButton->setEnabled(currentCommand == CommandType::AUTHENTICATE
+                                             || button->certificateInfo().cardActive);
                 }
                 ui->okButton->setFocus();
             });
@@ -666,7 +667,8 @@ void WebEidDialog::setupPinPadProgressBarAndEmitWait(const EidCertificateAndPinI
 void WebEidDialog::setupPinInput(const EidCertificateAndPinInfo& certAndPinInfo)
 {
     ui->lockedWarning->setHidden(certAndPinInfo.cardActive);
-    setupPinPrompt(certAndPinInfo.pinInfo, currentCommand == CommandType::AUTHENTICATE || certAndPinInfo.cardActive);
+    setupPinPrompt(certAndPinInfo.pinInfo,
+                   currentCommand == CommandType::AUTHENTICATE || certAndPinInfo.cardActive);
     // The allowed character ranges are from the SafeNet eToken guide:
     // 1. English uppercase letters (ASCII 0x41...0x5A).
     // 2. English lowercase letters (ASCII 0x61...0x7A).