Flaugh24 commented on code in PR #2688:
URL: https://github.com/apache/ignite-3/pull/2688#discussion_r1366701434


##########
modules/client-handler/src/main/java/org/apache/ignite/client/handler/ClientContext.java:
##########
@@ -79,6 +94,20 @@ public BitSet features() {
         return features;
     }
 
+    /**
+     * Gets the authentication request.
+     *
+     * @return Authentication request.
+     */
+    public AuthenticationRequest<?, ?> authenticationRequest() {

Review Comment:
   We store passwords in the configuration, so I don't see any difference here.
   
   I've been thinking about the possibility that basic providers could store 
more than one user. We plan to implement this feature soon. But now it appears 
that there is a better approach to prevent closing connections for users who 
haven't updated.



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

Reply via email to