diff options
author | Nils Schwabe | 2014-04-22 19:04:01 +0200 |
---|---|---|
committer | Nils Schwabe | 2014-04-22 19:04:01 +0200 |
commit | 59b4035f8c9d9db72f389e31c2941316e20e29f0 (patch) | |
tree | eabe180ab584ef26c0ed8341d358b67ac44d5586 /src/main/java/org | |
parent | ignore ftp.properties (diff) | |
download | masterserver-59b4035f8c9d9db72f389e31c2941316e20e29f0.tar.gz masterserver-59b4035f8c9d9db72f389e31c2941316e20e29f0.tar.xz masterserver-59b4035f8c9d9db72f389e31c2941316e20e29f0.zip |
Fixed something
Diffstat (limited to 'src/main/java/org')
-rw-r--r-- | src/main/java/org/openslx/imagemaster/Globals.java | 12 | ||||
-rw-r--r-- | src/main/java/org/openslx/imagemaster/server/ApiServer.java | 5 |
2 files changed, 15 insertions, 2 deletions
diff --git a/src/main/java/org/openslx/imagemaster/Globals.java b/src/main/java/org/openslx/imagemaster/Globals.java index 653e255..7dbbae4 100644 --- a/src/main/java/org/openslx/imagemaster/Globals.java +++ b/src/main/java/org/openslx/imagemaster/Globals.java @@ -22,14 +22,22 @@ public class Globals { public static final String ldapSearchFilter = "ldap_search_filter"; public static boolean propertiesValid() { - if (Globals.properties.getProperty(ftpBaseDir).isEmpty() + if (Globals.properties.getProperty(ftpBaseDir) == null + || Globals.properties.getProperty(ftpBaseDir).isEmpty() + || Globals.properties.getProperty(imageDir) == null || Globals.properties.getProperty(imageDir).isEmpty() + || Globals.properties.getProperty(ldapPort) == null || Globals.properties.getProperty(ldapPort).isEmpty() + || Globals.properties.getProperty(ldapHost) == null || Globals.properties.getProperty(ldapHost).isEmpty() + || Globals.properties.getProperty(ldapSsl) == null || Globals.properties.getProperty(ldapSsl).isEmpty() + || Globals.properties.getProperty(ldapBindQuery) == null || Globals.properties.getProperty(ldapBindQuery).isEmpty() + || Globals.properties.getProperty(ldapSearchBaseDn) == null || Globals.properties.getProperty(ldapSearchBaseDn).isEmpty() - || Globals.properties.getProperty(ldapSearchBaseDn).isEmpty() + || Globals.properties.getProperty(ldapSearchFilter) == null + || Globals.properties.getProperty(ldapSearchFilter).isEmpty() ) { return false; } diff --git a/src/main/java/org/openslx/imagemaster/server/ApiServer.java b/src/main/java/org/openslx/imagemaster/server/ApiServer.java index 1fcfa49..df0aa3a 100644 --- a/src/main/java/org/openslx/imagemaster/server/ApiServer.java +++ b/src/main/java/org/openslx/imagemaster/server/ApiServer.java @@ -97,6 +97,11 @@ public class ApiServer { // create new user FtpCredentials ftpCredentials = Globals.ftpServer.addUser(serverSessionId); + if (ftpCredentials == null) { + log.error("Could not create ftp credentials"); + return null; + } + if (!ImageProcessor.addImageDataToProcess(imageDescription, ftpCredentials.username)) { Globals.ftpServer.removeUser(serverSessionId); throw new TException("ImageData is not valid."); |