Add version parameter and cleanup

WE2-1204

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

Author: metsma

.github/workflows/cmake-linux-fedora.yml

@@ -14,7 +14,7 @@ jobs:
     container: fedora:${{ matrix.container }}
     strategy:
       matrix:
-        container: [42, 43]
+        container: [42, 43, 44]
 
     steps:
       - name: Install Deps

.github/workflows/cmake-linux-ubuntu.yml

@@ -17,12 +17,19 @@ jobs:
     container: ubuntu:${{matrix.container}}
     strategy:
       matrix:
-        container: ['22.04', '24.04', '25.04', '25.10']
+        container: ['22.04', '24.04', '25.10', '26.04']
         arch: ['amd64', 'arm64']
 
     steps:
       - name: Install dependencies
-        run: apt update -qq && apt install --no-install-recommends -y git lsb-release fakeroot build-essential devscripts debhelper lintian pkg-config cmake libpcsclite-dev libssl-dev libgtest-dev libgl-dev libqt6svg6-dev qt6-tools-dev qt6-tools-dev-tools qt6-l10n-tools
+        run: |
+          for f in /etc/apt/sources.list /etc/apt/sources.list.d/*.list /etc/apt/sources.list.d/*.sources; do
+            [ -f "$f" ] && sed -i 's|http://archive.ubuntu.com|http://azure.archive.ubuntu.com|g; s|http://security.ubuntu.com|http://azure.archive.ubuntu.com|g' "$f"
+          done
+          echo 'path-exclude=/usr/share/man/*'      >  /etc/dpkg/dpkg.cfg.d/99-nodocs
+          echo 'path-exclude=/usr/share/doc/*'      >> /etc/dpkg/dpkg.cfg.d/99-nodocs
+          echo 'path-exclude=/usr/share/doc-base/*' >> /etc/dpkg/dpkg.cfg.d/99-nodocs
+          apt update -qq && apt install --no-install-recommends -y git lsb-release fakeroot build-essential devscripts debhelper lintian pkg-config cmake libpcsclite-dev libssl-dev libgtest-dev libgl-dev libqt6svg6-dev qt6-tools-dev qt6-tools-dev-tools qt6-l10n-tools
 
       - uses: actions/checkout@v6
         with:

lib/libelectronic-id

@@ -33,6 +33,7 @@ add_library(controller STATIC
     utils/erasedata.hpp
     utils/observer_ptr.hpp
     utils/qdisablecopymove.hpp
+    utils/qt_comp.hpp
     utils/utils.hpp
     writeresponse.cpp
     writeresponse.hpp

src/controller/CMakeLists.txt

@@ -24,6 +24,7 @@
 #include "certandpininfo.hpp"
 #include "logging.hpp"
 #include "retriableerror.hpp"
+#include "utils/qt_comp.hpp"
 
 #include <QCommandLineParser>
 #include <QDir>
@@ -36,6 +37,11 @@
 #include <QStyleHints>
 #include <QTranslator>
 
+#include <algorithm>
+#include <array>
+
+using namespace Qt::Literals::StringLiterals;
+
 inline CommandWithArguments::second_type parseArgumentJson(const QString& argumentStr)
 {
     const auto argumentJson = QJsonDocument::fromJson(argumentStr.toUtf8());
@@ -51,26 +57,26 @@ Application::Application(int& argc, char** argv, const QString& name) :
     QApplication(argc, argv), translator(new QTranslator(this))
 {
     setApplicationName(name);
-    setApplicationDisplayName(QStringLiteral("Web eID"));
+    setApplicationDisplayName(u"Web eID"_s);
     setApplicationVersion(QStringLiteral(PROJECT_VERSION));
-    setOrganizationDomain(QStringLiteral("web-eid.eu"));
-    setOrganizationName(QStringLiteral("RIA"));
+    setOrganizationDomain(u"web-eid.eu"_s);
+    setOrganizationName(u"RIA"_s);
     setQuitOnLastWindowClosed(false);
 
     installTranslator(translator);
     loadTranslations();
 
     auto list = QUrl::idnWhitelist();
     list.append({
-        QStringLiteral("fi"),
-        QStringLiteral("ee"),
-        QStringLiteral("lt"),
-        QStringLiteral("lv"),
+        u"fi"_s,
+        u"ee"_s,
+        u"lt"_s,
+        u"lv"_s,
     });
     QUrl::setIdnWhitelist(list);
 
-    for (const QString& font : QDir(QStringLiteral(":/fonts")).entryList()) {
-        QFontDatabase::addApplicationFont(QStringLiteral(":/fonts/%1").arg(font));
+    for (const QString& font : QDir(u":/fonts"_s).entryList()) {
+        QFontDatabase::addApplicationFont(u":/fonts/%1"_s.arg(font));
     }
 
     registerMetatypes();
@@ -93,8 +99,7 @@ bool Application::isDarkTheme()
     // supported OS-s.
     static const bool isDarkTheme = [] {
         QProcess p;
-        p.start(QStringLiteral("gsettings"),
-                {"get", "org.gnome.desktop.interface", "color-scheme"});
+        p.start(u"gsettings"_s, {u"get"_s, u"org.gnome.desktop.interface"_s, u"color-scheme"_s});
         if (p.waitForFinished()) {
             return p.readAllStandardOutput().contains("dark");
         }
@@ -108,56 +113,52 @@ bool Application::isDarkTheme()
 
 void Application::loadTranslations(const QString& lang)
 {
-    static const QStringList SUPPORTED_LANGS {
-        QStringLiteral("en"), QStringLiteral("et"), QStringLiteral("fi"), QStringLiteral("hr"),
-        QStringLiteral("ru"), QStringLiteral("de"), QStringLiteral("fr"), QStringLiteral("nl"),
-        QStringLiteral("cs"), QStringLiteral("sk")};
+    static constexpr auto SUPPORTED_LANGS = std::to_array<QStringView>(
+        {u"en", u"et", u"fi", u"hr", u"ru", u"de", u"fr", u"nl", u"cs", u"sk"});
     QLocale locale;
-    QString langSetting = QSettings().value(QStringLiteral("lang"), lang).toString();
-    if (SUPPORTED_LANGS.contains(langSetting)) {
+    QString langSetting = QSettings().value(u"lang"_s, lang).toString();
+    if (std::ranges::find(SUPPORTED_LANGS, langSetting) != SUPPORTED_LANGS.cend()) {
         locale = QLocale(langSetting);
     }
-    void(translator->load(locale, QStringLiteral(":/translations/")));
+    void(translator->load(locale, u":/translations/"_s));
 }
 
 CommandWithArguments Application::parseArgs()
 {
     // On Windows Chrome, the native messaging host is also passed a command line argument with a
     // handle to the calling Chrome native window: --parent-window=<decimal handle value>.
     // We don't use it, but need to support it to avoid unknown option errors.
-    QCommandLineOption parentWindow(QStringLiteral("parent-window"),
-                                    QStringLiteral("Parent window handle (unused)"),
-                                    QStringLiteral("parent-window"));
+    QCommandLineOption parentWindow(u"parent-window"_s, u"Parent window handle (unused)"_s,
+                                    u"parent-window"_s);
 
-    QCommandLineOption aboutArgument(QStringLiteral("about"),
-                                     QStringLiteral("Show Web-eID about window"));
+    QCommandLineOption aboutArgument(u"about"_s, u"Show Web-eID about window"_s);
+    QCommandLineOption commandLineMode(
+        {u"c"_s, u"command-line-mode"_s},
+        u"Command-line mode, read commands from command line arguments instead of "
+        "standard input."_s);
 
     QCommandLineParser parser;
-    parser.setApplicationDescription(QStringLiteral(
-        "Application that communicates with the Web eID browser extension via standard input and "
+    parser.setApplicationDescription(
+        u"Application that communicates with the Web eID browser extension via standard input and "
         "output, but also works standalone in command-line mode. Performs PKI cryptographic "
-        "operations with eID smart cards for signing and authentication purposes."));
+        "operations with eID smart cards for signing and authentication purposes."_s);
 
     parser.addHelpOption();
-    parser.addOptions({{{"c", "command-line-mode"},
-                        "Command-line mode, read commands from command line arguments instead of "
-                        "standard input."},
-                       aboutArgument,
-                       parentWindow});
+    parser.addVersionOption();
+    parser.addOptions({commandLineMode, aboutArgument, parentWindow});
 
-    static const auto COMMANDS = "'" + CMDLINE_GET_SIGNING_CERTIFICATE + "', '"
-        + CMDLINE_AUTHENTICATE + "', '" + CMDLINE_SIGN + "'.";
+    static const auto COMMANDS = u"'%1', '%2', '%3'."_s.arg(CMDLINE_GET_SIGNING_CERTIFICATE,
+                                                            CMDLINE_AUTHENTICATE, CMDLINE_SIGN);
 
     parser.addPositionalArgument(
-        QStringLiteral("command"),
-        QStringLiteral("The command to execute in command-line mode, any of ") + COMMANDS);
-    parser.addPositionalArgument(
-        QStringLiteral("arguments"),
-        QStringLiteral("Arguments to the given command as a JSON-encoded string."));
+        u"command"_s, u"The command to execute in command-line mode, any of "_s + COMMANDS,
+        u"(%1|%2|%3)"_s.arg(CMDLINE_GET_SIGNING_CERTIFICATE, CMDLINE_AUTHENTICATE, CMDLINE_SIGN));
+    parser.addPositionalArgument(u"arguments"_s,
+                                 u"Arguments to the given command as a JSON-encoded string."_s);
 
     parser.process(arguments());
 
-    if (parser.isSet(QStringLiteral("command-line-mode"))) {
+    if (parser.isSet(commandLineMode)) {
         const auto args = parser.positionalArguments();
         if (args.size() != 2) {
             throw ArgumentError("Provide two positional arguments in command-line mode.");

src/controller/application.cpp

@@ -44,12 +44,12 @@ namespace
 // Use common base64-encoding defaults.
 constexpr auto BASE64_OPTIONS = QByteArray::Base64Encoding | QByteArray::KeepTrailingEquals;
 
-QVariantMap createAuthenticationToken(const QString& signatureAlgorithm,
+QVariantMap createAuthenticationToken(std::string_view signatureAlgorithm,
                                       const QByteArray& certificateDer, const QByteArray& signature)
 {
     return QVariantMap {
         {"unverifiedCertificate", QString(certificateDer.toBase64(BASE64_OPTIONS))},
-        {"algorithm", signatureAlgorithm},
+        {"algorithm", QLatin1String(signatureAlgorithm.data(), signatureAlgorithm.size())},
         {"signature", QString(signature)},
         {"format", QStringLiteral("web-eid:1.0")},
         {"appVersion",
@@ -123,8 +123,6 @@ QVariantMap Authenticate::onConfirm(WebEidUI* window,
                                     const EidCertificateAndPinInfo& certAndPinInfo)
 {
     try {
-        const auto signatureAlgorithm =
-            QString::fromStdString(certAndPinInfo.eid->authSignatureAlgorithm());
         pcsc_cpp::byte_vector pin;
         // Reserve space for APDU overhead (5 bytes) + PIN padding (16 bytes) to prevent PIN memory
         // reallocation. The 16-byte limit comes from the max PIN length of 12 bytes across all card
@@ -133,8 +131,8 @@ QVariantMap Authenticate::onConfirm(WebEidUI* window,
         getPin(pin, *certAndPinInfo.eid, window);
         const auto signature =
             createSignature(origin.url(), challengeNonce, *certAndPinInfo.eid, std::move(pin));
-        return createAuthenticationToken(signatureAlgorithm, certAndPinInfo.certificateBytesInDer,
-                                         signature);
+        return createAuthenticationToken(certAndPinInfo.eid->authSignatureAlgorithm(),
+                                         certAndPinInfo.certificateBytesInDer, signature);
 
     } catch (const VerifyPinFailed& failure) {
         switch (failure.status()) {

src/controller/command-handlers/authenticate.cpp

@@ -27,15 +27,12 @@
 #include <stdexcept>
 #include <map>
 
-const QString CMDLINE_GET_SIGNING_CERTIFICATE = QStringLiteral("get-signing-certificate");
-const QString CMDLINE_AUTHENTICATE = QStringLiteral("authenticate");
-const QString CMDLINE_SIGN = QStringLiteral("sign");
 // A special command for stdin mode for quitting the application after sending the version.
-const QString STDINMODE_QUIT = QStringLiteral("quit");
+constexpr QStringView STDINMODE_QUIT {u"quit"};
 
 CommandType::CommandType(const QString& cmdName)
 {
-    static const std::map<QString, CommandType> SUPPORTED_COMMANDS {
+    static const std::map<QStringView, CommandType> SUPPORTED_COMMANDS {
         {CMDLINE_GET_SIGNING_CERTIFICATE, CommandType::GET_SIGNING_CERTIFICATE},
         {CMDLINE_AUTHENTICATE, CommandType::AUTHENTICATE},
         {CMDLINE_SIGN, CommandType::SIGN},

src/controller/commands.cpp

@@ -52,8 +52,8 @@ class CommandType
     CommandTypeEnum value;
 };
 
-extern const QString CMDLINE_GET_SIGNING_CERTIFICATE;
-extern const QString CMDLINE_AUTHENTICATE;
-extern const QString CMDLINE_SIGN;
+constexpr QStringView CMDLINE_GET_SIGNING_CERTIFICATE {u"get-signing-certificate"};
+constexpr QStringView CMDLINE_AUTHENTICATE {u"authenticate"};
+constexpr QStringView CMDLINE_SIGN {u"sign"};
 
 using CommandWithArguments = std::pair<CommandType, QVariantMap>;

src/controller/commands.hpp

@@ -22,6 +22,8 @@
 
 #include "logging.hpp"
 
+#include "utils/qt_comp.hpp"
+
 #include <QCoreApplication>
 #include <QDateTime>
 #include <QDir>

src/controller/logging.cpp

@@ -26,14 +26,6 @@
 
 void setupLogging();
 
-#if QT_VERSION < QT_VERSION_CHECK(6, 5, 0)
-template <typename... Args>
-inline QDebug operator<<(QDebug out, const std::basic_string<char, Args...>& s)
-{
-    return out << QUtf8StringView(s);
-}
-#endif
-
 inline QDebug operator<<(QDebug out, const std::exception& e)
 {
     out << e.what();