You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "turing85 (via GitHub)" <gi...@apache.org> on 2023/05/15 20:17:21 UTC
[GitHub] [camel-quarkus] turing85 opened a new pull request, #4902: Bump xalan from version 2.7.2 to version 2.7.3
turing85 opened a new pull request, #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902
Bump xalan from version 2.7.2 to version 2.7.3
--
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
[GitHub] [camel-quarkus] zhfeng commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "zhfeng (via GitHub)" <gi...@apache.org>.
zhfeng commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1558621944
The XML test are weired. They are working in JVM mode but throwing the Exception during native building.
```
[ERROR] org/apache/camel/quarkus/component/xslt/generated/ClasspathTransform (wrong name: org/apache/xalan/xsltc/runtime/AbstractTranslet)
javax.xml.transform.TransformerException: org/apache/camel/quarkus/component/xslt/generated/ClasspathTransform (wrong name: org/apache/xalan/xsltc/runtime/AbstractTranslet)
at org.apache.xalan.xsltc.trax.TransformerFactoryImpl.passErrorsToListener (TransformerFactoryImpl.java:661)
at org.apache.xalan.xsltc.trax.TransformerFactoryImpl.newTemplates (TransformerFactoryImpl.java:836)
at org.apache.camel.quarkus.support.xalan.XalanTransformerFactory.newTemplates (XalanTransformerFactory.java:70)
at org.apache.camel.quarkus.component.xslt.deployment.XsltProcessor.xsltResources (XsltProcessor.java:119)
at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0 (Native Method)
at jdk.internal.reflect.NativeMethodAccessorImpl.invoke (NativeMethodAccessorImpl.java:77)
at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke (DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke (Method.java:568)
at io.quarkus.deployment.ExtensionLoader$3.execute (ExtensionLoader.java:909)
at io.quarkus.builder.BuildContext.run (BuildContext.java:281)
at org.jboss.threads.ContextHandler$1.runWith (ContextHandler.java:18)
at org.jboss.threads.EnhancedQueueExecutor$Task.run (EnhancedQueueExecutor.java:2449)
at org.jboss.threads.EnhancedQueueExecutor$ThreadBody.run (EnhancedQueueExecutor.java:1478)
at java.lang.Thread.run (Thread.java:833)
at org.jboss.threads.JBossThread.run (JBossThread.java:501)
[ERROR] Could not compile stylesheet
javax.xml.transform.TransformerConfigurationException: Could not compile stylesheet
at org.apache.xalan.xsltc.trax.TransformerFactoryImpl.newTemplates (TransformerFactoryImpl.java:832)
at org.apache.camel.quarkus.support.xalan.XalanTransformerFactory.newTemplates (XalanTransformerFactory.java:70)
at org.apache.camel.quarkus.component.xslt.deployment.XsltProcessor.xsltResources (XsltProcessor.java:119)
at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0 (Native Method)
at jdk.internal.reflect.NativeMethodAccessorImpl.invoke (NativeMethodAccessorImpl.java:77)
at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke (DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke (Method.java:568)
at io.quarkus.deployment.ExtensionLoader$3.execute (ExtensionLoader.java:909)
at io.quarkus.builder.BuildContext.run (BuildContext.java:281)
at org.jboss.threads.ContextHandler$1.runWith (ContextHandler.java:18)
at org.jboss.threads.EnhancedQueueExecutor$Task.run (EnhancedQueueExecutor.java:2449)
at org.jboss.threads.EnhancedQueueExecutor$ThreadBody.run (EnhancedQueueExecutor.java:1478)
at java.lang.Thread.run (Thread.java:833)
at org.jboss.threads.JBossThread.run (JBossThread.java:501)
```
I will take a look.
--
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
[GitHub] [camel-quarkus] turing85 commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "turing85 (via GitHub)" <gi...@apache.org>.
turing85 commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1573338165
Sooo... what's our course of action now? Xalan 2.7.2 is vulnerable to https://cve.circl.lu/cve/CVE-2022-34169 and 2.7.3 does not seem to work.
--
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
[GitHub] [camel-quarkus] ppalaga commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "ppalaga (via GitHub)" <gi...@apache.org>.
ppalaga commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1573420574
> I filed https://issues.apache.org/jira/browse/XALANJ-2664
Would you like to care for sending the PR, @zhfeng ?
--
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
[GitHub] [camel-quarkus] turing85 commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "turing85 (via GitHub)" <gi...@apache.org>.
turing85 commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1551157805
> > There is, however, a major concern here: [CVE-2022-34169](https://nvd.nist.gov/vuln/detail/CVE-2022-34169). For me, it is unclear whether the issue is **actually** fixed in `2.7.3`, or whether it is just "gone" due to the missing dependencies.
>
> The fix commit seems to be this one [apache/xalan-java@2e60d0a](https://github.com/apache/xalan-java/commit/2e60d0a9a5b822c4abf9051857973b1c6babfe81) That's definitely in Xalan.
Okay nice. Only question is now: what is missing so that the pipeline succeeds?
--
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
[GitHub] [camel-quarkus] zhfeng commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "zhfeng (via GitHub)" <gi...@apache.org>.
zhfeng commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1573453546
> > @ppalaga I'm just curious in what case the `TCCL` is `null`?
>
> When it is not set via `Thread.currentThread().setContextClassLoader(...)` - it is quite a normal situation.
IIRC, the default value of `TCCL` is System Class Loader. So if there is no `setContextClassLoader(null)` explicitly, it should be always not empty?
--
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
[GitHub] [camel-quarkus] turing85 commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "turing85 (via GitHub)" <gi...@apache.org>.
turing85 commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1550297864
... welp...
--
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
[GitHub] [camel-quarkus] ppalaga commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "ppalaga (via GitHub)" <gi...@apache.org>.
ppalaga commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1573418925
I filed https://issues.apache.org/jira/browse/XALANJ-2664
--
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
[GitHub] [camel-quarkus] ppalaga commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "ppalaga (via GitHub)" <gi...@apache.org>.
ppalaga commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1551986513
I was able to do some hacks to make the XML security test pass - see https://github.com/ppalaga/camel-quarkus/commits/pr4902
I unfortunately have no more time fix the failing XML test.
--
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
[GitHub] [camel-quarkus] ppalaga commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "ppalaga (via GitHub)" <gi...@apache.org>.
ppalaga commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1573362442
> something like `super(Thread.currentThread().getContextClassLoader() != null ? Thread.currentThread().getContextClassLoader() : ClassLoader.getSystemClassLoader())`
Or even better `super(Thread.currentThread().getContextClassLoader() != null ? XSLTC.class.getClassLoader() : ClassLoader.getSystemClassLoader())`
--
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
[GitHub] [camel-quarkus] ppalaga commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "ppalaga (via GitHub)" <gi...@apache.org>.
ppalaga commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1573417226
> @ppalaga I'm just curious in what case the `TCCL` is `null`?
When it is not set via `Thread.currentThread().setContextClassLoader(...)` - it is quite a normal situation.
--
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
[GitHub] [camel-quarkus] ppalaga commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "ppalaga (via GitHub)" <gi...@apache.org>.
ppalaga commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1573366856
Let's propose it in Xalan then. Should I help with formulating why we need it?
--
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
[GitHub] [camel-quarkus] jamesnetherton commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "jamesnetherton (via GitHub)" <gi...@apache.org>.
jamesnetherton commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1549061195
> Oh nice... so... should we close this PR and wait for an update instead?
It's not clear to me whether they are going to patch it or not....
I think we could work around it by adding `xalan:serializer` into `poms/bom/pom.xml` and also into `extensions-support/xalan/runtime/pom.xml`. Afterwards, if you run `mvn clean install` in `poms` it'll regenerate the BOM files.
--
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
[GitHub] [camel-quarkus] turing85 commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "turing85 (via GitHub)" <gi...@apache.org>.
turing85 commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1548618187
@oscerd could you re-approve the workflows? I had to force-push some changes in order to make the pipeline succeed.
--
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
[GitHub] [camel-quarkus] zhfeng commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "zhfeng (via GitHub)" <gi...@apache.org>.
zhfeng commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1573396198
> super(Thread.currentThread().getContextClassLoader() != null ? Thread.currentThread().getContextClassLoader() : XSLTC.class.getClassLoader())
@ppalaga I'm just curious in what case the `TCCL` is `null`?
--
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
[GitHub] [camel-quarkus] ppalaga commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "ppalaga (via GitHub)" <gi...@apache.org>.
ppalaga commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1573359336
@zhfeng could you please try changing the constructor of `ByteArrayClassLoader` so that it uses something like `super(Thread.currentThread().getContextClassLoader() != null ? Thread.currentThread().getContextClassLoader() : ClassLoader.getSystemClassLoader())`? If it works for us, we should file a Xalan issue and send a PR.
--
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
[GitHub] [camel-quarkus] ppalaga commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "ppalaga (via GitHub)" <gi...@apache.org>.
ppalaga commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1573376243
You mean I should file an issue on Xalan? - no problem, let me do it.
--
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
[GitHub] [camel-quarkus] zhfeng commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "zhfeng (via GitHub)" <gi...@apache.org>.
zhfeng commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1573054251
Well, it seems that xalan 2.7.3 added a new checking before dumping a generated translet class. See https://github.com/apache/xalan-java/blame/master/src/org/apache/xalan/xsltc/compiler/XSLTC.java#L860-L865
```java
byte[] classByteArray = clazz.getBytes();
ByteArrayClassLoader classLoader = new ByteArrayClassLoader(classByteArray);
Class clz = classLoader.findClass(clazz.getClassName());
```
And `ByteArrayClassLoader` can not find `org.apache.xalan.xsltc.runtime.AbstractTranslet` during the quarkus-maven-plugin runs `AugmentAction`. I guess that it uses a different class loader in `quarkus-maven-plugin` and `ByteArrayClassLoader` is inherited from SystemClassLoader.
I have no idea how to fix it and don't undertand the motivation for the checking in XSLTC.
--
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
[GitHub] [camel-quarkus] turing85 commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "turing85 (via GitHub)" <gi...@apache.org>.
turing85 commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1549368858
> > Oh nice... so... should we close this PR and wait for an update instead?
>
> It's not clear to me whether they are going to patch it or not....
>
> I think we could work around it by adding `xalan:serializer` into `poms/bom/pom.xml` and also into `extensions-support/xalan/runtime/pom.xml`. Afterwards, if you run `mvn clean install` in `poms` it'll regenerate the BOM files.
Okay, will update my PR as soon as I have some time. There is, however, a major concern here: [CVE-2022-34169](https://nvd.nist.gov/vuln/detail/CVE-2022-34169). For me, it is unclear whether the issue is **actually** fixed in `2.7.3`, or whether it is just "gone" due to the missing dependencies.
--
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
[GitHub] [camel-quarkus] ppalaga commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "ppalaga (via GitHub)" <gi...@apache.org>.
ppalaga commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1551181775
> what is missing so that the pipeline succeeds?
Let me have a look
--
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
[GitHub] [camel-quarkus] zhfeng commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "zhfeng (via GitHub)" <gi...@apache.org>.
zhfeng commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1573364331
@ppalaga yeah, I tried such changes and it works with `TCCL`.
--
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
[GitHub] [camel-quarkus] zhfeng commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "zhfeng (via GitHub)" <gi...@apache.org>.
zhfeng commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1573427726
Thanks @ppalaga and I will prepare a PR.
--
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
[GitHub] [camel-quarkus] ppalaga commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "ppalaga (via GitHub)" <gi...@apache.org>.
ppalaga commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1551124005
> There is, however, a major concern here: [CVE-2022-34169](https://nvd.nist.gov/vuln/detail/CVE-2022-34169). For me, it is unclear whether the issue is **actually** fixed in `2.7.3`, or whether it is just "gone" due to the missing dependencies.
The fix commit seems to be this one https://github.com/apache/xalan-java/commit/2e60d0a9a5b822c4abf9051857973b1c6babfe81
That's definitely in Xalan.
--
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
[GitHub] [camel-quarkus] jamesnetherton commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "jamesnetherton (via GitHub)" <gi...@apache.org>.
jamesnetherton commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1549050888
> It seem that there are ` java.lang.ClassNotFoundException: org.apache.xml.serializer.OutputPropertiesFactory`
The 2.7.3 POM is a bit screwed up compared with 2.7.2. It is missing dependencies.
https://issues.apache.org/jira/browse/XALANJ-2649
--
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
[GitHub] [camel-quarkus] zhfeng commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "zhfeng (via GitHub)" <gi...@apache.org>.
zhfeng commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1573368997
Doe we need to file an issue on [XALANJ](https://issues.apache.org/jira/projects/XALANJ/issues/XALANJ-2357?filter=allopenissues) ?
--
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
[GitHub] [camel-quarkus] turing85 commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "turing85 (via GitHub)" <gi...@apache.org>.
turing85 commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1549053240
Oh nice... so... should we close this PR and wait for an update instead?
--
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
[GitHub] [camel-quarkus] zhfeng commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "zhfeng (via GitHub)" <gi...@apache.org>.
zhfeng commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1549046298
It seem that there are ` java.lang.ClassNotFoundException: org.apache.xml.serializer.OutputPropertiesFactory`
--
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
[GitHub] [camel-quarkus] turing85 commented on pull request #4902: Bump xalan from version 2.7.2 to version 2.7.3
Posted by "turing85 (via GitHub)" <gi...@apache.org>.
turing85 commented on PR #4902:
URL: https://github.com/apache/camel-quarkus/pull/4902#issuecomment-1550073087
@jamesnetherton @oscerd @zhfeng Updated the PR, `xalan:serializer` is now in the dependencies. Could you re-approve to trigger the pipeline run?
--
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