Re: [I] Camel Quarkus 3.19.0/Quarkus 3.19.1 ignores empty body in .when() statement [camel-quarkus]
jamesnetherton commented on issue #7085: URL: https://github.com/apache/camel-quarkus/issues/7085#issuecomment-2872001991 Fix is available from CQ 3.20.0 & 3.22.0. -- 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: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] Camel Quarkus 3.19.0/Quarkus 3.19.1 ignores empty body in .when() statement [camel-quarkus]
jamesnetherton closed issue #7085: Camel Quarkus 3.19.0/Quarkus 3.19.1 ignores empty body in .when() statement URL: https://github.com/apache/camel-quarkus/issues/7085 -- 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: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] Camel Quarkus 3.19.0/Quarkus 3.19.1 ignores empty body in .when() statement [camel-quarkus]
davsclaus commented on issue #7085: URL: https://github.com/apache/camel-quarkus/issues/7085#issuecomment-2700621250 Okay there is a bug that the otherwise block is calling when no outputs. https://issues.apache.org/jira/browse/CAMEL-21832 -- 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: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] Camel Quarkus 3.19.0/Quarkus 3.19.1 ignores empty body in .when() statement [camel-quarkus]
janwesterkamp commented on issue #7085: URL: https://github.com/apache/camel-quarkus/issues/7085#issuecomment-2697507341 Why not? Do I have to add senseless statements only to fulfil your requirement, @davsclaus ? It was possible to do in 3.18.4, so 3.19.1 is introducing a breaking change here. Here my use case: I am checking a positive list of values with these statements and the .otherwhise() statement will catch all other values and breaks further processing (failing fast). I started this originally because the REST DSL had some limitations in input validation (might become better now), but now it's used for more complex checks too.. The validated values are further processed later in other routes (where the .when() statements contain actions). In my eyes, this is an valid use case and empty when() should be allowed in newer versions. -- 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: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] Camel Quarkus 3.19.0/Quarkus 3.19.1 ignores empty body in .when() statement [camel-quarkus]
davsclaus commented on issue #7085: URL: https://github.com/apache/camel-quarkus/issues/7085#issuecomment-2697449922 you cannot have empty when's so old code was wrong -- 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: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] Camel Quarkus 3.19.0/Quarkus 3.19.1 ignores empty body in .when() statement [camel-quarkus]
jamesnetherton commented on issue #7085: URL: https://github.com/apache/camel-quarkus/issues/7085#issuecomment-2697441177 I guess something changed between Camel 4.9.x and 4.10.x. @davsclaus WDYT? -- 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: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[I] Camel Quarkus 3.19.0/Quarkus 3.19.1 ignores empty body in .when() statement [camel-quarkus]
janwesterkamp opened a new issue, #7085:
URL: https://github.com/apache/camel-quarkus/issues/7085
### Bug description
I am using the choice/when statements to do input validation and sometimes
there are no statements inside.
After switching Quarkus version 3.18.4 to 3.19.1 (which includes Camel
Quarkus 3.19.0), these empty checks are ignored, which causes a breaking change.
I found a workaround for this by adding something like a .log() statement -
then it will be validated again.
However, I thing this is a bug that needs to be fixed.
Here is some example route to show the issue:
```
//FIXME
from("direct:test")
//TODO Set this to "2" for the workaround
.setHeader("test", constant("1"))
.choice()
.when(header("test").isEqualTo("1"))
//Empty - fails on Quarkus version 3.19.1
.when(header("test").isEqualTo("2"))
//FIXME Workaround for Quarkus version 3.19.1 - empty when body
causes issue!
.log(LoggingLevel.INFO, "Processing test=${header.test}")
.otherwise()
.throwException(new RuntimeException("Vaildation error!"))
.end()
;
```
May be the latest changes regarding .when() statements causing this as an
side effect?
--
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: commits-unsubscr...@camel.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
