Uploaded image for project: 'ZooKeeper'
  1. ZooKeeper
  2. ZOOKEEPER-2856

ZooKeeperSaslClient#respondToServer should log exception message of SaslException

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.4.10, 3.5.3
    • Fix Version/s: 3.4.11, 3.5.4, 3.6.0
    • Component/s: None
    • Labels:
      None

      Description

      When upstream like HBase call ZooKeeperSaslClient with security enabled, we sometimes get error in HBase logs like:

      SASL authentication failed using login context 'Client'.
      

      This error occures when getting SaslException in ZooKeeperSaslClient#respondToServer :

       catch (SaslException e) {
                      LOG.error("SASL authentication failed using login context '" +
                              this.getLoginContext() + "'.");
                      saslState = SaslState.FAILED;
                      gotLastPacket = true;
        }
      

      This error makes user confused without explicit exception message. So I think we can add exception message to the log.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                panyuxuan Pan Yuxuan
                Reporter:
                panyuxuan Pan Yuxuan
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: