DomGarguilo commented on code in PR #59:
URL: https://github.com/apache/accumulo-proxy/pull/59#discussion_r1055824518
##########
src/main/java/org/apache/accumulo/proxy/ProxyServer.java:
##########
@@ -129,8 +125,10 @@ public class ProxyServer implements AccumuloProxy.Iface {
public static final Logger logger =
LoggerFactory.getLogger(ProxyServer.class);
public static final String RPC_ACCUMULO_PRINCIPAL_MISMATCH_MSG = "RPC"
+ " principal did not match requested Accumulo principal";
- @SuppressWarnings("deprecation")
- protected org.apache.accumulo.core.client.Instance instance;
+
+ protected AccumuloClient client;
+
+ private final String secret;
Review Comment:
This is one of the main changes in this PR. The addition of this shared
secret allows for the client to authenticate to the proxy service without
telling the proxy service how to authenticate to accumulo.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
