drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bitblender <...@git.apache.org>
Subject [GitHub] drill pull request #997: DRILL-5882: C++ Client: [Threat Modeling] Drillbit ...
Date Wed, 18 Oct 2017 23:21:02 GMT
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.


---

Mime
View raw message