[CAMEL-16227] Netty with reuseChannel=true invokes wrong callback

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.7.2
• Fix Version/s: 3.7.3, 3.9.0
• Component/s: camel-netty
• Estimated Complexity: Unknown
• Assignee: Unassigned
• Reporter: Krzysztof Jamróz
• Created: 17/Feb/21 10:54
• Updated: 18/Feb/21 16:08
• Resolved: 18/Feb/21 16:08

Description

Netty when configured with reuseChannel=true invokes wrong AsyncCallback. It invokes callback registered by first netty invocation in given route multiple times. This can be observed by debugging NettyReuseChannelTest. This seems not to cause serious problems at least in simple cases, because both callbacks (from SendProcessor) do not do much and simply continue routing. I am not sure if in more complex cases this could cause more problems.

However there is one observable issue caused by this behavior: event notifications contain wrong endpoint and wrong time taken. I attached a test for it.

Another problem is that this behavior makes it harder (if possible at all) to fix CAMEL-16178 for case with reuseChannel=true.

Issue Links

• blocks: CAMEL-16178 Enrich with REST+netty hangs when connection is closed without response
• links to: GitHub Pull Request #5101