Restify req.url broken by #29

Question

Restify req.url broken by #29

rektide opened this issue 3 years ago · 2 comments

Hi, apologies,

In #29, req.path was made preferred to req.url, for finding the url. In restify however req.path is a fucntion (this link is to current master branch). This causes the url not to be logged for restify systems.

Answer 1 · 2022-02-22T19:20:57.000Z

Would you like to send a Pull Request to address this issue? Remember to add unit tests.

Answer 2 · 2022-02-22T19:42:59.000Z

Working on it, sure. I plan on testing req.path if present to see if it's a function. This feels like it'll have the highest compatibility, at the cost of some speed. Other implementation suggestions welcome.