Fix a bug with del on memcached

Author: AdamPflug

README.md

@@ -143,6 +143,9 @@ app.post('/auth',
 
 Changelog
 ---------
+### v0.4.2
+* BUG: In some cases when no callbacks were supplied memcached would drop the request. Ensure that memcached always sees a callback expressBrute isn't given one.
+
 ### v0.4.1
 * NEW: `refreshTimeoutOnRequest` option that allows you to prevent the remaining `lifetime` for a timer from being reset on each request (useful for implementing limits for set time frames, e.g. requests per day)
 * BUG: Lifetimes were not previously getting extended properly for instances of `ExpressBrute.MemoryStore`

lib/MemcachedStore.js

@@ -13,7 +13,9 @@ var MemcachedStore = module.exports = function (hosts, options) {
 MemcachedStore.prototype.__proto__ = AbstractClientStore.prototype;
 MemcachedStore.prototype.Memcached = Memcached;
 MemcachedStore.prototype.set = function (key, value, lifetime, callback) {
-	this.client.set(this.options.prefix+key, JSON.stringify(value), lifetime || 0, callback);
+	this.client.set(this.options.prefix+key, JSON.stringify(value), lifetime || 0, function (err, data) {
+		typeof callback == 'function' && callback.apply(this, arguments);
+	});
 };
 MemcachedStore.prototype.get = function (key, callback) {
 	this.client.get(this.options.prefix+key, function (err, data) {
@@ -30,7 +32,9 @@ MemcachedStore.prototype.get = function (key, callback) {
 	});
 };
 MemcachedStore.prototype.reset = function (key, callback) {
-	this.client.del(this.options.prefix+key, callback);
+	this.client.del(this.options.prefix+key, function (err, data) {
+		typeof callback == 'function' && callback.apply(this, arguments);
+	});
 };
 MemcachedStore.defaults = {
 	prefix: ''

package.json

@@ -1,6 +1,6 @@
 {
   "name": "express-brute",
-  "version": "0.4.1",
+  "version": "0.4.2",
   "description": "A brute-force protection middleware for express routes that rate limits incoming requests",
   "keywords": [
     "brute",