[QPID-1505] PlainSaslServer throws and immediately catches and wraps Sasl Exception

• 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: starter
• Assignee: Robbie Gemmell
• Reporter: Martin Ritchie
• Created: 03/Dec/08 14:09
• Updated: 11/Feb/15 20:06
• Resolved: 20/Feb/12 16:10

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.