ChrisHegarty commented on PR #13146: URL: https://github.com/apache/lucene/pull/13146#issuecomment-1977398961
> I think I could try to locate places in RandomizedRunner where it calls into JUnit without doPrivileged... not sure if it's worth the hassle though - maybe adding permissions for just those three jars is fine (we can compute the URLs and pass them as properties)? Putting aside the test specific usages mentioned above, then... so long as there is no test code on the stack when setAccessible is called, those three jars should be sufficient. At least from what I see on my machine. I do see the test count working fine with just those three jars (as well as the global grants removed). Here's an example from `./gradlew :lucene:core:test` <img width="797" alt="Screenshot 2024-03-04 at 20 14 32" src="https://github.com/apache/lucene/assets/62058229/c495e34a-2da9-4444-bd23-0d85c1da9315";> -- 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: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
