Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/997#discussion_r145317403 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -595,6 +611,12 @@ connectionStatus_t DrillClientImpl::validateHandshake(DrillUserProperties* prope switch(this->m_handshakeStatus) { case exec::user::SUCCESS: + // Check if client needs auth/encryption and server is not requiring it --- End diff -- Not too clear about the SASL flow, but I assume that if the server returning SUCCESS is sufficient to assume that there is no auth required by the server.
---