Extended IManagedMessageFailures to include numberOfAttempts

Refactored FaultManager to create proper error messages in case FLR is disabled.
Could we use a header instead to avoid the breaking change for now?

On Wed, Oct 1, 2014 at 5:52 AM, Szymon Pobiega <notifications@github.com>
wrote:

> In src/NServiceBus.Core/Faults/IManageMessageFailures.cs:
>
> > @@ -15,7 +15,7 @@ public interface IManageMessageFailures
> >          /// <summary>
> >          /// Invoked when a message has failed its processing the maximum number of time configured.
> >          /// </summary>
> > -        void ProcessingAlwaysFailsForMessage(TransportMessage message, Exception e);
> > +        void ProcessingAlwaysFailsForMessage(TransportMessage message, Exception e, int numberOfAttempts);
>
> should I move the associated issue out of 5.1?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Particular/NServiceBus/pull/2434/files#r18269476>.
>