Allow overriding native MessageId, ContentType and CorrelationID #1071
Conversation
| var properties = outgoingTransportOperation.Properties; | ||
| var headers = outgoingTransportOperation.Message.Headers; | ||
|
|
||
| if (properties.TryGetValue(Headers.CorrelationId, out var correlationId)) |
There was a problem hiding this comment.
As far as I recall, the properties get serialized into the outbox. It might be worthwhile considering removing those temporary properties after reading them or add a comment why they need to be preserved
There was a problem hiding this comment.
Quick question. Technically, already today, it is possible to add a customization delegate that allows you to customize the entire native message created. Any reason this can't be used?
There was a problem hiding this comment.
And yes, it seems the customizer should also be removed from the properties dictionary because I doubt we can every serialize action delegates into outbox records.
There was a problem hiding this comment.
- I didn't know that
TransportTransactionwas itself aContextBag🤯 - Looked at the type you linked but I haven't figured out how that could be used as so far all invoked types I see are internal
So, I need additional research to validate if this could be used for the ******* connector.
irinascurtu
requested review from
SzymonPobiega and
tmasternak
and removed request for
SzymonPobiega and
tmasternak
2 participants
Alternatives:
Action<TransportOperation, ServiceBusMessage>that any customizations on the native message type and use this from the ******** connector.