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
Thanks, @MatthewLymer. I'm trying to recall if I had a reason for this and I can't think of one. I think it would be reasonable to make it an internal setter. Since this would be a breaking change, it would need to go into a v4 release. I'd welcome a PR against the v4 branch if you have time. Thanks!
Hello and thank you for the awesome library, it's very useful for me!
One thing I noticed is that
ICorrelationContextAccessor
has a setter on its only propertyCorrelationContext
.When this dependency is injected into a class, is it expected that the dependent type is able to modify the context via the interface?
My thinking is that dependents that need the correlation context shouldn't be able to change it too.
Can you clarify whether this is intended, or whether you'd accept a PR to change this behavior?
Cheers,
Matthew
The text was updated successfully, but these errors were encountered: