rhtyd commented on a change in pull request #4860:
URL: https://github.com/apache/cloudstack/pull/4860#discussion_r600408568
##########
File path: server/src/main/java/com/cloud/api/ApiServer.java
##########
@@ -1130,7 +1130,7 @@ public ResponseObject loginUser(final HttpSession
session, final String username
return createLoginResponse(session);
}
- throw new CloudAuthenticationException("Failed to authenticate user "
+ username + " in domain " + domainId + "; please provide valid credentials");
+ throw new CloudAuthenticationException("Failed to authenticate user "
+ username + " in domain " + userDomain.getPath() + "; please provide valid
credentials");
Review comment:
minor nit - General convention would be to override the `toString()`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
