Adam Retter
Adam Retter
**NOTE** this requires Jetty 11, i.e. https://github.com/eXist-db/exist/pull/4575
> Please consider choosing a different base branch for others to be able to see the actual change-set. @line-o As this requires Java 17, and therefore requires that PR (and...
> Yes. As I wrote, you change this PRs base branch. The base branch of this PR is the Java17 one. @line-o That is not possible as the base branch...
@lopsotronic The XML files that you provided make use of the Dublin Core namespace. Could this be the thing that is generating the traffic? Can you please use Wireshark (or...
I note that eXist-db 6 has a new Catalog Resolver which may be helpful
@dizzzz Apologies I only just saw your PR for updating to Java 11 now. I undertook this work as I need access to the new SAX APIs that are only...
> Why are those casts from ByteBuffer to Buffer as ByteBuffer already extends from it? I'm a bit confused here. This would reduce the number of changed files significantly though:...
> This needs addressing https://sonarcloud.io/project/issues?resolved=false&types=BUG&pullRequest=4574&id=eXist-db_exist&open=AYN_QOnx66Y7jh0OfK-r > The if condition needs to change to guard against an NPE https://sonarcloud.io/code?id=eXist-db_exist&pullRequest=4574&selected=eXist-db_exist%3Aexist-core%2Fsrc%2Fmain%2Fjava%2Forg%2Fexist%2Fsource%2FSourceFactory.java&line=82 > Or contextPath must be nullable here as well https://sonarcloud.io/code?id=eXist-db_exist&pullRequest=4574&selected=eXist-db_exist%3Aexist-core%2Fsrc%2Fmain%2Fjava%2Forg%2Fexist%2Fsource%2FSourceFactory.java&line=154 @line-o I...
> As for the ByteBuffer shenanigans I found this after a quick search: https://www.morling.dev/blog/bytebuffer-and-the-dreaded-nosuchmethoderror/ > Could the preferred method described in the article be also a solution for us? @line-o...
> Do we have -source 8 -target 8 still somewhere which could be replaced by --release 8. Or can we drop java8 support completely. Is ajc keeping us from doing...