r3wt/express-map

cool routing/autoloading library for express.

• 👋 Hi, I’m @r3wt
• 👀 I’m interested in React, Typescript, Rust, Scala, C++
• 🌱 I’m currently learning Unreal Engine 5
• 💞️ I’m looking to collaborate on Open Source Projects
• 📫 How to reach me [email protected]

express-map's Issues

improve reliability of parsing handlers.

Strings should be trimmed on both sides of each parsed item.

add testing framework

line 131 references `this` inside of Array.map() callback function

I'm suprised it actually still works. should be replaced with _self, since _self is the express app in the calling scope.

change to Array.isArray()

express-map/index.js

Line 66 in 9ffb762

if(!(middleware instanceof Array)){

investigate leaking global cPath

express-map/index.js

Line 12 in 2c4e99e

let cPath = false;//cache controller path

I'd have to check but i believe because of block scoping of let this is potentially creating a global cPath variable.

if module does not export the callable, `undefined` is displayed as controller name

possible features:

1. parsing string global middleware. currently only function middleware are able to be used.

currently supported

var Middleware = require('controllers/Middleware');

app.map('/api/v1/test', Middlewarer.parseJWT, Middleware.isUser, {
 'GET /user':'User.loadOne'
});

proposed support

app.map('/api/v1/test', 'Middlewarer.parseJWT, Middleware.isUser', {
 'GET /user':'User.loadOne'
});

2. support comma seperated http methods. make sure to prevent duplicate handlers.

currently supported

app.map('/api/v1/test',{
  'GET /:id':'Test.get',
  'POST /:id':'Test.create',
  'PUT /:id':'Test.update',
  'Delete /:id':'Test.delete'
});

proposed support

app.map('/api/v1/test',{
  'GET, POST, PUT, DELETE /:id':'Test.crud'
});