scratchfoundation/node-redis-rate-limiter
1
0
Fork 0
mirror of https://github.com/scratchfoundation/node-redis-rate-limiter.git synced 2024-12-02 20:17:03 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

Catch invalid rate shorthand notations

Browse source
This commit is contained in:
Romain Prieto 2014-08-22 15:50:49 +10:00
parent 2dbff9abae
commit 8d92f4df7e
2 changed files with 25 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/options.js
View file

@ -21,6 +21,8 @@ exports.canonical = function(opts) {
assert.ok(match, 'Invalid rate: ' + opts.rate); assert.ok(match, 'Invalid rate: ' + opts.rate);
canon.limit = parseInt(match[1], 10); canon.limit = parseInt(match[1], 10);
canon.window = moment.duration(1, match[2]) / 1000; canon.window = moment.duration(1, match[2]) / 1000;
assert.notEqual(canon.limit, 0, 'Invalid rate limit: ' + opts.rate);
assert.notEqual(canon.window, 0, 'Invalid rate window: ' + opts.rate);
} }
// Limit + window // Limit + window

26
test/options.spec.js
View file

@ -95,14 +95,34 @@ describe('Options', function() {
assertRate('5000/d', 5000, 86400); assertRate('5000/d', 5000, 86400);
}); });
it('has to be a valid rate', function() { it('has to be a valid format', function() {
(function() { (function() {
var opts = options.canonical({ var opts = options.canonical({
redis: {}, redis: {},
key: 'ip', key: 'ip',
rate: '50 things' rate: 'foobar'
}); });
}).should.throw('Invalid rate: 50 things'); }).should.throw('Invalid rate: foobar');
});
it('has to be a valid limit', function() {
(function() {
var opts = options.canonical({
redis: {},
key: 'ip',
rate: '0/hour'
});
}).should.throw('Invalid rate limit: 0/hour');
});
it('has to be a valid unit', function() {
(function() {
var opts = options.canonical({
redis: {},
key: 'ip',
rate: '50/century'
});
}).should.throw('Invalid rate window: 50/century');
}); });
}); });