Refactor: move the validation of certificates & key functions under Server class

Rename method
Add log messages

src/base/http/server.cpp

@@ -61,11 +61,27 @@ Server::~Server()
 }
 
 #ifndef QT_NO_OPENSSL
-void Server::enableHttps(const QList<QSslCertificate> &certificates, const QSslKey &key)
+bool Server::setupHttps(const QByteArray &certificates, const QByteArray &key)
 {
-    m_certificates = certificates;
+    QSslKey sslKey(key, QSsl::Rsa);
-    m_key = key;
+    if (sslKey.isNull())
+        sslKey = QSslKey(key, QSsl::Ec);
+
+    const QList<QSslCertificate> certs = QSslCertificate::fromData(certificates);
+    const bool areCertsValid = !certs.empty() && std::all_of(certs.begin(), certs.end(), [](const QSslCertificate &c) { return !c.isNull(); });
+
+    if (!sslKey.isNull() && areCertsValid)
+    {
+        m_key = sslKey;
+        m_certificates = certs;
         m_https = true;
+        return true;
+    }
+    else
+    {
+        disableHttps();
+        return false;
+    }
 }
 
 void Server::disableHttps()

src/base/http/server.h

@@ -55,7 +55,7 @@ namespace Http
         ~Server();
 
 #ifndef QT_NO_OPENSSL
-        void enableHttps(const QList<QSslCertificate> &certificates, const QSslKey &key);
+        bool setupHttps(const QByteArray &certificates, const QByteArray &key);
         void disableHttps();
 #endif
 

src/webui/webui.cpp

@@ -74,14 +74,13 @@ void WebUI::init()
 
 #ifndef QT_NO_OPENSSL
         if (pref->isWebUiHttpsEnabled()) {
-            QList<QSslCertificate> certs = QSslCertificate::fromData(pref->getWebUiHttpsCertificate());
+            const QByteArray certs = pref->getWebUiHttpsCertificate();
-            QSslKey key;
+            const QByteArray key = pref->getWebUiHttpsKey();
-            key = QSslKey(pref->getWebUiHttpsKey(), QSsl::Rsa);
+            bool success = m_httpServer->setupHttps(certs, key);
-            bool certsIsNull = std::any_of(certs.begin(), certs.end(), [](QSslCertificate c) { return c.isNull(); });
+            if (success)
-            if (!certsIsNull && !certs.empty() && !key.isNull())
+                logger->addMessage(tr("Web UI: https setup successful"));
-                m_httpServer->enableHttps(certs, key);
             else
-                m_httpServer->disableHttps();
+                logger->addMessage(tr("Web UI: https setup failed, fallback to http"), Log::CRITICAL);
         }
         else {
             m_httpServer->disableHttps();