thenativeweb / get-routes Goto Github PK

var express = require("express");
var Router = express.Router;

var app = express();
var router = new Router();
router.get("/test", (req, res) => { res.send(); });

app.use(router);
// ---------
var getRoutes = require("get-routes");
console.log(routes); // empty

It reads that middleware.route is empty, but doesn't check to see if middleware.handler is of type Router.

I don't know if this still works. To do something similar I had to use the following code

    const getRouter = () => {
      return app._router
    }

    getRouter().stack.forEach(function (r) {
      if (r.handle.stack)
        r.handle.stack.forEach((item) => {
          if (item.route)
            console.log(item.route.methods,item.route.path)
        })
    })

Which is completely different from your code? I am running "express": "^4.16.2"

For routes that use the "all" method get-routes fails with "Invalid method acl". This applies to routes like this:

app.all('/foo/bar', foo.bar);

Either just pull in the one function needed - https://www.npmjs.com/package/lodash.forown

orrr

replace with for of:

for (const middleware of app._router.stack) { processMiddleware(middleware); }

So I can do a lot more with your version than https://github.com/vijay4495/express-routes-catalogue/blob/master/src/index.ts#L85-L92 but they handle the nested router as recommended by the express generator

app = express();
const indexRouter = express.Router();
indexRouter .get('/', (req, res) => res.send("OK"));
app.use('/', indexRouter);

I think look you just need to loop through middleware.handle.stack if it comes up.. I'll try to get a PR up

Whenever I get all routes in my projects, it gives me a text "... 7 more items"

Would it be possible to cut a new release for this package? I would like to use the updated dependencies due to some security issues with the version of lodash that is being used in the currently published version.

Thanks!