Only scenario I can think of where MessageId is missing is native
integration via eg Rabbit. But I need to check, lets remove it


On Mon, May 5, 2014 at 5:57 AM, John Simons <notifications@github.com>wrote:

> In
> src/NServiceBus.NHibernate/TimeoutPersisters/NHibernate/TimeoutStorage.cs:
>
> >
> > +            string messageId;
> > +            if (timeout.Headers.TryGetValue(Headers.MessageId, out messageId))
> > +            {
> > +                Guid.TryParse(messageId, out timeoutId);
> > +
> > +            }
> > +
> > +            if (timeoutId == Guid.Empty)
> > +            {
> > +                timeoutId = Guid.NewGuid();
>
> Exactly, we either have a messageId and use that or we need to remove
> TransactionScopeOption.Suppress
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Particular/NServiceBus.NHibernate/pull/21/files#r12266851>
> .
>