Hi Sinisa, thanks for reporting this. Please file this issue on JIRA.
But AFAIK the response generator is supposed to create the response,
the cmd api class should not add the password. I don't remember if it
was an issue, let's ask Min as she did a lot of the response
generating stuff.

Hi Min, can you check please?

Regards.

On Thu, Feb 14, 2013 at 3:36 PM, Sinisa Denic
<sde...@peacebellservers.com> wrote:
> Sorry I forgot to set cs-dev mailing list in cc in my previous post so other 
> users can see.
> I will repeat:
>
> In order to get password on UI after reset password for virtualmachine in 
> CS4.1.0-SNAPSHOT
> I had to made this modification:
>
> --- 
> a/api/src/org/apache/cloudstack/api/command/user/vm/ResetVMPasswordCmd.java
> +++ 
> b/api/src/org/apache/cloudstack/api/command/user/vm/ResetVMPasswordCmd.java
> @@ -114,6 +114,7 @@ public class ResetVMPasswordCmd extends BaseAsyncCmd {
>          if (result != null){
>              UserVmResponse response = 
> _responseGenerator.createUserVmResponse("virtualmachine", result).get(0);
>              response.setResponseName(getCommandName());
> +            response.setPassword(password);
>              this.setResponseObject(response);
>          } else {
>              throw new ServerApiException(ApiErrorCode.INTERNAL_ERROR, 
> "Failed to reset vm password");
>
> Do you maybe know where the problem could reside,as I spent a few days on it 
> :(?

Reply via email to