make it clear that the user needs to deal with NServiceBus messages in addition to native messages

[PhilBastian]

related to Particular/NServiceBus.AzureStorageQueues#1264

[jpalac]

This seems to be the same as the paragraph above or am I not seeing the difference?

[PhilBastian]

one less sentence in the warning

[jpalac]

I see it now - wouldn't it be the same in both though it we are backporting the change?

[abparticular]

I'm not sure on the structure of the snippet/sample. Shouldn't it be something more like (pseudocode)

return CanDeserialize(body) ? new MessageWrapper() : null

or

try
{
  return new Deserialize(body);
}
catch
{
  return null;
}

I'm concerned about the focus on checking for base64 (an internal implementation detail) rather than making it clear that if the message can't be handled by the custom unwrapper, then null should be returned.

[abparticular]

Sorry I didn't mean to approve this PR