-
Notifications
You must be signed in to change notification settings - Fork 307
Profiling/observing the middleware chain #318
Comments
Bump @Eilon @muratg. Note this has been designed by @davidfowl and @lodejard. |
cc @Tratcher |
@davidfowl @lodejard Your design details were never captured. We need to meet up and re-hash them. |
For my part, the general concept was that we, the user or any other interested parties could "touch" a given point which would setup the one " Besides just raw timing data, part of the trick was also to try and get a meaningful name out. We had discussed that this could be done by convention and any middleware author could opt to have a name given to their middleware. Exactly how this was done, wasn't exactly settled but we had talked about maybe using |
@glennc Can you schedule a meeting to finalize the design here? @avanderhoorn and @davidfowl already met before and had a sketch for a design, but let's get it decided on and written down. |
@avanderhoorn Unfortunately we haven't yet had a meeting to close on the design. cc @davidfowl |
We and other profiling/debugging groups/teams/projects have the need to be able to observe the ins/outs of each middleware in the chain as its excited. Previous attempts at this required each listener to replace the default app builder and wrap/modify the middleware. This is fragile and breaks down when multiple observers are in play.
This hook/event/notification should also make available the HttpContext, so if specific consumers want to observe changes that middleware makes to the context it can. Additionally, it should make available contextual details that the middleware can opt into providing (specifically, a meaningful name and any other data the middleware want to make available to consumers - for instance settings, config, decisions, etc).
Pretty much across the board, this is the top request we have from users when speaking about what they want to see when creating vNext apps.
Had this discussion with @davidfowl @rynowak @anurse @lodejard @nikmd23
The text was updated successfully, but these errors were encountered: