Remove fromPunycode

Add ee, fi, lt, lv to allowed IDN parsing list

WE2-1173

Signed-off-by: Raul Metsma <[email protected]>

Author: metsma

src/controller/application.cpp

@@ -60,6 +60,15 @@ Application::Application(int& argc, char** argv, const QString& name) :
     installTranslator(translator);
     loadTranslations();
 
+    auto list = QUrl::idnWhitelist();
+    list.append({
+        QStringLiteral("fi"),
+        QStringLiteral("ee"),
+        QStringLiteral("lt"),
+        QStringLiteral("lv"),
+    });
+    QUrl::setIdnWhitelist(list);
+
     for (const QString& font : QDir(QStringLiteral(":/fonts")).entryList()) {
         QFontDatabase::addApplicationFont(QStringLiteral(":/fonts/%1").arg(font));
     }

src/ui/CMakeLists.txt

@@ -1,7 +1,6 @@
 add_library(ui STATIC
     certificatewidget.cpp
     certificatewidget.hpp
-    punycode.hpp
     ui.cpp
     webeiddialog.cpp
     webeiddialog.hpp

src/ui/dialog.ui

@@ -513,6 +513,9 @@ border-radius: 4px;</string>
            <property name="focusPolicy">
             <enum>Qt::TabFocus</enum>
            </property>
+           <property name="textFormat">
+            <enum>Qt::PlainText</enum>
+           </property>
           </widget>
          </item>
         </layout>
@@ -586,6 +589,9 @@ border-radius: 4px;</string>
            <property name="focusPolicy">
             <enum>Qt::TabFocus</enum>
            </property>
+           <property name="textFormat">
+            <enum>Qt::PlainText</enum>
+           </property>
           </widget>
          </item>
         </layout>

src/ui/punycode.hpp

@@ -22,7 +22,6 @@
 
 #include "webeiddialog.hpp"
 #include "application.hpp"
-#include "punycode.hpp"
 
 #include "ui_dialog.h"
 
@@ -43,8 +42,6 @@
 #include <unistd.h>
 #endif
 
-#include <unordered_map>
-
 #if QT_VERSION < QT_VERSION_CHECK(6, 4, 0)
 constexpr inline QLatin1String operator"" _L1(const char* str, size_t size) noexcept
 {
@@ -336,7 +333,7 @@ void WebEidDialog::onSmartCardStatusUpdate(const RetriableError status)
 void WebEidDialog::onMultipleCertificatesReady(
     const QUrl& origin, const std::vector<EidCertificateAndPinInfo>& certAndPinInfos)
 {
-    ui->selectCertificateOriginLabel->setText(fromPunycode(origin));
+    ui->selectCertificateOriginLabel->setText(origin.host(QUrl::PrettyDecoded));
     setupCertificateAndPinInfo(certAndPinInfos);
 
     switch (currentCommand) {
@@ -390,7 +387,7 @@ void WebEidDialog::onMultipleCertificatesReady(
 void WebEidDialog::onSingleCertificateReady(const QUrl& origin,
                                             const EidCertificateAndPinInfo& certAndPinInfo)
 {
-    ui->selectCertificateOriginLabel->setText(fromPunycode(origin));
+    ui->selectCertificateOriginLabel->setText(origin.host(QUrl::PrettyDecoded));
     ui->pinInputOriginLabel->setText(ui->selectCertificateOriginLabel->text());
     const bool useExternalPinDialog = certAndPinInfo.eid->providesExternalPinDialog();
 

src/ui/webeiddialog.cpp

@@ -26,8 +26,6 @@
 #include "controller.hpp"
 #include "command-handlers/getcertificate.hpp"
 
-#include "../ui/punycode.hpp"
-
 #include "mock-ui.hpp"
 #include "getcommandhandler-mock.hpp"
 
@@ -72,8 +70,6 @@ private slots:
 
     void authenticate_validArgumentsResultInValidToken();
 
-    void fromPunycode_decodesEeDomain();
-
     void quit_exits();
 
 private:
@@ -201,12 +197,6 @@ void WebEidTests::authenticate_validArgumentsResultInValidToken()
              QStringLiteral("MIIEAzCCA2WgAwIBAgIQOWkBW"));
 }
 
-void WebEidTests::fromPunycode_decodesEeDomain()
-{
-    QCOMPARE(fromPunycode(QUrl(QStringLiteral("https://xn--igusnunik-p7af.ee"))),
-             QStringLiteral("\u00F5igusn\u00F5unik.ee"));
-}
-
 void WebEidTests::quit_exits()
 {
     try {