davsclaus commented on code in PR #27520:
URL: https://github.com/apache/camel/pull/27520#discussion_r4213200927
##########
core/camel-base/src/main/java/org/apache/camel/impl/event/ExchangeFailureHandledEvent.java:
##########
@@ -70,6 +70,11 @@ public final String toString() {
String uri = URISupport.sanitizeUri(deadLetterUri);
return getExchange().getExchangeId() + " exchange failed"
+ " and sent to dead letter channel: " + uri;
+ } else if (failureHandler == null || failureHandler.toString() == null
+ || "null".equals(failureHandler.toString())) {
Review Comment:
The null check is the point here: Pipeline.toString() returns its id, which
is null for the steps of an onException without an id. That is the case the
events printed as "processor: null", and String.valueOf(failureHandler) returns
that null too (tried it first: the new ExchangeFailureHandledEventTextTest
failed with it). Keeping the check.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
