search

Particular / NServiceBus

Build, version, and monitor better microservices with the most powerful service platform for .NET
https://particular.net/nservicebus/
Other
2.1k stars 647 forks source link

Throw a meaningful exception when InsertBefore|After targets a terminator #3772

Open andreasohlund opened 8 years ago

andreasohlund commented 8 years ago

Instead of throwing:

Registration 'InvokeHandlers' specified in the insertbefore of the 'LogIncomingMessageBehavior' step does not exist.

we should say:

InvokeHandlers is a terminator and will always be final step in the pipeline, you can safely remove the call to InsertBefore|After from your registration code

boblangley commented 8 years ago

@Particular/tf-v6-launch-plat-dev We decided this does not have to make it into V6 launch and can be done in a minor, but if we can slip it in before launch we will. So we will defer the decision to just in time so we can include it if possible.

tmasternak commented 8 years ago

@Particular/nservicebus-maintainers I've removed the Consider for: Project V6 launch label. If you think that should be done before RC and is small enough please add the Project: V6 launch label yourself.