From d070827622d16a69c7e9d281a4fd12de794b81cd Mon Sep 17 00:00:00 2001 From: Raul Metsma Date: Mon, 5 Jun 2023 11:34:20 +0300 Subject: [PATCH 1/4] Use word wrap WE2-767 Signed-off-by: Raul Metsma --- src/ui/certificatewidget.cpp | 15 +++--- src/ui/dialog.ui | 100 +++++++++++++++++++---------------- src/ui/translations/fr.ts | 52 +++++++++--------- 3 files changed, 88 insertions(+), 79 deletions(-) diff --git a/src/ui/certificatewidget.cpp b/src/ui/certificatewidget.cpp index 6bd53b7d..0e07066c 100644 --- a/src/ui/certificatewidget.cpp +++ b/src/ui/certificatewidget.cpp @@ -47,23 +47,24 @@ CertificateWidgetInfo::CertificateWidgetInfo(QWidget* self) : icon(new QLabel(self)), info(new QLabel(self)), warnIcon(new QLabel(self)), warn(new QLabel(CertificateWidget::tr("Pin locked"), self)) { - icon->setPixmap(QStringLiteral(":/images/id-card.svg")); - warnIcon->setPixmap(QStringLiteral(":/images/fatal.svg")); - if (qApp->isDarkTheme()) { + if (Application::isDarkTheme()) { icon->setPixmap(QStringLiteral(":/images/id-card_dark.svg")); warnIcon->setPixmap(QStringLiteral(":/images/fatal_dark.svg")); + } else { + icon->setPixmap(QStringLiteral(":/images/id-card.svg")); + warnIcon->setPixmap(QStringLiteral(":/images/fatal.svg")); } warnIcon->hide(); warnIcon->installEventFilter(self); warn->setObjectName(QStringLiteral("warn")); warn->hide(); - QHBoxLayout* layout = new QHBoxLayout(self); + auto* layout = new QHBoxLayout(self); layout->setContentsMargins(20, 0, 20, 0); - layout->setSpacing(20); + layout->setSpacing(10); layout->addWidget(icon); layout->addWidget(info, 1); layout->addWidget(warnIcon); - QHBoxLayout* warnLayout = new QHBoxLayout; + auto* warnLayout = new QHBoxLayout; warnLayout->setSpacing(6); warnLayout->addWidget(warnIcon); warnLayout->addWidget(warn); @@ -91,7 +92,7 @@ void CertificateWidgetInfo::setCertificateInfo(const CardCertificateAndPinInfo& { warn->setText(CertificateWidget::tr("Pin locked")); certAndPinInfo = cardCertPinInfo; - const auto certInfo = cardCertPinInfo.certInfo; + const auto& certInfo = cardCertPinInfo.certInfo; QString warning, effectiveDate = certInfo.effectiveDate, expiryDate = certInfo.expiryDate; if (certInfo.notEffective) { effectiveDate = displayInRed(effectiveDate); diff --git a/src/ui/dialog.ui b/src/ui/dialog.ui index 21077e86..9f14d41b 100644 --- a/src/ui/dialog.ui +++ b/src/ui/dialog.ui @@ -6,13 +6,13 @@ 0 0 - 500 - 495 + 550 + 510 - 500 + 550 0 @@ -527,7 +527,7 @@ height: 24px; Qt::TabFocus - Authenticate + Authenticate @@ -545,7 +545,7 @@ height: 24px; Qt::TabFocus - By authenticating, I agree to the transfer of my name and personal identification code to the service provider. + By authenticating, I agree to the transfer of my name and personal identification code to the service provider. true @@ -569,49 +569,48 @@ height: 24px; - - + + + 0 + + 8 - + - - - - - - Select another certificate - - - - :/images/left.svg:/images/left.svg - - - - 24 - 24 - - - - Qt::ToolButtonTextBesideIcon - - - - - - - Qt::Horizontal - - - - 40 - 20 - - - - - + + + + Select another certificate + + + + :/images/left.svg:/images/left.svg + + + + 24 + 24 + + + + Qt::ToolButtonTextBesideIcon + + + + + + + Qt::Horizontal + + + + 40 + 0 + + + @@ -630,7 +629,7 @@ height: 24px; - Enter PIN1 for authentication + Enter PIN1 for authentication Qt::AlignCenter @@ -652,6 +651,12 @@ height: 24px; + + + 0 + 36 + + Qt::TabFocus @@ -661,6 +666,9 @@ height: 24px; Qt::AlignCenter + + true + @@ -688,7 +696,7 @@ height: 24px; 20 - 0 + 5 diff --git a/src/ui/translations/fr.ts b/src/ui/translations/fr.ts index 19db893a..72220236 100644 --- a/src/ui/translations/fr.ts +++ b/src/ui/translations/fr.ts @@ -20,7 +20,7 @@ <b>%1</b><br />Issuer: %2<br />Valid: %3 to %4%5 - <b>%1</b><br />Émetteur: %2<br />Valide: %3 jusqu'à %4%5 + <b>%1</b><br />Émetteur: %2<br />Valide: %3 jusqu'à %4%5 Pin locked @@ -31,23 +31,23 @@ WebEidDialog By choosing the certificate, I agree to the transfer of my name and personal identification code to the service provider. - En sélectionnant le certificat, j'accepte le transfert de mon nom et de mon code d'identification personnel au fournisseur de services. + En sélectionnant le certificat, j'accepte le transfert de mon nom et de mon code d'identification personnel au fournisseur de services. By authenticating, I agree to the transfer of my name and personal identification code to the service provider. - En m'authentifiant, j'accepte le transfert de mon nom et de mon code d'identification personnel au fournisseur de services. + En m'authentifiant, j'accepte le transfert de mon nom et de mon code d'identification personnel au fournisseur de services. By signing, I agree to the transfer of my name and personal identification code to the service provider. - En signant, j'accepte le transfert de mon nom et de mon code d'identification personnel au fournisseur de services. + En signant, j'accepte le transfert de mon nom et de mon code d'identification personnel au fournisseur de services. The smart card service required to use the ID-card is not running. Please start the smart card service and try again. - Le service de carte à puce requis pour utiliser la carte d'identité n'est pas en cours d'exécution. Veuillez démarrer le service de carte à puce et réessayer. + Le service de carte à puce requis pour utiliser la carte d'identité n'est pas en cours d'exécution. Veuillez démarrer le service de carte à puce et réessayer. <b>Card reader not connected.</b> Please connect the card reader to the computer. - <b>Lecteur de carte non connecté.</b> Veuillez connecter le lecteur de carte à l'ordinateur. + <b>Lecteur de carte non connecté.</b> Veuillez connecter le lecteur de carte à l'ordinateur. Invalid PIN length @@ -59,7 +59,7 @@ Enter PIN1 for authentication - Entrez PIN1 pour l'authentification + Entrez PIN1 pour l'authentification Authenticate @@ -101,7 +101,7 @@ PinPad timed out waiting for customer interaction. - PinPad a expiré en attendant l'interaction du client. + PinPad a expiré en attendant l'interaction du client. PIN entry cancelled. @@ -125,23 +125,23 @@ The ID-card was removed from the reader. Please insert the ID-card into the reader. - La carte d'identité été retirée du lecteur. Veuillez insérer la carte d'identité dans le lecteur. + La carte d'identité été retirée du lecteur. Veuillez insérer la carte d'identité dans le lecteur. Operation failed. Make sure that the ID-card and the card reader are connected correctly. - L'opération a échoué. Assurez-vous que la carte d'identité et le lecteur de carte sont correctement connectés. + L'opération a échoué. Assurez-vous que la carte d'identité et le lecteur de carte sont correctement connectés. Check the ID-card and the reader connection - Vérifiez la carte d'identité et la connexion du lecteur + Vérifiez la carte d'identité et la connexion du lecteur Connection to the ID-card or reader failed. Make sure that the ID-card and the card reader are connected correctly. - La connexion à la carte d'identité ou au lecteur a échoué. Assurez-vous que la carte d'identité et le lecteur de carte sont correctement connectés. + La connexion à la carte d'identité ou au lecteur a échoué. Assurez-vous que la carte d'identité et le lecteur de carte sont correctement connectés. The desired operation cannot be performed with the inserted ID-card. Make sure that the ID-card is supported by the Web eID application. - L'opération souhaitée ne peut pas être effectuée avec la carte d'identité insérée. Assurez-vous que la carte d'identité est prise en charge par l'application Web eID. + L'opération souhaitée ne peut pas être effectuée avec la carte d'identité insérée. Assurez-vous que la carte d'identité est prise en charge par l'application Web eID. Operation not supported @@ -153,7 +153,7 @@ Operation failed - L'opération a échoué + L'opération a échoué Card driver error. Please try again. @@ -165,11 +165,11 @@ An error occurred in the Smart Card service required to use the ID-card. Make sure that the ID-card and the card reader are connected correctly or relaunch the Smart Card service. - Une erreur s'est produite dans le service Smart Card requis pour utiliser la carte d'identité. Assurez-vous que la carte d'identité et le lecteur de carte sont correctement connectés ou relancez le service Smart Card. + Une erreur s'est produite dans le service Smart Card requis pour utiliser la carte d'identité. Assurez-vous que la carte d'identité et le lecteur de carte sont correctement connectés ou relancez le service Smart Card. The card in the reader is not supported. Make sure that the entered ID-card is supported by the Web eID application. - La carte dans le lecteur n'est pas prise en charge. Assurez-vous que la carte d'identité saisie est prise en charge par l'application Web eID. + La carte dans le lecteur n'est pas prise en charge. Assurez-vous que la carte d'identité saisie est prise en charge par l'application Web eID. Unknown error @@ -181,7 +181,7 @@ PIN1 for authentication - PIN1 pour l'authentification + PIN1 pour l'authentification PIN2 for signing @@ -209,7 +209,7 @@ Operation failed. More detailed information can be found in the log files. - L'opération a échoué. Des informations plus détaillées peuvent être trouvées dans les fichiers journaux. + L'opération a échoué. Des informations plus détaillées peuvent être trouvées dans les fichiers journaux. Version: %1 @@ -217,7 +217,7 @@ Web eID extension is disabled. The extension must be enabled in the web browser to authenticate and sign with an ID-card. - L'extension Web eID est désactivée. L'extension doit être activée dans le navigateur web pour s'authentifier et signer avec une carte d'identité. + L'extension Web eID est désactivée. L'extension doit être activée dans le navigateur web pour s'authentifier et signer avec une carte d'identité. Open Safari settings... @@ -225,7 +225,7 @@ <b>This is a companion application to the Web eID browser extension</b> and cannot be used independently. To uninstall the extension, remove this application. - <b>Il s'agit d'une application complémentaire à l'extension de navigateur Web eID</b> et ne peut pas être utilisé indépendamment. Pour désinstaller l'extension, supprimez cette application. + <b>Il s'agit d'une application complémentaire à l'extension de navigateur Web eID</b> et ne peut pas être utilisé indépendamment. Pour désinstaller l'extension, supprimez cette application. About Web eID @@ -237,7 +237,7 @@ Get help here: <a href="https://www.id.ee">id.ee</a> - Recevez de l'aide ici: <a href="https://www.id.ee">id.ee</a> + Recevez de l'aide ici: <a href="https://www.id.ee">id.ee</a> Incorrect PIN. @@ -249,23 +249,23 @@ Operation failed. Make sure that the driver of the corresponding card reader is used. Read more <a href="https://www.id.ee/en/article/using-pinpad-card-reader-drivers/">here</a>. - L'opération a échoué. Assurez-vous que le pilote du lecteur de carte correspondant est utilisé. En savoir plus<a href="https://www.id.ee/en/article/using-pinpad-card-reader-drivers/">here</a>. + L'opération a échoué. Assurez-vous que le pilote du lecteur de carte correspondant est utilisé. En savoir plus<a href="https://www.id.ee/en/article/using-pinpad-card-reader-drivers/">here</a>. The certificate of the ID card in the reader does not match the originally submitted certificate. Please insert the original ID card. - Le certificat de la carte d'identité dans le lecteur ne correspond pas au certificat soumis à l'origine. Veuillez insérer la carte d'identité originale. + Le certificat de la carte d'identité dans le lecteur ne correspond pas au certificat soumis à l'origine. Veuillez insérer la carte d'identité originale. The inserted ID-card does not contain a certificate for the requested operation. Please insert an ID-card that supports the requested operation. - La carte d'identité insérée ne contient pas de certificat pour l'opération demandée. Veuillez insérer une carte d'identité qui prend en charge l'opération demandée. + La carte d'identité insérée ne contient pas de certificat pour l'opération demandée. Veuillez insérer une carte d'identité qui prend en charge l'opération demandée. Please enter PIN for authentication in the PIN dialog window that opens. - Veuillez entrer le code PIN pour l'authentification dans la fenêtre de dialogue PIN qui s'ouvre. + Veuillez entrer le code PIN pour l'authentification dans la fenêtre de dialogue PIN qui s'ouvre. Please enter PIN for signing in the PIN dialog window that opens. - Veuillez saisir le code PIN pour signer dans la fenêtre de dialogue PIN qui s'ouvre. + Veuillez saisir le code PIN pour signer dans la fenêtre de dialogue PIN qui s'ouvre. EN From dcb38bedd314d7e424270a1b77e0b625db09e2ab Mon Sep 17 00:00:00 2001 From: Raul Metsma Date: Wed, 5 Jul 2023 09:30:29 +0300 Subject: [PATCH 2/4] Fix crash with FinEID cards WE2-751 Signed-off-by: Raul Metsma --- lib/libelectronic-id | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/libelectronic-id b/lib/libelectronic-id index 6aba432a..63af0820 160000 --- a/lib/libelectronic-id +++ b/lib/libelectronic-id @@ -1 +1 @@ -Subproject commit 6aba432a47a9de75ae74a6b93bbb23662667a96f +Subproject commit 63af0820583c3801034188dd0ed36d1b148afcd9 From de7e072021120c1b09e8632d855c58d6c6b7d7ef Mon Sep 17 00:00:00 2001 From: Klemens Nanni Date: Sat, 8 Jul 2023 14:46:36 +0300 Subject: [PATCH 3/4] Linux, BSD: Add BUNDLE_XPI option The official Debian package includes this file, but other systems do not neccessarily need it; at least OpenBSD is unable to download files from the intenet during packaging, so this fails ether way. Signed-off-by: Klemens Nanni --- debian/rules | 2 ++ src/app/CMakeLists.txt | 11 ++++++++--- 2 files changed, 10 insertions(+), 3 deletions(-) diff --git a/debian/rules b/debian/rules index 05f42190..9415b886 100755 --- a/debian/rules +++ b/debian/rules @@ -1,4 +1,6 @@ #!/usr/bin/make -f include /usr/share/cdbs/1/rules/debhelper.mk include /usr/share/cdbs/1/class/cmake.mk +DEB_CMAKE_EXTRA_FLAGS = \ + -DBUNDLE_XPI=ON DEB_MAKE_CHECK_TARGET = test diff --git a/src/app/CMakeLists.txt b/src/app/CMakeLists.txt index b86d5b35..fdc9e308 100644 --- a/src/app/CMakeLists.txt +++ b/src/app/CMakeLists.txt @@ -162,8 +162,11 @@ elseif(APPLE) ) endif() else() + option(BUNDLE_XPI "Download and bundle the Firefox extension" OFF) include(GNUInstallDirs) - file(DOWNLOAD ${FIREFOX_URL} ${CMAKE_CURRENT_BINARY_DIR}/${FIREFOX_UUID}.xpi) + if (BUNDLE_XPI) + file(DOWNLOAD ${FIREFOX_URL} ${CMAKE_CURRENT_BINARY_DIR}/${FIREFOX_UUID}.xpi) + endif() set(WEBEID_PATH ${CMAKE_INSTALL_FULL_BINDIR}/web-eid) install(TARGETS web-eid DESTINATION ${CMAKE_INSTALL_BINDIR}) if(EXISTS "/etc/debian_version") @@ -179,8 +182,10 @@ else() DESTINATION ${CMAKE_INSTALL_DATADIR}/google-chrome/extensions) install(FILES ${CMAKE_SOURCE_DIR}/install/web-eid.desktop DESTINATION ${CMAKE_INSTALL_DATADIR}/applications) - install(FILES ${CMAKE_CURRENT_BINARY_DIR}/${FIREFOX_UUID}.xpi - DESTINATION ${CMAKE_INSTALL_DATADIR}/mozilla/extensions/{ec8030f7-c20a-464f-9b0e-13a3a9e97384}) + if (BUNDLE_XPI) + install(FILES ${CMAKE_CURRENT_BINARY_DIR}/${FIREFOX_UUID}.xpi + DESTINATION ${CMAKE_INSTALL_DATADIR}/mozilla/extensions/{ec8030f7-c20a-464f-9b0e-13a3a9e97384}) + endif() foreach(RES 16 32 128 256 512) install( FILES ${CMAKE_SOURCE_DIR}/install/appicon_${RES}.png From 0448f7eabd7dd0735ae95d83fdf7e5f27bc9b2a8 Mon Sep 17 00:00:00 2001 From: Mart Somermaa Date: Tue, 11 Jul 2023 20:58:57 +0300 Subject: [PATCH 4/4] deps(libelectronic-id): don't connect to or use the card in Pkcs11ElectronicID WE2-807 Signed-off-by: Mart Somermaa --- lib/libelectronic-id | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/libelectronic-id b/lib/libelectronic-id index 63af0820..fb84aa8c 160000 --- a/lib/libelectronic-id +++ b/lib/libelectronic-id @@ -1 +1 @@ -Subproject commit 63af0820583c3801034188dd0ed36d1b148afcd9 +Subproject commit fb84aa8cf7ae1073f5472b7093d4df2ed95dc718