[SSHD-102] Add error logging to org.apache.sshd.server.jaas.JaasPasswordAuthenticator.authenticate()

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.4.0, 0.5.0
• Fix Version/s: 0.6.0
• Environment: sshd JAAS used with LDAP based authentication.
• Assignee: Guillaume Nodet
• Reporter: Torsten Mielke
• Created: 15/Dec/10 11:20
• Updated: 17/Dec/10 09:06
• Resolved: 17/Dec/10 09:06

Description

The JaasPasswordAuthenticator.authenticate() method currently silently ignores any exceptions being raised and simply returns false in case of any authentication erorrs.
In environments where sshd is used in other frameworks like OSGi, it becomes very difficult to trouble shoot the reasons for the authentication failure.
Rather than simply returning false, I propose to print a logging statement at the least.

catch (Exception e)

{ log.error("Authentication failed with error: " + e.getMessage() + ", cause: " + e.getCause() ); return false; }