On Fri, 17 Oct 2025 03:02:26 GMT, Anthony Scarpino <[email protected]> wrote:
>> What I mean is that it is very rare for an API to be specified to throw a >> `RuntimeException`. It is typically a subclass. Also, I know the PEM API is >> more geared towards throwing unchecked exceptions, but this is an existing >> lower-level class that already throws checked exceptions when things go >> wrong, so it does feel to me a bit inconsistent to be throwing runtime >> exceptions when crypto operations fail in the new methods. >> >> Consider having another look at the way errors are treated in the next API >> update. > > If I remember correctly, in the preview 1 code review this was > `GeneralSecurityException` and there was a comment to change it. I'm happy > to go back to GSE. Sorry, `SecurityException` was the plan. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/27147#discussion_r2453812969
