Issue Details (XML | Word | Printable)

Key: DIRSERVER-640
Type: Improvement Improvement
Status: Open Open
Priority: Major Major
Assignee: Unassigned
Reporter: Ralf Hauser
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Directory ApacheDS

bring error hints from CustomAuthenticators extending AbstractAuthenticator back to the client.

Created: 11/Jun/06 08:16 PM   Updated: 15/Aug/09 08:43 AM
Return to search
Component/s: ldap
Affects Version/s: 1.0-RC3
Fix Version/s: 2.0.0-RC1

Time Tracking:
Not Specified

File Attachments:
  Size
Text File Licensed for inclusion in ASF works AuthenticationService.java.patch 2006-06-18 02:21 AM Ralf Hauser 2 kB
Environment: windows/linux


 Description  « Hide
For the authentication, I use a CustomAuthenticator that extends AbstractAuthenticator.

If the authentication fails I use LdapAuthenticationException or LdapNoPermissionException and I appreciate a lot to be able to provide some hint (String explanation) why the exception was thrown.

Unfortunately, this hint never reaches the client. I only sees "error code 49 - Bind failed" - the equivalent is visible in the server log as
<<Ldap Result
            Result code : (ResultCodeEnum[INVALIDCREDENTIALS=49]) invalidCredentials
            Matched DN : 'null'
            Error message : 'Bind failed'>>

It appears that the culprit is org.apache.directory.server.core.authn.AuthenticationService.bind(NextInterceptor next, Name bindDn, byte[] credentials, List mechanisms, String saslAuthId) throws NamingException

 where that expception is caught, neither its class is analyzed in detail nor is there any attempt to use "explanations" when re-throwing even though an LdapAuthenticationException constructor does exist that takes a "msg" for explanations.

Therefore my suggestion: please make sure that it is possible to provide a user more information by optionally appending an "explantion" to the 'Bind failed' a client currently sees in an ldap client.

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Ralf Hauser added a comment - 18/Jun/06 02:21 AM
see also DIRSERVER-649

Ralf Hauser made changes - 18/Jun/06 02:21 AM
Field Original Value New Value
Attachment AuthenticationService.java.patch [ 12335586 ]
Emmanuel Lecharny added a comment - 09/Jul/08 01:25 PM
Postponed

Emmanuel Lecharny made changes - 09/Jul/08 01:25 PM
Fix Version/s 1.5.4 [ 12313147 ]
Alex Karasulu added a comment - 27/Aug/08 07:27 PM
Postponed for authn/authz push in 1.5.6 when all these issues can be tackled.

Alex Karasulu made changes - 27/Aug/08 07:27 PM
Fix Version/s 1.5.4 [ 12313147 ]
Fix Version/s 1.5.6 [ 12313383 ]
Emmanuel Lecharny added a comment - 15/Aug/09 08:43 AM
Moved to 2.0.0-RC1, we won't release a 1.5.6

Emmanuel Lecharny made changes - 15/Aug/09 08:43 AM
Fix Version/s 1.5.6 [ 12313383 ]
Fix Version/s 2.0.0-RC1 [ 12313387 ]