MikeThomsen commented on a change in pull request #3547: [WIP] NIFI-5254 Update
Groovy dependecies to version 2.5.4
URL: https://github.com/apache/nifi/pull/3547#discussion_r297102184
##########
File path:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-properties-loader/src/test/groovy/org/apache/nifi/properties/AESSensitivePropertyProviderTest.groovy
##########
@@ -86,7 +86,7 @@ class AESSensitivePropertyProviderTest extends
GroovyTestCase {
if (Cipher.getMaxAllowedKeyLength("AES") < keySize) {
throw new IllegalArgumentException("The JCE unlimited
strength cryptographic jurisdiction policies are not installed, so the max key
size is 128 bits")
}
- return KEY_256_HEX[0..<(keySize / 4)]
+ return KEY_256_HEX[0..<keySize.intdiv(4)]
Review comment:
Curious what the difference is here. When I changed to 2.5.7 I remember the
original line returning 49 instead of 48.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
