-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Jakarta EE 10 and Java 17 #154
Conversation
This allows for breaking changes like Jakarta EE 9 updates etc.
The multipart feature never worked for real in the first place. The used library to support RFC2387 (multipart/related), Apache Commons Fileupload, was written to support RFC1867 (multipart/form-data). Thus, the support was broken and Dataverse never enabled it. This commit removes the commons-fileupload dependency and replaces the parsing of the Content-Disposition header for the filename with the Jersey header parser. This is necessary as the library is not compatible with Jakarta EE (v2.0 with Jakarta support was never released). See-also: https://www.mail-archive.com/sword-app-tech@lists.sourceforge.net/msg00327.html See-also: https://github.com/IQSS/dataverse/blob/c5d1df209aa960651a53cc67f456449a6f7158e0/src/main/java/edu/harvard/iq/dataverse/api/datadeposit/CollectionDepositManagerImpl.java#L171-L177
… to Maven central
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After removing the Jakarta branch from the Github Action this PR is good to go!
Thanks @pdurbin !
Co-authored-by: Oliver Bertuch <poikilotherm@users.noreply.github.com>
SonarCloud Quality Gate failed. 0 Bugs 0.0% Coverage The version of Java (11.0.20) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17. Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
This is an upgrade to Jakarta EE 10 (including the javax -> jakarta namespace change) and Java 17.
Related Dataverse issue: