You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Jan 24, 2021. It is now read-only.
Now that we have added the new route syntax, we need to make sure that the Before and After hooks are brought up to par. I propose the following changes
Perform an overview of the declaration syntax and make it align with the route syntax better. This means supporting various overloads for sync and async versions
Make it possible for them to be part of the ordinary route processing flow, meaning they would be able to pass through the content negotiation system. Currently the content negotiation is performed in the DefaultRouteInvoker, but hook handling is done in the InvokePreRequestHook and InvokePostRequestHook of NancyEngine (they're called before/after the DefaultRouteInvoker is called).
The text was updated successfully, but these errors were encountered:
Now that we have added the new route syntax, we need to make sure that the
Before
andAfter
hooks are brought up to par. I propose the following changesDefaultRouteInvoker
, but hook handling is done in theInvokePreRequestHook
andInvokePostRequestHook
ofNancyEngine
(they're called before/after theDefaultRouteInvoker
is called).The text was updated successfully, but these errors were encountered: