You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2021/05/04 21:38:22 UTC

[GitHub] [accumulo] ctubbsii commented on a change in pull request #2064: Add option to disable encryption to AESCryptoService

ctubbsii commented on a change in pull request #2064:
URL: https://github.com/apache/accumulo/pull/2064#discussion_r626009470



##########
File path: core/src/main/java/org/apache/accumulo/core/spi/crypto/AESCryptoService.java
##########
@@ -63,6 +63,12 @@
  * Example implementation of AES encryption for Accumulo
  */
 public class AESCryptoService implements CryptoService {
+  // properties required for using this service
+  public static final String CRYPTO_PREFIX = "instance.crypto.opts.";
+  public static final String KEY_URI = CRYPTO_PREFIX + "key.uri";
+  // optional properties
+  // defaults to true
+  public static final String ENCRYPT_ENABLED = CRYPTO_PREFIX + "enabled";

Review comment:
       Should these be private?




-- 
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:
users@infra.apache.org