You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by du...@apache.org on 2014/06/25 20:53:03 UTC
git commit: updated refs/heads/4.4-forward to 96412e3
Repository: cloudstack
Updated Branches:
refs/heads/4.4-forward 2498f6568 -> 96412e3e5
Allow for length of password on VMs to be set via global settings
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/96412e3e
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/96412e3e
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/96412e3e
Branch: refs/heads/4.4-forward
Commit: 96412e3e58fd1ced9d269e4552aaa6410bedf556
Parents: 2498f65
Author: Ian Duffy <ia...@paddypower.com>
Authored: Wed Jun 25 19:51:31 2014 +0100
Committer: Ian Duffy <ia...@paddypower.com>
Committed: Wed Jun 25 19:51:31 2014 +0100
----------------------------------------------------------------------
server/src/com/cloud/server/ManagementServerImpl.java | 3 ++-
server/src/com/cloud/vm/UserVmManagerImpl.java | 13 ++++++-------
setup/db/db/schema-430to440.sql | 1 +
3 files changed, 9 insertions(+), 8 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/96412e3e/server/src/com/cloud/server/ManagementServerImpl.java
----------------------------------------------------------------------
diff --git a/server/src/com/cloud/server/ManagementServerImpl.java b/server/src/com/cloud/server/ManagementServerImpl.java
index f373e8b..1e9a91e 100755
--- a/server/src/com/cloud/server/ManagementServerImpl.java
+++ b/server/src/com/cloud/server/ManagementServerImpl.java
@@ -894,7 +894,8 @@ public class ManagementServerImpl extends ManagerBase implements ManagementServe
@Override
public String generateRandomPassword() {
- return PasswordGenerator.generateRandomPassword(6);
+ Integer passwordLength = Integer.parseInt(_configDao.getValue("vm.password.length"));
+ return PasswordGenerator.generateRandomPassword(passwordLength);
}
@Override
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/96412e3e/server/src/com/cloud/vm/UserVmManagerImpl.java
----------------------------------------------------------------------
diff --git a/server/src/com/cloud/vm/UserVmManagerImpl.java b/server/src/com/cloud/vm/UserVmManagerImpl.java
index 68503ae..cd4eb21 100755
--- a/server/src/com/cloud/vm/UserVmManagerImpl.java
+++ b/server/src/com/cloud/vm/UserVmManagerImpl.java
@@ -35,6 +35,7 @@ import javax.ejb.Local;
import javax.inject.Inject;
import javax.naming.ConfigurationException;
+import com.cloud.server.ManagementService;
import org.apache.commons.codec.binary.Base64;
import org.apache.log4j.Logger;
@@ -457,6 +458,8 @@ public class UserVmManagerImpl extends ManagerBase implements UserVmManager, Vir
VolumeApiService _volumeService;
@Inject
DataStoreManager _dataStoreMgr;
+ @Inject
+ ManagementService _mgr;
protected ScheduledExecutorService _executor = null;
protected int _expungeInterval;
@@ -642,7 +645,7 @@ public class UserVmManagerImpl extends ManagerBase implements UserVmManager, Vir
String password = null;
String sshPublicKey = s.getPublicKey();
if (template != null && template.getEnablePassword()) {
- password = generateRandomPassword();
+ password = _mgr.generateRandomPassword();
}
boolean result = resetVMSSHKeyInternal(vmId, sshPublicKey, password);
@@ -3353,10 +3356,6 @@ public class UserVmManagerImpl extends ManagerBase implements UserVmManager, Vir
}
}
- public String generateRandomPassword() {
- return PasswordGenerator.generateRandomPassword(6);
- }
-
@Override
public Pair<UserVmVO, Map<VirtualMachineProfile.Param, Object>> startVirtualMachine(long vmId, Long hostId, Map<VirtualMachineProfile.Param, Object> additionalParams, String deploymentPlannerToUse)
throws ConcurrentOperationException, ResourceUnavailableException, InsufficientCapacityException {
@@ -3431,7 +3430,7 @@ public class UserVmManagerImpl extends ManagerBase implements UserVmManager, Vir
String password = "saved_password";
if (template.getEnablePassword()) {
- password = generateRandomPassword();
+ password = _mgr.generateRandomPassword();
}
if (!validPassword(password)) {
@@ -4705,7 +4704,7 @@ public class UserVmManagerImpl extends ManagerBase implements UserVmManager, Vir
}
if (template.getEnablePassword()) {
- String password = generateRandomPassword();
+ String password = _mgr.generateRandomPassword();
boolean result = resetVMPasswordInternal(vmId, password);
if (result) {
vm.setPassword(password);
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/96412e3e/setup/db/db/schema-430to440.sql
----------------------------------------------------------------------
diff --git a/setup/db/db/schema-430to440.sql b/setup/db/db/schema-430to440.sql
index 0293986..6c9783a 100644
--- a/setup/db/db/schema-430to440.sql
+++ b/setup/db/db/schema-430to440.sql
@@ -2451,3 +2451,4 @@ CREATE TABLE `cloud`.`snapshot_policy_details` (
CONSTRAINT `fk_snapshot_policy_details__snapshot_policy_id` FOREIGN KEY `fk_snapshot_policy_details__snapshot_policy_id`(`policy_id`) REFERENCES `snapshot_policy`(`id`) ON DELETE CASCADE
) ENGINE=InnoDB DEFAULT CHARSET=utf8;
+INSERT INTO `cloud`.`configuration`(category, instance, component, name, value, description, default_value) VALUES ('Secure', 'DEFAULT', 'management-server', 'vm.password.length', '6', 'Specifies the length of a randomly generated password', '6') ON DUPLICATE KEY UPDATE category='Secure';
\ No newline at end of file