Hadoop Common
  1. Hadoop Common
  2. HADOOP-6718

Client does not close connection when an exception happens during SASL negotiation

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Duplicate
    • Affects Version/s: 0.22.0
    • Fix Version/s: 0.22.0
    • Component/s: ipc
    • Labels:
      None

      Description

      setupSaslConnection in the RPC client might fail to successfully set up a sasl connection (e.g. if the principal is wrongly configured). It throws an exception back to the caller (setupIOstreams). setupIOstreams marks the connection as closed but it doesn't really close the socket.

      1. 6718-bp20.patch
        0.8 kB
        Devaraj Das

        Issue Links

          Activity

          Hide
          Devaraj Das added a comment -

          Attaching a patch for Y20S. Not for commit.

          Show
          Devaraj Das added a comment - Attaching a patch for Y20S. Not for commit.
          Hide
          Jakob Homan added a comment -

          Seems reasonable. +1.

          Show
          Jakob Homan added a comment - Seems reasonable. +1.
          Hide
          Soundararajan Velu added a comment -

          Devaraj, Patch looks great, but shouldn't we catch Exception instead of IOException, Just in case... and then handle the situation.

          Show
          Soundararajan Velu added a comment - Devaraj, Patch looks great, but shouldn't we catch Exception instead of IOException, Just in case... and then handle the situation.
          Hide
          Konstantin Shvachko added a comment -

          Incorporated in HADOOP-6706 for 0.22.

          Show
          Konstantin Shvachko added a comment - Incorporated in HADOOP-6706 for 0.22.

            People

            • Assignee:
              Devaraj Das
              Reporter:
              Devaraj Das
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development