rebus-org / Rebus.ServiceProvider

:bus: Microsoft Extensions Dependency Injection container adapter for Rebus
https://mookid.dk/category/rebus
Other
67 stars 34 forks source link

Make ServiceProvider available in outgoing steps again #68

Closed SamPersson closed 2 years ago

SamPersson commented 2 years ago

Hello! When upgrading to version 8 I noticed the service provider was no longer available in outgoing steps. In version 7 and earlier it was available in both incoming and outgoing steps. It's not that hard to work around, but unless there was a reason for changing it maybe the old behavior could just be restored like this?


Rebus is MIT-licensed. The code submitted in this pull request needs to carry the MIT license too. By leaving this text in, I hereby acknowledge that the code submitted in the pull request has the MIT license and can be merged with the Rebus codebase.

CLAassistant commented 2 years ago

CLA assistant check
All committers have signed the CLA.

mookid8000 commented 2 years ago

Oh, that's definitely an oversight! Thanks for fixing it 👍 could you sign the CLA thing above? Then I will merge it & build a new package right away.

SamPersson commented 2 years ago

I signed it! Thank you 🙂

SamPersson commented 2 years ago

@mookid8000 Is this waiting for anything else? 🙂

mookid8000 commented 2 years ago

Sorry! 😅 It's out as Rebus.ServiceProvider 8.3.0 on NuGet.org now! Thanks for fixing this 👍