requarks / express-brute-loki

A Loki.js store for Express-Brute

Requarks

Requests and Bugs Tracking Management System

Under development

Milestones

• Base Layout
• Setup
• Dashboard
  • Rotating backgrounds
  • Stats
  • Todos
  • Pinned
  • Active Projects
• Global Search
• Create
  • Category selection screen
  • Request
  • Attachments
• Review
  • List view
  • Filters
  • Search
• Item
• Projects
  • Project
  • Sprints
  • Create Project
  • Edit Project
  • Add Teams / Members
• Teams
  • Team
  • Create Team
  • Edit Team
  • Add Team Members
• Settings
  • Profile
  • Avatar
  • Preferences
  • Notifications
• Administration

express-brute-loki's Issues

moment not required

There is no require for the moment library used at 79:24

lifetime = lifetime ? moment().add(lifetime, 'seconds').toDate() : undefined;

Adding the following fixes it:

moment = require('moment');

Moreover this line should became like this to avoid a warning:

lifetime = lifetime ? moment().add(lifetime, 'seconds').toDate() : undefined;

Missing moment ffs.

value.lastRequest.getTime is not a function

Unsure what's causing this, but occurs when handling requests with this module. Confirmed the expected behaviour when using express-brute with MemoryStore().

TypeError: value.lastRequest.getTime is not a function
    at .<anonymous> (/home/xxx/node_modules/express-brute/index.js:100:47)
    at LokiStore.get (/home/xxx/node_modules/express-brute-loki/lib/express-brute-loki.js:111:3)
    at .<anonymous> (/home/xxx/node_modules/express-brute/index.js:82:15)
    at keyFunc (/home/xxx/node_modules/express-brute/index.js:39:41)
    at .<anonymous> (/home/xxx/node_modules/express-brute/index.js:47:3)
    at Layer.handle [as handle_request] (/home/xxx/node_modules/express/lib/router/layer.js:95:5)
    at next (/home/xxx/node_modules/express/lib/router/route.js:137:13)
    at Route.dispatch (/home/xxx/node_modules/express/lib/router/route.js:112:3)
    at Layer.handle [as handle_request] (/home/xxx/node_modules/express/lib/router/layer.js:95:5)
    at /home/xxx/node_modules/express/lib/router/index.js:281:22

Code:

var store = new ExpressBruteLokiStore({
  path: './brute.db'
});
//var store = new ExpressBrute.MemoryStore(); // stores state locally, don't use this in production 
var bruteforce = new ExpressBrute(store);

app.post('/auth',
    bruteforce.prevent, // error 429 if we hit this route too often 
    function (req, res, next) {
        res.send('Success!');
    }
);

ReferenceError: sid is not defined

At line 81 sid there is an error:

let s = this.collection.find({ sid });

should probably became like this:

let s = this.collection.find({ sid: key });