Skip to content

Commit 247b534

Browse files
committed
also take domainId into account
1 parent d08de80 commit 247b534

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

server/src/main/java/org/apache/cloudstack/network/ssl/CertServiceImpl.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ public SslCertResponse uploadSslCert(final UploadSslCertCmd certCmd) {
132132
final Account caller = ctx.getCallingAccount();
133133

134134
Account owner;
135-
if (StringUtils.isNotBlank(certCmd.getAccountName()) || certCmd.getProjectId() != null) {
135+
if ((StringUtils.isNotBlank(certCmd.getAccountName()) && certCmd.getDomainId() != null) || certCmd.getProjectId() != null) {
136136
owner = _accountMgr.finalizeOwner(caller, certCmd.getAccountName(), certCmd.getDomainId(), certCmd.getProjectId());
137137
} else {
138138
owner = caller;
@@ -228,7 +228,7 @@ public List<SslCertResponse> listSslCerts(final ListSslCertsCmd listSslCertCmd)
228228
}
229229

230230
Account owner;
231-
if (StringUtils.isNotBlank(accountName) || projectId != null) {
231+
if ((StringUtils.isNotBlank(accountName) && domainId != null) || projectId != null) {
232232
owner = _accountMgr.finalizeOwner(caller, accountName, domainId, projectId);
233233
} else {
234234
owner = caller;

0 commit comments

Comments
 (0)