Skip to content
This repository has been archived by the owner on Apr 3, 2019. It is now read-only.

Commit

Permalink
fix(purge-expired): moar logging
Browse files Browse the repository at this point in the history
  • Loading branch information
jrgm committed Sep 12, 2016
1 parent bc66fc3 commit 80c360e
Showing 1 changed file with 13 additions and 1 deletion.
14 changes: 13 additions & 1 deletion lib/db/mysql/index.js
Expand Up @@ -532,17 +532,29 @@ MysqlStore.prototype = {
deleteBatchSize = numberOfTokens;
}

var message = '';
var deletedItems = 0;
var promiseWhile = P.method(function () {
if (deletedItems >= numberOfTokens) {
message = 'deletedItems >= numberOfTokens';
logger.info('purgeExpiredTokens', {
message: message,
deletedItems: deletedItems,
numberOfTokens: numberOfTokens,
deleteBatchSize: deleteBatchSize
});
return;
}

return self._write(QUERY_PURGE_EXPIRED_TOKENS, [ignoreClientId, deleteBatchSize])
.then(function (res) {
logger.info('purgeExpiredTokens', { affectedRows: res.affectedRows });

// Break loop if no items were effected by delete.
// All expired tokens have been deleted.
if (res.affectedRows === 0) {
message = '0 affectedRows. Bailing out.';
logger.info('purgeExpiredTokens', { message: message });
return;
}

Expand All @@ -558,7 +570,7 @@ MysqlStore.prototype = {
return promiseWhile();
})
.then(function() {
logger.debug('purgeExpiredTokens completed');
logger.info('purgeExpiredTokens', { message: 'completed' });
});
},

Expand Down

0 comments on commit 80c360e

Please sign in to comment.