DaanHoogland commented on a change in pull request #2710: comply with api key
constraint
URL: https://github.com/apache/cloudstack/pull/2710#discussion_r195325911
##########
File path: server/src/com/cloud/user/AccountManagerImpl.java
##########
@@ -1734,6 +1734,8 @@ public Boolean doInTransaction(TransactionStatus status)
{
UserVO newUser = new UserVO(user);
Review comment:
that would be a nice extra feature for the future ;)
No, actions in the old account must be tracable to a user in that account.
As the ownership of assets is with accounts and not users, and the physical
user is going to loose access to their old assets anyway, it is more convenient
to copy the login data of the user to a new user object.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
