Uploaded image for project: 'Qpid'
  1. Qpid
  2. QPID-1505

PlainSaslServer throws and immediately catches and wraps Sasl Exception

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: M3, M4, 0.14
    • Fix Version/s: 0.15
    • Component/s: Broker-J
    • Labels:

      Description

      Summary:

      in evaluateResponse Line 94 throws a SaslException saying Authentication Failed but this is then immediately caught on line 97 as SaslException extends IOException. Where by the SaslException is Wrapped in a new SaslException and thrown.

      We probably don't want to do this, something simple like moving the if....else... out of the try...catch would suffice.

        Attachments

          Activity

            People

            • Assignee:
              gemmellr Robbie Gemmell
              Reporter:
              ritchiem Martin Ritchie
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: