BrighterCommand / Brighter

A framework for building messaging apps with .NET and C#.
https://www.goparamore.io/
MIT License
2.03k stars 257 forks source link

Only configure publisher confirmation callbacks once [v9] #3256

Closed dhickie closed 3 months ago

dhickie commented 3 months ago

This fixes #3246 by ensuring that publish confirmation delegates are only registered once. It does this in a slightly different way than the one that was suggested in the issue, as I noticed that v10 fixes this by moving the callback registration to inside the ExternalBusServices constructor. In v9 ExternalBusServices doesn't have an explicit constructor, so I've moved the call to ConfigureCallbacks to inside the lock where the static instance is initialised in CommandProcessor.