Cancellation token in exception handlers

[IharBury]

We would like to use delays in our exception handlers. However currently there is no easy way to abort those delays when the subscription is terminated. Adding a CancellationToken to the delegate would fix this issue.

[IharBury]

The same issue exists for the ShouldRetry delegate in the projector.

[dennisdoomen]

Good point. Would be a breaking chance unfortunately.

[IharBury]

Would be a breaking chance unfortunately.

We could just add new delegates and keep the old ones.

[IharBury]

Discussed with @dennisdoomen. We will extend SubscriptionInfo with the cancellation token. That will fix the issue with HandleException delegate. We will the issue in projector specific delegates later (or maybe we will not fix it at all there) because the Dispatcher level exception handling should be enough for most use cases.

[dennisdoomen]

So this is now resolved?

[IharBury]

Unless we want to provide the cancellation token for projector specific error handlers, we can close all the three issues.

[dennisdoomen]

Don't see any priority in this now.