[DIRSERVER-1229] KerberosProtocolHandler has server and client swapped in logErrorMessage

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.5.3
• Component/s: changepw
• Environment: source-code review
• Assignee: Unassigned
• Reporter: gerke forcare
• Created: 13/Aug/08 10:38
• Updated: 07/Apr/09 22:04
• Resolved: 18/Aug/08 21:59

Description

One can read in the source (org.apache.directory.server.kerberos.protocol.KerberosProtocolHandler logErrorMessage):
sb.append( "Responding to request with error:" );
sb.append( "\n\t" + "explanatory text: " + error.getExplanatoryText() );
sb.append( "\n\t" + "error code: " + error.getErrorCode() );
sb.append( "\n\t" + "clientPrincipal: " + error.getClientPrincipal() );
sb.append( "\n\t" + "client time: " + error.getServerTime() );
sb.append( "\n\t" + "serverPrincipal: " + error.getServerPrincipal() );
sb.append( "\n\t" + "server time: " + error.getClientTime() );
which might need to read:
sb.append( "Responding to request with error:" );
sb.append( "\n\t" + "explanatory text: " + error.getExplanatoryText() );
sb.append( "\n\t" + "error code: " + error.getErrorCode() );
sb.append( "\n\t" + "clientPrincipal: " + error.getClientPrincipal() );
sb.append( "\n\t" + "client time: " + error.getClientTime() );
sb.append( "\n\t" + "serverPrincipal: " + error.getServerPrincipal() );
sb.append( "\n\t" + "server time: " + error.getServerTime() );