Server: fix refreshing token that fails
authorChocobozzz <florian.bigard@gmail.com>
Tue, 1 Nov 2016 17:16:47 +0000 (18:16 +0100)
committerChocobozzz <florian.bigard@gmail.com>
Tue, 1 Nov 2016 17:16:47 +0000 (18:16 +0100)
server/lib/oauth-model.js
server/models/oauth-token.js

index 45f796796ca85880f67183a4af5853649a62008b..922ef0b1f0b588ad1e2277c1d6a143d15b170886 100644 (file)
@@ -60,7 +60,7 @@ function getUser (username, password) {
 }
 
 function revokeToken (token) {
-  return OAuthToken.getByRefreshToken(token.refreshToken).then(function (tokenDB) {
+  return OAuthToken.getByRefreshTokenAndPopulateUser(token.refreshToken).then(function (tokenDB) {
     if (tokenDB) tokenDB.remove()
 
     /*
@@ -72,6 +72,7 @@ function revokeToken (token) {
     */
     const expiredToken = tokenDB
     expiredToken.refreshTokenExpiresAt = new Date('2015-05-28T06:59:53.000Z')
+
     return expiredToken
   })
 }
index 5beb47bedb29ab230a1f93b87839764acdad4d23..04fb9540db06c0ab5c6da62c3d23e7e21f7d328a 100644 (file)
@@ -20,7 +20,7 @@ OAuthTokenSchema.path('user').required(true)
 OAuthTokenSchema.statics = {
   getByRefreshTokenAndPopulateClient,
   getByTokenAndPopulateUser,
-  getByRefreshToken,
+  getByRefreshTokenAndPopulateUser,
   removeByUserId
 }
 
@@ -51,8 +51,8 @@ function getByTokenAndPopulateUser (bearerToken) {
   return this.findOne({ accessToken: bearerToken }).populate('user').exec()
 }
 
-function getByRefreshToken (refreshToken) {
-  return this.findOne({ refreshToken: refreshToken }).exec()
+function getByRefreshTokenAndPopulateUser (refreshToken) {
+  return this.findOne({ refreshToken: refreshToken }).populate('user').exec()
 }
 
 function removeByUserId (userId, callback) {