NFC-82 Add error view and style for auth errors

SanderKondratjevNortal · SanderKondratjevNortal · commit 583e8bd50db3 · 2025-10-26T14:56:40.000+02:00
Signed-off-by: Sander Kondratjev &lt;sander.kondratjev@nortal.com&gt;
diff --git a/example/src/main/java/eu/webeid/example/web/rest/SigningController.java b/example/src/main/java/eu/webeid/example/web/rest/SigningController.java
@@ -78,7 +78,7 @@ public ResponseEntity<MobileInitRequest> mobileInit(WebEidAuthentication authent
 
     @GetMapping("mobile/certificate")
     public ModelAndView mobileCertificateResponse() {
-        return new ModelAndView("mobile-callback");
+        return new ModelAndView("webeid-callback");
     }
 
     @SuppressWarnings("javax.annotation.Tainted")
@@ -89,7 +89,7 @@ public ResponseEntity<MobileInitRequest> mobileCertificate(@RequestBody Certific
 
     @GetMapping("mobile/signature")
     public ModelAndView mobileSignatureResponse() {
-        return new ModelAndView("mobile-callback");
+        return new ModelAndView("webeid-callback");
     }
 
     @PostMapping("mobile/signature")
diff --git a/example/src/main/resources/templates/webeid-callback.html b/example/src/main/resources/templates/webeid-callback.html
@@ -7,14 +7,26 @@
     <meta id="csrftoken" name="csrftoken" th:content="${_csrf.token}"/>
     <meta id="csrfheadername" name="csrfheadername" th:content="${_csrf.headerName}"/>
     <title>Completing signing…</title>
+    <link rel="stylesheet" href="/css/bootstrap.min.css"/>
     <link rel="stylesheet" href="/css/main.css"/>
 </head>
 
 <body class="loading-page">
 <h2>Completing signing…</h2>
 <div class="spinner"></div>
 
-<script>
+<div id="error-message" class="alert alert-danger" style="display: none;" role="alert">
+    <div class="message"></div>
+    <pre class="details"></pre>
+</div>
+
+<p class="text-center p-4" style="display: none;" id="error-actions">
+    <button id="back-button" class="btn btn-primary">Back</button>
+</p>
+
+<script type="module">
+    import { showErrorMessage } from '/js/errors.js';
+
     (async () => {
         try {
             const fragment = window.location.hash.slice(1);
@@ -26,6 +38,24 @@ <h2>Completing signing…</h2>
             const decoded = atob(fragment);
             const payload = JSON.parse(decoded);
 
+            if (payload.error) {
+                console.error("WebEID mobile error:", payload);
+
+                showErrorMessage({
+                    code: payload.code ?? "UNKNOWN_ERROR",
+                    message: payload.message ?? "Signing failed",
+                });
+
+                const actions = document.getElementById("error-actions");
+                const goBackButton = document.getElementById("back-button");
+                actions.style.display = "block";
+                goBackButton.addEventListener("click", () => {
+                    window.location.replace("/welcome?error=webeid-callback");
+                });
+
+                return;
+            }
+
             const csrfHeaderName = document.querySelector('#csrfheadername').content;
             const csrfToken = document.querySelector('#csrftoken').content;
             const path = window.location.pathname;
@@ -71,7 +101,7 @@ <h2>Completing signing…</h2>
 
         } catch (error) {
             console.error("Mobile callback failed", error);
-            window.location.replace("/welcome?error=mobile-callback");
+            window.location.replace("/welcome?error=webeid-callback");
         }
     })();
 </script>
diff --git a/example/src/main/resources/templates/webeid-login.html b/example/src/main/resources/templates/webeid-login.html
@@ -3,6 +3,8 @@
 <head>
     <meta charset="utf-8">
     <title>Signing you in…</title>
+    <link rel="stylesheet" href="/css/bootstrap.min.css"/>
+    <link rel="stylesheet" href="/css/main.css"/>
 </head>
 <body>
 <div id="error-message" class="alert alert-danger" style="display: none;" role="alert">

Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ public ResponseEntity<MobileInitRequest> mobileInit(WebEidAuthentication authent`
`78`	`78`
`79`	`79`	`@GetMapping("mobile/certificate")`
`80`	`80`	`public ModelAndView mobileCertificateResponse() {`
`81`		`- return new ModelAndView("mobile-callback");`
	`81`	`+ return new ModelAndView("webeid-callback");`
`82`	`82`	`}`
`83`	`83`
`84`	`84`	`@SuppressWarnings("javax.annotation.Tainted")`
`@@ -89,7 +89,7 @@ public ResponseEntity<MobileInitRequest> mobileCertificate(@RequestBody Certific`
`89`	`89`
`90`	`90`	`@GetMapping("mobile/signature")`
`91`	`91`	`public ModelAndView mobileSignatureResponse() {`
`92`		`- return new ModelAndView("mobile-callback");`
	`92`	`+ return new ModelAndView("webeid-callback");`
`93`	`93`	`}`
`94`	`94`
`95`	`95`	`@PostMapping("mobile/signature")`