Merge pull request #26 from fleetbase/dev-v0.2.0

v0.2.0
fleetbase · Jan 18, 2024 · c5e7463 · c5e7463
2 parents 21c722d + fc33971
commit c5e7463
Show file tree

Hide file tree

Showing 10 changed files with 38 additions and 85 deletions.
diff --git a/addon/adapters/application.js b/addon/adapters/application.js
@@ -95,7 +95,9 @@ export default class ApplicationAdapter extends RESTAdapter {
             const localStorageSession = JSON.parse(window.localStorage.getItem('ember_simple_auth-session'));
             if (localStorageSession) {
                 const { authenticated } = localStorageSession;
-                token = authenticated.token;
+                if (authenticated) {
+                    token = authenticated.token;
+                }
 
                 // Check isAuthenticated again
                 isAuthenticated = !!token;

diff --git a/addon/authenticators/fleetbase.js b/addon/authenticators/fleetbase.js
@@ -1,5 +1,17 @@
 import Base from 'ember-simple-auth/authenticators/base';
 import { inject as service } from '@ember/service';
+import getWithDefault from '../utils/get-with-default';
+
+export class AuthenticationError extends Error {
+    constructor(message, code) {
+        super(message);
+        this.code = code;
+    }
+
+    getCode() {
+        return this.code;
+    }
+}
 
 export default class FleetbaseAuthenticator extends Base {
     /**
@@ -35,7 +47,7 @@ export default class FleetbaseAuthenticator extends Base {
             )
             .then((response) => {
                 if (response.restore === false) {
-                    return Promise.reject(new Error(response.error));
+                    return Promise.reject(new AuthenticationError(response.error));
                 }
 
                 return response;
@@ -52,7 +64,10 @@ export default class FleetbaseAuthenticator extends Base {
     authenticate(credentials = {}, remember = false, path = 'auth/login') {
         return this.fetch.post(path, { ...credentials, remember }).then((response) => {
             if (response.errors) {
-                return Promise.reject(new Error(response.errors.firstObject ?? 'Authentication failed!'));
+                const errorMessage = getWithDefault(response.errors, '0', 'Authentication failed!');
+                const errorCode = getWithDefault(response, 'code');
+
+                return Promise.reject(new AuthenticationError(errorMessage, errorCode));
             }
 
             return response;

diff --git a/addon/services/session.js b/addon/services/session.js
@@ -2,6 +2,7 @@ import SimpleAuthSessionService from 'ember-simple-auth/services/session';
 import { tracked } from '@glimmer/tracking';
 import { inject as service } from '@ember/service';
 import { later } from '@ember/runloop';
+import getWithDefault from '../utils/get-with-default';
 
 export default class SessionService extends SimpleAuthSessionService {
     /**
@@ -97,12 +98,12 @@ export default class SessionService extends SimpleAuthSessionService {
             const user = await this.currentUser.load();
 
             if (!user) {
-                return this.invalidateWithLoader(`Session authentication failed...`);
+                return this.invalidateWithLoader('Session authentication failed...');
             }
 
             return user;
         } catch (error) {
-            await this.invalidateWithLoader(error.message ?? `Session authentication failed...`);
+            await this.invalidateWithLoader(getWithDefault(error, 'message', 'Session authentication failed...'));
         }
     }
 
@@ -124,7 +125,7 @@ export default class SessionService extends SimpleAuthSessionService {
                             transition.abort();
                         }
 
-                        reject(invalidateWithLoader(`Session authentication failed...`));
+                        reject(invalidateWithLoader('Session authentication failed...'));
                     }
 
                     resolve(user);
@@ -218,4 +219,17 @@ export default class SessionService extends SimpleAuthSessionService {
 
         return Math.round((now - date) / 1000);
     }
+
+    /**
+     * Checks for the presence of two-factor authentication for a given user identity.
+     *
+     * @param {String} identity
+     * @return {Promise}
+     * @throws {Error}
+     */
+    checkForTwoFactor(identity) {
+        return this.fetch.get('two-fa/check', { identity }).catch((error) => {
+            throw new Error(error.message);
+        });
+    }
 }
diff --git a/addon/utils/extract-latitude.js b/addon/utils/extract-latitude.js
diff --git a/addon/utils/extract-longitude.js b/addon/utils/extract-longitude.js
diff --git a/app/utils/extract-latitude.js b/app/utils/extract-latitude.js
diff --git a/app/utils/extract-longitude.js b/app/utils/extract-longitude.js
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@fleetbase/ember-core",
-  "version": "0.1.9",
+  "version": "0.2.0",
   "description": "Provides all the core services, decorators and utilities for building a Fleetbase extension for the Console.",
   "keywords": [
     "fleetbase-core",

diff --git a/tests/unit/utils/extract-latitude-test.js b/tests/unit/utils/extract-latitude-test.js
diff --git a/tests/unit/utils/extract-longitude-test.js b/tests/unit/utils/extract-longitude-test.js