jt2594838 commented on code in PR #447: URL: https://github.com/apache/tsfile/pull/447#discussion_r2009349975
########## java/tsfile/src/main/java/org/apache/tsfile/encrypt/EncryptUtils.java: ########## @@ -106,78 +122,86 @@ public static String byteArrayToHexString(byte[] bytes) { return sb.toString(); } - public static String getNormalKeyStr() { + public static String computeNormalKeyStr() { + final MessageDigest md; try { - MessageDigest md = MessageDigest.getInstance("SHA-256"); - md.update("IoTDB is the best".getBytes()); - md.update(TSFileDescriptor.getInstance().getConfig().getEncryptKey().getBytes()); - byte[] data_key = Arrays.copyOfRange(md.digest(), 0, 16); - data_key = - IEncryptor.getEncryptor( - TSFileDescriptor.getInstance().getConfig().getEncryptType(), - TSFileDescriptor.getInstance().getConfig().getEncryptKey().getBytes()) - .encrypt(data_key); - - StringBuilder valueStr = new StringBuilder(); - - for (byte b : data_key) { - valueStr.append(b).append(","); - } - - valueStr.deleteCharAt(valueStr.length() - 1); - String str = valueStr.toString(); - - return str; - } catch (Exception e) { + md = MessageDigest.getInstance("SHA-256"); + } catch (NoSuchAlgorithmException e) { throw new EncryptException( - "SHA-256 function not found while using SHA-256 to generate data key", e); + "SHA-256 algorithm not found while using SHA-256 to generate data key", e); } + md.update("IoTDB is the best".getBytes()); + md.update(TSFileDescriptor.getInstance().getConfig().getEncryptKey().getBytes()); + byte[] data_key = Arrays.copyOfRange(md.digest(), 0, 16); + data_key = + IEncryptor.getEncryptor( + TSFileDescriptor.getInstance().getConfig().getEncryptType(), + TSFileDescriptor.getInstance().getConfig().getEncryptKey().getBytes()) + .encrypt(data_key); + + StringBuilder valueStr = new StringBuilder(); + + for (byte b : data_key) { + valueStr.append(b).append(","); + } + + valueStr.deleteCharAt(valueStr.length() - 1); + return valueStr.toString(); } Review Comment: What is the difference between this method and `getNormalKeyStr(TSFileDescriptor.getInstance().getConfig())`? -- 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. To unsubscribe, e-mail: notifications-unsubscr...@tsfile.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org