search

tuupola / server-timing-middleware

PSR-7 & PSR-15 middleware to add the Server-Timing header
MIT License
198 stars 9 forks source link

generate more than 1 Server-Timing header #27

Open DanielRuf opened 2 months ago

DanielRuf commented 2 months ago

It would be good to have it configurable, if one (big) or multiple (short) headers should be created.

https://github.com/tuupola/server-timing-middleware/blob/master/src/ServerTimingMiddleware.php#L133

DanielRuf commented 2 months ago

See also https://github.com/tuupola/server-timing-middleware/issues/25#issuecomment-2102056831

tuupola commented 2 months ago

Yeah, agreed. Quickly thinking having one flag for choosing between single header and multiple headers would be a clean solution. Having a configurable header size in bytes feels more error prone.