On Fri, 24 Sep 2021 21:59:04 GMT, Valerie Peng <valer...@openjdk.org> wrote:
>> This code change removes weak etypes from the default list so it's safer to >> enable one of them. See the corresponding CSR at >> https://bugs.openjdk.java.net/browse/JDK-8274207 for more explanation. BTW, >> please review the CSR as well. > > src/java.security.jgss/share/classes/sun/security/krb5/internal/crypto/EType.java > line 90: > >> 88: try { >> 89: Config cfg = Config.getInstance(); >> 90: allowWeakCrypto = cfg.getBooleanObject("libdefaults", >> "allow_weak_crypto") > > nit: exceeds 80 chars? A little. I've learnt to accept lines a little bit longer than 80. ------------- PR: https://git.openjdk.java.net/jdk/pull/5654