Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/997#discussion_r145568130 --- 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 -- Yes. The control flow goes through the AUTH_REQUIRED case when the server requires auth.
---