You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by sw...@apache.org on 2014/11/20 16:02:58 UTC

git commit: updated refs/heads/hotfix/4.4-7952 to 9f83a4d

Repository: cloudstack
Updated Branches:
  refs/heads/hotfix/4.4-7952 [created] 9f83a4d43


CLOUDSTACK-7952: Remove private key from SslCertResponse (listSslCerts)

Signed-off-by: Will Stevens <ws...@cloudops.com>


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/9f83a4d4
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/9f83a4d4
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/9f83a4d4

Branch: refs/heads/hotfix/4.4-7952
Commit: 9f83a4d43b36306c9c9ea231c1b25db40d616691
Parents: 892ba6e
Author: Syed <sa...@cloudops.com>
Authored: Fri Oct 31 23:24:44 2014 -0400
Committer: Will Stevens <ws...@cloudops.com>
Committed: Thu Nov 20 09:46:55 2014 -0500

----------------------------------------------------------------------
 .../org/apache/cloudstack/api/response/SslCertResponse.java  | 8 --------
 .../org/apache/cloudstack/network/lb/CertServiceImpl.java    | 1 -
 2 files changed, 9 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/9f83a4d4/api/src/org/apache/cloudstack/api/response/SslCertResponse.java
----------------------------------------------------------------------
diff --git a/api/src/org/apache/cloudstack/api/response/SslCertResponse.java b/api/src/org/apache/cloudstack/api/response/SslCertResponse.java
index 67011f9..8b3db46 100644
--- a/api/src/org/apache/cloudstack/api/response/SslCertResponse.java
+++ b/api/src/org/apache/cloudstack/api/response/SslCertResponse.java
@@ -40,10 +40,6 @@ public class SslCertResponse extends BaseResponse {
     @Param(description = "certificate")
     private String certificate;
 
-    @SerializedName(ApiConstants.PRIVATE_KEY)
-    @Param(description = "private key")
-    private String privatekey;
-
     @SerializedName(ApiConstants.ACCOUNT)
     @Param(description = "account for the certificate")
     private String accountName;
@@ -87,10 +83,6 @@ public class SslCertResponse extends BaseResponse {
         this.certificate = cert;
     }
 
-    public void setPrivatekey(String key) {
-        this.privatekey = key;
-    }
-
     public void setAccountName(String accountName) {
         this.accountName = accountName;
     }

http://git-wip-us.apache.org/repos/asf/cloudstack/blob/9f83a4d4/server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
----------------------------------------------------------------------
diff --git a/server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java b/server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
index cbc7dd8..8315bee 100644
--- a/server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
+++ b/server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
@@ -320,7 +320,6 @@ public class CertServiceImpl implements CertService {
         response.setObjectName("sslcert");
         response.setId(cert.getUuid());
         response.setCertificate(cert.getCertificate());
-        response.setPrivatekey(cert.getKey());
         response.setFingerprint(cert.getFingerPrint());
 
         if (cert.getChain() != null)