milamberspace closed pull request #1902: CLOUDSTACK-9736 Incoherent validation
and error message when you chan?
URL: https://github.com/apache/cloudstack/pull/1902
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):
diff --git a/server/src/com/cloud/configuration/ConfigurationManagerImpl.java
b/server/src/com/cloud/configuration/ConfigurationManagerImpl.java
index 10136c6bf32..847f1330caf 100644
--- a/server/src/com/cloud/configuration/ConfigurationManagerImpl.java
+++ b/server/src/com/cloud/configuration/ConfigurationManagerImpl.java
@@ -797,8 +797,8 @@ private String validateConfigurationValue(final String
name, String value, final
throw new InvalidParameterValueException("Please enter a
positive value for the configuration parameter:" + name);
}
//TODO - better validation for all password pamameters
- if ("vm.password.length".equalsIgnoreCase(name) && val < 10) {
- throw new InvalidParameterValueException("Please enter a
value greater than 6 for the configuration parameter:" + name);
+ if ("vm.password.length".equalsIgnoreCase(name) && val < 6) {
+ throw new InvalidParameterValueException("Please enter a
value greater or equal than 6 for the configuration parameter:" + name);
}
if ("remote.access.vpn.psk.length".equalsIgnoreCase(name)) {
if (val < 8) {
diff --git a/server/src/com/cloud/server/ManagementServerImpl.java
b/server/src/com/cloud/server/ManagementServerImpl.java
index b6a263705fa..04ee1f2e4d1 100644
--- a/server/src/com/cloud/server/ManagementServerImpl.java
+++ b/server/src/com/cloud/server/ManagementServerImpl.java
@@ -684,7 +684,7 @@
public class ManagementServerImpl extends ManagerBase implements
ManagementServer, Configurable {
public static final Logger s_logger =
Logger.getLogger(ManagementServerImpl.class.getName());
- static final ConfigKey<Integer> vmPasswordLength = new
ConfigKey<Integer>("Advanced", Integer.class, "vm.password.length", "10",
+ static final ConfigKey<Integer> vmPasswordLength = new
ConfigKey<Integer>("Advanced", Integer.class, "vm.password.length", "6",
"Specifies the length of a randomly generated password", false);
@Inject
public AccountManager _accountMgr;
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
