janhoy commented on PR #3122:
URL: https://github.com/apache/solr/pull/3122#issuecomment-4849208552
Pushed a commit to complete this major upgrade. s3mock 4.x swapped its
embedded server from Jetty to Tomcat, which caused two failures:
1. **Licenses** — new transitive
`tomcat-embed-core`/`tomcat-embed-websocket` jars had no LICENSE/NOTICE.
Renamed the existing `tomcat-embed-el-LICENSE/NOTICE` to the shared `tomcat-`
prefix so it covers all `tomcat-embed-*` artifacts (all Apache-2.0).
2. **All tests failing (HTTP 500)** — Tomcat 10.1 is incompatible with
Solr's test SecurityManager (`PrivilegedSetAccessControlContext` throws under
an SM). Disabled the SecurityManager for this test-only module via `ext {
useSecurityManager = false }`, matching what `extraction` and `gcs-repository`
already do.
Verified locally: `:solr:modules:s3-repository:check` and the full test
suite pass.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
