exist icon indicating copy to clipboard operation
exist copied to clipboard

Upgrade to Jetty 11

Open adamretter opened this issue 3 years ago • 3 comments

Requires Java 17 support to be merged first - https://github.com/eXist-db/exist/pull/4574 (and so at this time it contains all commits of that PR also).

  • This required me to also release a new version of Apache XML-RPC which supports Jakarta 5 - https://search.maven.org/artifact/com.evolvedbinary.thirdparty.org.apache.xmlrpc/xmlrpc/5.0.0/pom
  • This also required me to release an updated version of Milton WebDav which supports Jakarta 5 - https://search.maven.org/artifact/org.exist-db.thirdparty.com.ettrema/milton/1.8.1.3-jakarta5/pom

adamretter avatar Sep 26 '22 23:09 adamretter

@reinhapa A lot of your comments are duplicated from https://github.com/eXist-db/exist/pull/4574 as this PR is stacked atop that one as it is required.

adamretter avatar Sep 27 '22 13:09 adamretter

image

dizzzz avatar Sep 30 '22 15:09 dizzzz

@dizzzz Where are you seeing that error?

adamretter avatar Oct 01 '22 07:10 adamretter

@reinhapa If you have a moment... I cannot understand why I am getting the javadoc error in the Javadoc CI run, as theproject has no dependency on javax.servlet any more after switching to Jakarta. Any ideas?

adamretter avatar Oct 01 '22 07:10 adamretter

@dizzzz Where are you seeing that error?

https://github.com/eXist-db/exist/actions/runs/3154141844/jobs/5131407589

At least the Ubuntu build, error last lines

dizzzz avatar Oct 01 '22 20:10 dizzzz

https://github.com/eXist-db/exist/actions/runs/3154141844/jobs/5131407589 At least the Ubuntu build, error last lines

@dizzzz I think that must be an old run, the current one is passing just fine.

adamretter avatar Oct 05 '22 17:10 adamretter

As we learned from #4586 uploading smaller payloads encoded as multipart/form-data is not tested. This test should be added as part of this change to ensure this is still working.

line-o avatar Oct 13 '22 13:10 line-o