You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "apupier (via GitHub)" <gi...@apache.org> on 2025/04/03 13:28:17 UTC
[PR] Upgrade smooks example to 4.11.0 [camel-jbang-examples]
apupier opened a new pull request, #31:
URL: https://github.com/apache/camel-jbang-examples/pull/31
(no comment)
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Upgrade smooks example to 4.11.0 [camel-jbang-examples]
Posted by "apupier (via GitHub)" <gi...@apache.org>.
apupier commented on code in PR #31:
URL: https://github.com/apache/camel-jbang-examples/pull/31#discussion_r2027041326
##########
edi-x12-as2/camel/App.java:
##########
@@ -1,4 +1,4 @@
//DEPS org.apache.activemq:artemis-jakarta-server:2.40.0
-//DEPS org.apache.camel:camel-http:4.10.0
-//DEPS org.smooks.cartridges.edi:smooks-edi-cartridge:2.1.0
-//DEPS org.smooks.cartridges:smooks-templating-cartridge:2.1.2
\ No newline at end of file
+//DEPS org.apache.camel:camel-http:4.11.0
+//DEPS org.smooks.cartridges.edi:smooks-edi-cartridge:2.2.1
+//DEPS org.smooks.cartridges:smooks-templating-cartridge:2.2.1
Review Comment:
https://github.com/apache/camel/blob/b4f0c8d00d2dd7f5a9397d8422d3b6e52998505d/parent/pom.xml#L462
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Upgrade smooks example to 4.11.0 [camel-jbang-examples]
Posted by "cjmamo (via GitHub)" <gi...@apache.org>.
cjmamo commented on PR #31:
URL: https://github.com/apache/camel-jbang-examples/pull/31#issuecomment-2808690920
Debugged it a little. The [ediMessageContentType](https://github.com/apupier/camel-jbang-examples/blob/bd3c1a852f50ab1f8b80eea91d85e6fd75f4f715/edi-x12-as2/test/edi-x12-as2.camel.it.yaml#L34) parameter in the test request is causing the following error with the latest version of Camel:
>Caused by: java.lang.IllegalArgumentException: MIME type may not contain reserved characters
If I drop the `charset` nested param from `ediMessageContentType` such that `application%2Fedi-x12%3Bcharset%3DUS-ASCII` becomes `application%2Fedi-x12`, then the test passes. What's weird is that this error doesn't crop up from [edi-x12-as2.camel.yaml](https://github.com/apupier/camel-jbang-examples/blob/c29bfef880dc3b65f3b4400779aa6b6b86afe0a7/edi-x12-as2/camel/edi-x12-as2.camel.yaml#L60) despite `ediMessageContentType` containing the `charset` param
Could it be a regression in Camel or Citrus?
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Upgrade smooks example to 4.11.0 [camel-jbang-examples]
Posted by "apupier (via GitHub)" <gi...@apache.org>.
apupier commented on PR #31:
URL: https://github.com/apache/camel-jbang-examples/pull/31#issuecomment-2812100648
woohoo tests passing.
Thanks for the help!
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Upgrade smooks example to 4.11.0 [camel-jbang-examples]
Posted by "cjmamo (via GitHub)" <gi...@apache.org>.
cjmamo commented on PR #31:
URL: https://github.com/apache/camel-jbang-examples/pull/31#issuecomment-2809279186
```
Failed to create EDI message entity
at org.apache.camel.support.component.ApiMethodHelper.invokeMethod(ApiMethodHelper.java:529) ~[camel-support-4.11.0.jar:4.11.0]
at org.apache.camel.support.component.AbstractApiProducer.doInvokeMethod(AbstractApiProducer.java:122) ~[camel-support-4.11.0.jar:4.11.0]
at org.apache.camel.support.component.AbstractApiProducer$1.run(AbstractApiProducer.java:89) ~[camel-support-4.11.0.jar:4.11.0]
at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:539) ~[?:?]
at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264) ~[?:?]
at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:304) ~[?:?]
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) ~[?:?]
at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) ~[?:?]
at java.base/java.lang.Thread.run(Thread.java:840) ~[?:?]
Caused by: org.apache.hc.core5.http.HttpException: Failed to create EDI message entity
at org.apache.camel.component.as2.api.AS2ClientManager.send(AS2ClientManager.java:284) ~[camel-as2-api-4.11.0.jar:4.11.0]
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:?]
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) ~[?:?]
at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:?]
at java.base/java.lang.reflect.Method.invoke(Method.java:569) ~[?:?]
at org.apache.camel.support.component.ApiMethodHelper.invokeMethod(ApiMethodHelper.java:521) ~[camel-support-4.11.0.jar:4.11.0]
at org.apache.camel.support.component.AbstractApiProducer.doInvokeMethod(AbstractApiProducer.java:122) ~[camel-support-4.11.0.jar:4.11.0]
at org.apache.camel.support.component.AbstractApiProducer$1.run(AbstractApiProducer.java:89) ~[camel-support-4.11.0.jar:4.11.0]
at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:539) ~[?:?]
at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264) ~[?:?]
at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:304) ~[?:?]
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) ~[?:?]
at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) ~[?:?]
at java.base/java.lang.Thread.run(Thread.java:840) ~[?:?]
Caused by: java.lang.IllegalArgumentException: MIME type may not contain reserved characters
at org.apache.hc.core5.util.Args.check(Args.java:41) ~[httpcore5-5.3.1.jar:5.3.1]
at org.apache.hc.core5.http.ContentType.create(ContentType.java:319) ~[httpcore5-5.3.1.jar:5.3.1]
at org.apache.hc.core5.http.ContentType.create(ContentType.java:347) ~[httpcore5-5.3.1.jar:5.3.1]
at org.apache.camel.component.as2.api.AS2ClientManager.send(AS2ClientManager.java:269) ~[camel-as2-api-4.11.0.jar:4.11.0]
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:?]
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) ~[?:?]
at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:?]
at java.base/java.lang.reflect.Method.invoke(Method.java:569) ~[?:?]
at org.apache.camel.support.component.ApiMethodHelper.invokeMethod(ApiMethodHelper.java:521) ~[camel-support-4.11.0.jar:4.11.0]
at org.apache.camel.support.component.AbstractApiProducer.doInvokeMethod(AbstractApiProducer.java:122) ~[camel-support-4.11.0.jar:4.11.0]
at org.apache.camel.support.component.AbstractApiProducer$1.run(AbstractApiProducer.java:89) ~[camel-support-4.11.0.jar:4.11.0]
at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:539) ~[?:?]
at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264) ~[?:?]
at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:304) ~[?:?]
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) ~[?:?]
at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) ~[?:?]
at java.base/java.lang.Thread.run(Thread.java:840) ~[?:?]
```
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Upgrade smooks example to 4.11.0 [camel-jbang-examples]
Posted by "apupier (via GitHub)" <gi...@apache.org>.
apupier commented on PR #31:
URL: https://github.com/apache/camel-jbang-examples/pull/31#issuecomment-2775846774
some stuff will need to be adapted to upgrade the Camel version:
```
Caused by: org.xml.sax.SAXParseException; cvc-complex-type.2.4.a: Invalid content was found starting with element '{"https://www.smooks.org/xsd/smooks/freemarker-2.1.xsd":freemarker}'. One of '{"https://www.smooks.org/xsd/smooks-2.0.xsd":abstract-resource-config}' is expected.
```
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Upgrade smooks example to 4.11.0 [camel-jbang-examples]
Posted by "cjmamo (via GitHub)" <gi...@apache.org>.
cjmamo commented on PR #31:
URL: https://github.com/apache/camel-jbang-examples/pull/31#issuecomment-2812077781
@apupier the changes pushed are referencing the latest changes in the JavaBean cartridge (e.g., `javabean-2.0.xsd`) which has not yet been released.
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Upgrade smooks example to 4.11.0 [camel-jbang-examples]
Posted by "davsclaus (via GitHub)" <gi...@apache.org>.
davsclaus commented on PR #31:
URL: https://github.com/apache/camel-jbang-examples/pull/31#issuecomment-2809191736
Do you have the full stacktrace of that error
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Upgrade smooks example to 4.11.0 [camel-jbang-examples]
Posted by "davsclaus (via GitHub)" <gi...@apache.org>.
davsclaus commented on PR #31:
URL: https://github.com/apache/camel-jbang-examples/pull/31#issuecomment-2799867761
@cjmamo maybe you can help a bit to update this example
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Upgrade smooks example to 4.11.0 [camel-jbang-examples]
Posted by "apupier (via GitHub)" <gi...@apache.org>.
apupier merged PR #31:
URL: https://github.com/apache/camel-jbang-examples/pull/31
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Upgrade smooks example to 4.11.0 [camel-jbang-examples]
Posted by "apupier (via GitHub)" <gi...@apache.org>.
apupier commented on PR #31:
URL: https://github.com/apache/camel-jbang-examples/pull/31#issuecomment-2811959508
Seems related to an API break https://github.com/apache/camel/commit/5d964ff96680d58985250701352de1a384ab6806 https://camel.apache.org/manual/camel-4x-upgrade-guide-4_11.html#_camel_as2
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org