refactor: add redirect_uri, response_type and grant_type to server-si…

…de authentication request (#166)
nuxt-community · May 7, 2018 · 0602c60 · 0602c60
1 parent 8e9f414
commit 0602c60
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 65 deletions.
diff --git a/lib/providers/_utils.js b/lib/providers/_utils.js
@@ -33,7 +33,12 @@ function addAuthorize (strategy) {
       }
 
       formMiddleware(req, res, () => {
-        const { code } = req.body
+        const {
+          code,
+          redirect_uri: redirectUri = strategy.redirect_uri,
+          response_type: responseType = strategy.response_type,
+          grant_type: grantType = strategy.grant_type
+        } = req.body
 
         if (!code) {
           return next()
@@ -46,6 +51,9 @@ function addAuthorize (strategy) {
             data: {
               client_id: clientID,
               client_secret: clientSecret,
+              grant_type: grantType,
+              response_type: responseType,
+              redirect_uri: redirectUri,
               code
             },
             headers: {

diff --git a/lib/providers/laravel.passport.js b/lib/providers/laravel.passport.js
@@ -1,6 +1,4 @@
-const axios = require('axios')
-const bodyParser = require('body-parser')
-const { assignDefaults } = require('./_utils')
+const { assignDefaults, addAuthorize } = require('./_utils')
 
 module.exports = function laravelPassport (strategy) {
   assignDefaults(strategy, {
@@ -15,65 +13,5 @@ module.exports = function laravelPassport (strategy) {
     scope: '*'
   })
 
-  // Get client_secret, client_id and token_endpoint
-  const clientSecret = strategy.client_secret
-  const clientID = strategy.client_id
-  const tokenEndpoint = strategy.token_endpoint
-
-  // IMPORTANT: remove client_secret from generated bundle
-  delete strategy.client_secret
-
-  // Endpoint
-  const endpoint = `/_auth/oauth/${strategy._name}/authorize`
-  strategy.access_token_endpoint = endpoint
-
-  // Set response_type to code
-  strategy.response_type = 'code'
-
-  // Form parser
-  const formMiddleware = bodyParser.urlencoded()
-
-  // Register endpoint
-  this.options.serverMiddleware.unshift({
-    path: endpoint,
-    handler: (req, res, next) => {
-      if (req.method !== 'POST') {
-        return next()
-      }
-
-      formMiddleware(req, res, () => {
-        const {
-          code,
-          redirect_uri: redirectUri = strategy.redirect_uri,
-          response_type: responseType = strategy.response_type,
-          grant_type: grantType = strategy.grant_type
-        } = req.body
-
-        if (!code) {
-          return next()
-        }
-
-        axios
-          .request({
-            method: 'post',
-            url: tokenEndpoint,
-            data: {
-              client_id: clientID,
-              client_secret: clientSecret,
-              grant_type: grantType,
-              response_type: responseType,
-              redirect_uri: redirectUri,
-              code
-            },
-            headers: {
-              Accept: 'application/json'
-            }
-          })
-          .then(response => {
-            res.end(JSON.stringify(response.data))
-          })
-          .catch(error => next(error))
-      })
-    }
-  })
+  addAuthorize.call(this, strategy)
 }