DuendeSoftware/IdentityServer

Consider allowing multiple IEventSink registrations

AndersAbel opened this issue · 3 comments

Consider allowing multiple IEventSink registrations

Do you think this would be a breaking change? If so, I will push it to the 8.0 milestone.

For normal usage I don't think that this should be breaking. We would change our DefaultEventService to take an IEnumerable<IEventSink> instead of IEventSink. That would not affect any existing users.

If someone did derive from our DefaultEventService it would be a breaking change to them as the constructor parameter changes.

A more in depth analysis need to be done as part of the implementation work.