search

bithavoc / express-winston

express.js middleware for winstonjs
https://www.npmjs.com/package/express-winston
MIT License
798 stars 186 forks source link

Use req.originalUrl instead of req.url when available #75

Closed Carlangueitor closed 9 years ago

Carlangueitor commented 9 years ago

Fixes the problems with nested routers (see #57) in default and express formats.

Carlangueitor commented 9 years ago

cc @bithavoc @floatingLomas

floatingLomas commented 9 years ago

Instead of adding all the burden of evaluate, can we just do a req.url = req.url || req.originalUrl; as soon as we start building messages?

floatingLomas commented 9 years ago

(I meant them the other way around)

Carlangueitor commented 9 years ago

Yeah! Much better.

floatingLomas commented 9 years ago

Merged at EPCOT Center. Lolz.

Carlangueitor commented 9 years ago

Cool! thanks! :+1: