feat(core): return response from loginWith

docs/schemes/local.md

@@ -35,6 +35,7 @@ auth: {
       },
       // tokenRequired: true,
       // tokenType: 'bearer'
+      // autoFetchUser: true
     }
   }
 }
@@ -83,3 +84,9 @@ This option can be used to disable all token handling. Useful for Cookie only fl
 - Default: `Bearer`
 
  Authorization header type to be used in axios requests.
+
+ ### `autoFetchUser`
+
+ - Default: `true`
+
+ This option can be used to disable user fetch after login. It is useful when your login response already have the user.

lib/core/auth.js

@@ -128,10 +128,12 @@ export default class Auth {
       return Promise.resolve()
     }
 
-    return this.wrapLogin(this.strategy.login(...arguments)).catch(error => {
-      this.callOnError(error, { method: 'login' })
-      return Promise.reject(error)
-    })
+    return this.wrapLogin(this.strategy.login(...arguments))
+      .then(response => response)
+      .catch(error => {
+        this.callOnError(error, { method: 'login' })
+        return Promise.reject(error)
+      })
   }
 
   fetchUser () {
@@ -259,7 +261,7 @@ export default class Auth {
     return this.$storage.getState('busy')
   }
 
-  request (endpoint, defaults) {
+  request (endpoint, defaults, withResponse) {
     const _endpoint =
       typeof defaults === 'object'
         ? Object.assign({}, defaults, endpoint)
@@ -274,10 +276,15 @@ export default class Auth {
     return this.ctx.app.$axios
       .request(_endpoint)
       .then(response => {
-        if (_endpoint.propertyName) {
-          return getProp(response.data, _endpoint.propertyName)
+        const result = _endpoint.propertyName ? getProp(response.data, _endpoint.propertyName) : response.data
+
+        if (withResponse) {
+          return {
+            response,
+            result
+          }
         } else {
-          return response.data
+          return result
         }
       })
       .catch(error => {
@@ -310,8 +317,9 @@ export default class Auth {
     this.error = null
 
     return Promise.resolve(promise)
-      .then(() => {
+      .then(response => {
         this.$storage.setState('busy', false)
+        return response
       })
       .catch(error => {
         this.$storage.setState('busy', false)

lib/schemes/local.js

@@ -37,9 +37,10 @@ export default class LocalScheme {
     // Ditch any leftover local tokens before attempting to log in
     await this._logoutLocally()
 
-    const result = await this.$auth.request(
+    const { response, result } = await this.$auth.request(
       endpoint,
-      this.options.endpoints.login
+      this.options.endpoints.login,
+      true
     )
 
     if (this.options.tokenRequired) {
@@ -51,7 +52,11 @@ export default class LocalScheme {
       this._setToken(token)
     }
 
-    return this.fetchUser()
+    if (this.options.autoFetchUser) {
+      await this.fetchUser()
+    }
+
+    return response
   }
 
   async setUserToken (tokenValue) {
@@ -116,5 +121,6 @@ const DEFAULTS = {
   tokenRequired: true,
   tokenType: 'Bearer',
   globalToken: true,
-  tokenName: 'Authorization'
+  tokenName: 'Authorization',
+  autoFetchUser: true
 }