tried to implement the server auth. (did not work)

added ImageProcessor to process image after ftp upload

4 files changed, 30 insertions, 13 deletions

diff --git a/src/main/java/org/openslx/imagemaster/server/ApiServer.java b/src/main/java/org/openslx/imagemaster/server/ApiServer.java
index 2c49a0d..258c490 100644
--- a/src/main/java/org/openslx/imagemaster/server/ApiServer.java
+++ b/src/main/java/org/openslx/imagemaster/server/ApiServer.java
@@ -103,6 +103,9 @@ public class ApiServer {
 		 * 
 		 * And removed with: removeUser(username);
 		 */
+		if (ServerSessionManager.getSession(serverSessionData.sessionId) == null) {
+			throw new AuthenticationException(AuthenticationError.GENERIC_ERROR, "No valid serverSessionData");
+		}
 
 		String generatedUser = RandomString.generate(10, false);
 		String generatedPass = RandomString.generate(16, true);
diff --git a/src/main/java/org/openslx/imagemaster/server/ImageProcessor.java b/src/main/java/org/openslx/imagemaster/server/ImageProcessor.java
new file mode 100644
index 0000000..e10ea21
--- /dev/null
+++ b/src/main/java/org/openslx/imagemaster/server/ImageProcessor.java
@@ -0,0 +1,14 @@
+package org.openslx.imagemaster.server;
+
+import org.apache.log4j.Logger;
+
+public class ImageProcessor {
+	
+	private static Logger log = Logger.getLogger(ImageProcessor.class);
+	public static void processImageAfterUpload(String username, String filename) {
+		/*
+		 * TODO: Process the image after download
+		 */
+		
+	}
+}
diff --git a/src/main/java/org/openslx/imagemaster/server/MasterFtpServer.java b/src/main/java/org/openslx/imagemaster/server/MasterFtpServer.java
index 9ec5cf7..973c768 100644
--- a/src/main/java/org/openslx/imagemaster/server/MasterFtpServer.java
+++ b/src/main/java/org/openslx/imagemaster/server/MasterFtpServer.java
@@ -19,7 +19,7 @@ import org.apache.ftpserver.usermanager.impl.WritePermission;
 import org.apache.log4j.Logger;
 
 public class MasterFtpServer implements Runnable {
-	private static Logger log = Logger.getLogger( ApiServer.class );
+	private static Logger log = Logger.getLogger( MasterFtpServer.class );
 	private FtpServer server;
 	private UserManager userManager;
 	
diff --git a/src/main/java/org/openslx/imagemaster/server/MasterFtplet.java b/src/main/java/org/openslx/imagemaster/server/MasterFtplet.java
index 73d69d2..a1b9a90 100644
--- a/src/main/java/org/openslx/imagemaster/server/MasterFtplet.java
+++ b/src/main/java/org/openslx/imagemaster/server/MasterFtplet.java
@@ -10,29 +10,25 @@ import org.apache.ftpserver.ftplet.Ftplet;
 import org.apache.ftpserver.ftplet.FtpletContext;
 import org.apache.ftpserver.ftplet.FtpletResult;
 import org.apache.log4j.Logger;
-import org.openslx.imagemaster.App;
 
 public class MasterFtplet implements Ftplet {
-	private static Logger log = Logger.getLogger(App.class);
+	private static Logger log = Logger.getLogger(Ftplet.class);
 
 	@Override
 	public void init(FtpletContext ftpletContext) throws FtpException {
-		// TODO Auto-generated method stub
-
+		// not used
 	}
 
 	@Override
 	public void destroy() {
-		// TODO Auto-generated method stub
-
+		// not used
 	}
 
 	@Override
 	public FtpletResult beforeCommand(FtpSession session, FtpRequest request)
 			throws FtpException, IOException {
-		if (session.getUser() != null && request.getCommand() == "QUIT") {
-			// TODO: tell server that user finished uploading
-			String user = session.getUser().getName();
+		if (session.getUser() != null) {
+			log.info(session.getUser().getName() + " issued command: " + request.getRequestLine());
 		}
 		return null;
 	}
@@ -40,21 +36,25 @@ public class MasterFtplet implements Ftplet {
 	@Override
 	public FtpletResult afterCommand(FtpSession session, FtpRequest request,
 			FtpReply reply) throws FtpException, IOException {
-		// TODO Auto-generated method stub
+		if (session.getUser() != null) {
+			log.info(session.getUser().getName() + " is done with command: " + request.getRequestLine());
+			// TODO: delete temporary user
+			ImageProcessor.processImageAfterUpload(session.getUser().getName(), request.getRequestLine().split(" ")[1]);
+		}
 		return null;
 	}
 
 	@Override
 	public FtpletResult onConnect(FtpSession session) throws FtpException,
 			IOException {
-		// TODO Auto-generated method stub
+		// not used
 		return null;
 	}
 
 	@Override
 	public FtpletResult onDisconnect(FtpSession session) throws FtpException,
 			IOException {
-		// TODO Auto-generated method stub
+		// not used
 		return null;
 	}