----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14305/#review27218 -----------------------------------------------------------
api/src/org/apache/cloudstack/api/response/SSHKeyPairResponse.java <https://reviews.apache.org/r/14305/#comment52933> Why this is changed to protected? api/src/org/apache/cloudstack/api/response/SSHKeyPairResponse.java <https://reviews.apache.org/r/14305/#comment52934> Protected? This is not required - Koushik Das On Oct. 18, 2013, 12:43 p.m., Harikrishna Patnala wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/14305/ > ----------------------------------------------------------- > > (Updated Oct. 18, 2013, 12:43 p.m.) > > > Review request for cloudstack and Koushik Das. > > > Bugs: CLOUDSTACK-4493 > https://issues.apache.org/jira/browse/CLOUDSTACK-4493 > > > Repository: cloudstack-git > > > Description > ------- > > CLOUDSTACK-4493: registerSSHKeyPair API doc contains wrong API response > (private key) > Changing the response object of register and list ssh keypairs APIs > > > Diffs > ----- > > api/src/org/apache/cloudstack/api/command/user/ssh/CreateSSHKeyPairCmd.java > 56bec7e > api/src/org/apache/cloudstack/api/response/CreateSSHKeyPairResponse.java > PRE-CREATION > api/src/org/apache/cloudstack/api/response/SSHKeyPairResponse.java 2791853 > > Diff: https://reviews.apache.org/r/14305/diff/ > > > Testing > ------- > > > Thanks, > > Harikrishna Patnala > >