NServiceBus/NServiceBus — comment on commit 41444d10586be258d1ed40b9941eeb02206c7ca5
File:  line 0
By johannesg:

Well, yes, it does work in this particular case. But it could show up again
if an internalized component was registered as, for example, a transient.
It does not seem to be in the saga persister though so maybe it's not
really an issue, but I don't know about other places.

I'm in the nitpick corner, I know, but I still think that internalized
components should be hidden from the container as much as possible to avoid
"leaks". Best way to do that would be to wrap them into another class.
Then, components would not have to be named at all but could use the
container default.

2012/2/5 Andreas hlund <
reply@reply.github.com
>

> I tested and it seems to work without naming the instances. Tried running a
> external raven from the pubsub sample and it worked. Can you do some
> testing and see if there is still an issue?
>
> On Sun, Feb 5, 2012 at 10:20 PM, Johannes Gustafsson <
> reply@reply.github.com
> > wrote:
>
> > Components registered using IContainer.Configure() should also be named,
> > right?
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/NServiceBus/NServiceBus/commit/41444d10586be258d1ed40b9941eeb02206c7ca5#commitcomment-934750
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/NServiceBus/NServiceBus/commit/41444d10586be258d1ed40b9941eeb02206c7ca5#commitcomment-934751
>