Merge pull request #115 from ccummings/401-error-with-no-token

client should return a 401 error code when no token is provided
feathersjs · Aug 29, 2018 · f7ca483 · f7ca483
1 parent ce8333d
commit f7ca483
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/packages/authentication/src/client/index.js b/packages/authentication/src/client/index.js
@@ -1,5 +1,6 @@
 import * as hooks from './hooks';
 import { connected, authenticateSocket, getJWT, getStorage } from './utils';
+import errors from 'feathers-errors';
 
 const defaults = {
   tokenKey: 'feathers-jwt',
@@ -25,7 +26,7 @@ export default function(opts = {}) {
       if (!options.type) {
         getOptions = getJWT(config.tokenKey, this.get('storage')).then(token => {
           if (!token) {
-            return Promise.reject(new Error(`Could not find stored JWT and no authentication type was given`));
+            return Promise.reject(new errors.NotAuthenticated(`Could not find stored JWT and no authentication type was given`));
           }
 
           return { type: 'token', token };

diff --git a/packages/authentication/test/client/index.test.js b/packages/authentication/test/client/index.test.js
@@ -76,6 +76,7 @@ const setupTests = initApp => {
       .then(() => done(new Error('Should never get here')))
       .catch(error => {
         assert.equal(error.message, 'Could not find stored JWT and no authentication type was given');
+        assert.equal(error.code, 401);
         done();
       });
   });