You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2021/02/24 08:37:09 UTC
[GitHub] [accumulo] ctubbsii opened a new pull request #1947: Modernize EE dependencies
ctubbsii opened a new pull request #1947:
URL: https://github.com/apache/accumulo/pull/1947
Update Java EE dependencies to Jakarta EE 9
* Update dependencies in pom.xml
* Update import statements (`javax.*` to `jakarta.*`)
* Update jetty to 11 (very minor edit required in EmbeddedWebServer to
fix a variable's assigned type to match the constructed instance type)
* Update jersey to 3
* Update hibernate-validator to 7
* Use bom for dependency management for jakartaee, jetty, jersey, hk2,
jackson, and jaxb
* Remove unnecessary maven-dependency-plugin warning suppressions in
pom.xml
* Minimize bundled jars, removing any that did not appear to be required
for functionality
Additionally, fix #1898 by updating LICENSE/NOTICE files while updating
dependencies:
* Relocate ellipsis datatables plugin and jquery to subdirectories to
make it easier to reference in LICENSE and associate with distinct
upstream sources for future updates
* Remove unused select2.css
* Review all bundled external resources (js/css) for monitor and update
LICENSE and NOTICE files accordingly
* Review all bundled jars in lib directory of binary tarball and update
LICENSE and NOTICE files accordingly
* Standardize the structure of the LICENSE/NOTICE files to avoid
duplication of external license texts where possible (especially for
EPL-2.0, which is massive) and to use the Maven coordinates from
`component.xml` for easier maintenance
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] ctubbsii commented on pull request #1947: Modernize EE dependencies
Posted by GitBox <gi...@apache.org>.
ctubbsii commented on pull request #1947:
URL: https://github.com/apache/accumulo/pull/1947#issuecomment-785335866
I created https://github.com/FasterXML/jackson-bom/issues/40 to deal with the fact that I couldn't use the jackson bom for managing some of the Jakarta EE variants of jackson libraries.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] ctubbsii merged pull request #1947: Modernize EE dependencies
Posted by GitBox <gi...@apache.org>.
ctubbsii merged pull request #1947:
URL: https://github.com/apache/accumulo/pull/1947
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] ctubbsii commented on pull request #1947: Modernize EE dependencies
Posted by GitBox <gi...@apache.org>.
ctubbsii commented on pull request #1947:
URL: https://github.com/apache/accumulo/pull/1947#issuecomment-785269544
There is a bug with MonitorSslIT with this on my machine, that I'm trying to track down. It seems that there's a mismatch in the way the new SSL handling code on the server identifies the certificate to present and the way our http client code in the test initiates the request, that causes the client to pass SNI information (which the client sets to null) that prevents the test server from satisfying the request. I'm still tracking this down.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] ctubbsii edited a comment on pull request #1947: Modernize EE dependencies
Posted by GitBox <gi...@apache.org>.
ctubbsii edited a comment on pull request #1947:
URL: https://github.com/apache/accumulo/pull/1947#issuecomment-785269544
There is a bug with MonitorSslIT with this on my machine, that I'm trying to track down. It seems that there's a mismatch in the way the new SSL handling code on the server identifies the certificate to present and the way our http client code in the test initiates the request, that causes the client to pass SNI information (which the client sets to null) that prevents the test server from satisfying the request. I'm still tracking this down.
EDIT: fixed in newer commits
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org