davsclaus commented on pull request #5543: URL: https://github.com/apache/camel/pull/5543#issuecomment-839542763
Okay I did a bit of test, and we have tests from camel-core that would create an exchange from another endpoint (such as file endpoint) and then send to direct endpoint via a producer template to be routed. In that situation then your patch will override the existing from endpoint (the file endpoint) with the route endpoint (direct), which would be wrong IMHO. So you should check for null, before setting the endpoint -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org