[GitHub] [accumulo] ctubbsii commented on issue #1522: Make ClientOpts use a password converter that respects the env:PASSWORD

[GitBox <gi...@apache.org>]

ctubbsii commented on issue #1522: Make ClientOpts use a password converter that respects the env:PASSWORD
URL: https://github.com/apache/accumulo/pull/1522#issuecomment-588473063
 
 
   Now that I know what class you're using, I feel I must offer a disclaimer. That class has largely remained untouched since the initial public release of Accumulo. It was always intended to be a "use-at-your-own-risk" administrator surgery utility, but since it has largely received on attention across versions which separated the root table, switched to absolute paths, added multiple-volume support, and numerous metadata schema changes, I'd be surprised if it worked reliably and suggest it be used with *extreme* care. It even predates the use of RFiles, still referring to them as "Map Files".
   
   As for its use of the password, as a system utility, it could probably be rewritten to use the system password / system context.
   
   For 2.x, there's no need to add command-line features for passwords, because newer utilities should use either the system user (via the system properties file) or behave as a client (using client properties file, which already has commons-configuration2 features to interface with system environment and property injection).

----------------------------------------------------------------
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


With regards,
Apache Git Services