In payload enricher: expose setRequiresReply in Schema and default it to true [INT-2258]

[spring-operator]

Gunnar Hillert opened INT-2258 and commented

We need to expose setRequiresReply(boolean requiresReply) in the XSD Schema for the Payload Enricher and I think we should also default it to "true". It is interesting that it defaults to "false" as this this may cause by default "silently dropped messages", potentially a serious issues.

We should never drop messages silently but rather make this an explicit user choice. Therefore, default requiresReply to "true".

---

Affects: 2.1 M3

Issue Links:

• Payload Enricher - Handle "Null" Messages due to reply-timeout [INT-2257] #6240 Payload Enricher - Handle "Null" Messages due to reply-timeout
  ("depends on")

[spring-operator]

Gunnar Hillert commented

Added the attribute to the schema. Defaulting the attribute to "true" is enforced through the schema only. In the ContentEnricher's underlying AbstractReplyProducingMessageHandler, the property 'requiresReply' still defaults to false.

I wonder if we should consider defaulting it to 'true' globally.

[spring-operator]

Gunnar Hillert commented

Issued pull request: #204