Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Poison message in MSMQ fails to be sent to remote error queue and is retried indefinitely #4788

Merged
merged 2 commits into from Jun 7, 2017
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
6 changes: 3 additions & 3 deletions src/NServiceBus.Core/Transports/Msmq/ReceiveStrategy.cs
Expand Up @@ -83,7 +83,7 @@ protected bool TryExtractHeaders(Message message, out Dictionary<string, string>

protected void MovePoisonMessageToErrorQueue(Message message, MessageQueueTransaction transaction)
{
var error = $"Message '{message.Id}' is classfied as a poison message and will be moved to '{errorQueue.QueueName}'";
var error = $"Message '{message.Id}' is classified as a poison message and will be moved to the configured error queue.";

Logger.Error(error);

Expand All @@ -92,7 +92,7 @@ protected void MovePoisonMessageToErrorQueue(Message message, MessageQueueTransa

protected void MovePoisonMessageToErrorQueue(Message message, MessageQueueTransactionType transactionType)
{
var error = $"Message '{message.Id}' is classfied as a poison message and will be moved to '{errorQueue.QueueName}'";
var error = $"Message '{message.Id}' is classified as a poison message and will be moved to the configured error queue.";

Logger.Error(error);

Expand Down Expand Up @@ -149,4 +149,4 @@ static async Task<byte[]> ReadStream(Stream bodyStream)

static ILog Logger = LogManager.GetLogger<ReceiveStrategy>();
}
}
}