-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
How to replicate previous api for registering pre/post processors #963
Comments
Dozens? They're supposed to be analogous to filters in ASP.NET Core. Cross cutting concerns across multiple handlers with no domain logic. But yes, because order matters you need to register them explicitly in the order you want them to execute. It was a mistake to ever auto-register them. |
We have generic commands/handlers for CRUD operations and are using PostProcessors for side effects (notifications, updating related records etc.) that are specific to a certain type. We don't have any open PostProcessors and order doesn't matter to us. We are using PipelineBehaviours for cross-cutting concerns. |
Ah jeez my bad I was thinking pipeline behaviors not pre/post processors. One option is to include an "AutoRegisterRequestProcessors" flag or similar so that those can get registered by default as well. Those are ordered as well which is why they were switched to manual, but if every handler only has 0-1 processors then order wouldn't matter. |
This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days. |
@jbogard do I still need to register the behaviours manually? Looks like this was taken out: |
I found a bug which I have fixed: |
This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days. |
This issue was closed because it has been stalled for 14 days with no activity. |
The guide for migrating to v12.1 is a bit unclear to me: https://github.com/jbogard/MediatR/wiki/Migration-Guide-12.0-to-12.1
We have dozens of pre/post processors and this makes it seem as if I would need to add them all individually when upgrading to 12.1
The text was updated successfully, but these errors were encountered: