On Mon, 27 Oct 2025 19:38:46 GMT, Sean Mullan <[email protected]> wrote:
>> Hi @AlanBateman, have you had time to review my previous message? > > @franferrax Have you tried Alan's suggestion? > > Also, since this is a regression, and you think it will take longer to come > up with a fix, can I suggest as a temporary fix you revert to using > `FileInputStream` to load the security properties file (in the default > location or via the -Djava.security.properties system property), and only > call `loadFromPath` if an include directive is specified? Is that feasible? > If so, you can then address this issue separately. This should at least fix > the common case where no include directive is specified. It would be good to > get the primary use case fixed in 26. Hi @seanjmullan, thanks for your review, I replied to the three comments. I will start by removing `resolve()` when a cycle is detected, which will require test updates. For the remaining two, I'll wait for your responses before making any change. ------------- PR Comment: https://git.openjdk.org/jdk/pull/24465#issuecomment-3523525678
