Github user parthchandra commented on a diff in the pull request:
https://github.com/apache/drill/pull/997#discussion_r145317288
--- 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
+ if(clientNeedsAuthentication(properties) ||
clientNeedsEncryption(properties)) {
--- End diff --
Generally, all error messages come from errmsgs.cpp so we can localize them
when we need to.
---
