revoke-handler: throw InvalidClientError if client_id does not match with token.client.id

Author: visvk

lib/handlers/revoke-handler.js

@@ -239,7 +239,7 @@ RevokeHandler.prototype.getRefreshToken = function(token, client) {
       }
 
       if (token.client.id !== client.id) {
-        throw new InvalidTokenError('Invalid token: refresh token client is invalid');
+        throw new InvalidClientError('Invalid client: client is invalid');
       }
 
       if (token.refreshTokenExpiresAt && !(token.refreshTokenExpiresAt instanceof Date)) {
@@ -274,7 +274,7 @@ RevokeHandler.prototype.getAccessToken = function(token, client) {
       }
 
       if (accessToken.client.id !== client.id) {
-        throw new InvalidTokenError('Invalid token: access token client is invalid');
+        throw new InvalidClientError('Invalid client: client is invalid');
       }
 
       if (accessToken.accessTokenExpiresAt && !(accessToken.accessTokenExpiresAt instanceof Date)) {

test/integration/handlers/revoke-handler_test.js

@@ -635,8 +635,8 @@ describe('RevokeHandler integration', function() {
       return handler.getRefreshToken('hash', client)
         .then(should.fail)
         .catch(function(e) {
-          e.should.be.an.instanceOf(InvalidTokenError);
-          e.message.should.equal('Invalid token: refresh token client is invalid');
+          e.should.be.an.instanceOf(InvalidClientError);
+          e.message.should.equal('Invalid client: client is invalid');
         });
     });
 
@@ -712,8 +712,8 @@ describe('RevokeHandler integration', function() {
       return handler.getAccessToken('hash', client)
         .then(should.fail)
         .catch(function(e) {
-          e.should.be.an.instanceOf(InvalidTokenError);
-          e.message.should.equal('Invalid token: access token client is invalid');
+          e.should.be.an.instanceOf(InvalidClientError);
+          e.message.should.equal('Invalid client: client is invalid');
         });
     });