fix (admin): wrong login shouldn't display the error notification

mickael-kerjean · Oct 12, 2023 · 0f50300 · 0f50300
1 parent 1aad868
commit 0f50300
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/public/pages/adminpage/ctrl_login.js b/public/pages/adminpage/ctrl_login.js
@@ -42,9 +42,14 @@ export default async function(render) {
         rxjs.map(() => ({ password: qs($form, "[name=\"password\"]").value })),
         rxjs.switchMap((creds) => authenticate$(creds).pipe(
             rxjs.catchError((err) => {
-                if (err instanceof AjaxError && err.code() === "INTERNAL_SERVER_ERROR") {
-                    ctrlError(err)(render);
-                    return rxjs.EMPTY;
+                if (err instanceof AjaxError) {
+                    switch(err.code()) {
+                    case "INTERNAL_SERVER_ERROR":
+                        ctrlError(err)(render);
+                        return rxjs.EMPTY;
+                    case "FORBIDDEN":
+                        return rxjs.of(false);
+                    }
                 }
                 notification.error(err && err.message);
                 return rxjs.of(false);