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.

Sign up for GitHub

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

Jump to bottom

Corrupted MSMQ Message Headers may result in message loss #2584

Closed
SimonCropp opened this issue Nov 24, 2014 · 1 comment
Closed

Corrupted MSMQ Message Headers may result in message loss #2584

SimonCropp opened this issue Nov 24, 2014 · 1 comment
Labels
Bug
Milestone
4.1.7

Comments

@SimonCropp
Copy link
Contributor

Who’s affected:

Anyone using MSMQ Transport and is also doing one of the following:

  • Manual editing of the headers of messages which may result in an invalid xml of the headers being sent to the endpoint.
  • Anyone dealing with an integration scenario where native message are being sent for example from Java where the headers may not contain valid xml.

Symptoms:

The message will be discarded.

Fixed so when NServiceBus.MsmqUtilities.Convert throws the message will be sent to the error queue

Clone of #2463
@SimonCropp SimonCropp added the Bug label Nov 24, 2014
@SimonCropp SimonCropp added this to the 4.1.7 milestone Nov 24, 2014
SimonCropp added a commit that referenced this issue Nov 28, 2014
@SimonCropp
Throw from NServiceBus.MsmqUtilities.Convert will result in message loss
caea668 
fixes #2584
@indualagarsamy indualagarsamy changed the title Throw from NServiceBus.MsmqUtilities.Convert will result in message loss Corrupted MSMQ Message Headers may result in message loss Mar 28, 2015
@indualagarsamy
Copy link
Contributor

Closing issue - fixed by the referenced commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug
Projects
None yet
Milestone
4.1.7
Development

No branches or pull requests
2 participants
@SimonCropp @indualagarsamy