response-time

response-time copied to clipboard

Version: 2.3.2 ``` (!) Use of eval is strongly discouraged https://rollupjs.org/guide/en/#avoiding-eval ../../../node_modules/response-time/node_modules/depd/index.js 408: 409: // eslint-disable-next-line no-eval 410: var deprecatedfn = eval('(function (' + args + ') {\n' + ^...

AlexBroadbent

Add support for request-received middleware

4

The `request-received` middleware at https://github.com/cabinjs/request-received exposes a `Symbol.for` property, which this package could conditionally consume if interested (as opposed to just generating `process.hrtime()`. e.g. https://github.com/expressjs/response-time/blob/master/index.js#L53 would change from `var startAt...

niftylettuce

Injecting response times into view templates

5

Reminder: https://stackoverflow.com/questions/30019235/how-to-access-all-response-times-on-every-route-in-express-4/30029024#30029024 Thanks @dougwilson.

kevinSuttle

Since the codebase is not returning the `next` explicitly, when you have an async interface involving this library, the Promise exit early getting the implicit `undefined` until now returned.

Kikobeats

Discussion: Proposed Changes for expressjs/response-time Next Major Version

4

This issue aims to discuss potential changes for the next major version of `expressjs/response-time`. ### Key Areas to Consider: 1. API Simplification - BREAKING CHANGE - **Standardized Options Object**: Remove...

carpasse