RFR: 8366211: Block signature scheme names to be used with CertificateSignature algorithm constraints usage

Mikhail Yankelevich myankelevich at openjdk.org
Thu Aug 28 14:53:42 UTC 2025


On Thu, 28 Aug 2025 14:39:17 GMT, Artur Barashev <abarashev at openjdk.org> wrote:

>> test/jdk/sun/security/ssl/SignatureScheme/BlockSignatureSchemesForCert.java line 46:
>> 
>>> 44:                 "rsa_pss_pss_sha256 usage CertificateSignature");
>>> 45: 
>>> 46:         runAndCheckException(
>> 
>> Minor: could you please make it `Utils.runAndCheckException` if there are going to be changes? Not worth another commit on it's own, but would improve the readability imo
>
> I think it's a matter of personal preference, I usually use just `runAndCheckException` in tests. In the production code I prepend the class name to static methods. I wonder if there are any coding style recommendations about it?

I agree, this is a preference thing. A few style guides: 
* Google style guide https://google.github.io/styleguide/javaguide.html#s6.3-static-members says to call with the class name.
*  But on the other hand Oracle one says it's ok (10.2) https://www.oracle.com/java/technologies/javase/codeconventions-programmingpractices.html

Overall, I asked for it, as I think almost all instances of usage of this method in the tests are with `Utils` and it's a bit hard to see where this method is coming from. But I'm ok if you want to keep it as is

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/26970#discussion_r2307667092


More information about the security-dev mailing list