bugfixing

src/main/java/de/gurkengewuerz/termbin/Config.java

@@ -50,6 +50,7 @@ public class Config extends JSONObject {
 
         JSONObject uploadserver = new JSONObject();
         uploadserver.put("port", 8888);
+        uploadserver.put("timeout", 5);
         uploadserver.put("bind", "0.0.0.0");
         this.put("uploadserver", uploadserver);
 

src/main/java/de/gurkengewuerz/termbin/Server/UploadServer.java

@@ -53,7 +53,7 @@ public class UploadServer {
             this.clientIP = socketOfServer.getInetAddress().toString().substring(1);
             this.client = "client#" + clientNumber + "@" + clientIP + ":" + socketOfServer.getPort();
             try {
-                this.socketOfServer.setSoTimeout(2000);
+                this.socketOfServer.setSoTimeout(Termbin.getConfig().getUploadServerConfig().getInt("timeout") * 1000);
             } catch (SocketException e) {
                 Logger.getLogger(getClass().getName()).log(Level.SEVERE, null, e);
             }
@@ -91,19 +91,27 @@ public class UploadServer {
                             return;
                         }
                     }
-                } catch (SocketTimeoutException e) {
+                } catch (SocketTimeoutException ignored) {
-                    Logger.getLogger(getClass().getName()).log(Level.INFO, "closed " + client + " with error invalid data");
-                    os.write("Invalid data");
-                    os.newLine();
-                    os.flush();
-                    socketOfServer.close();
-                    return;
                 }
 
                 byte[] data = baos.toByteArray();
                 if (data.length > 3) {
                     String dataString = new String(data, "UTF-8");
 
+                    boolean invalid = false;
+                    if (dataString.startsWith("GET") || dataString.startsWith("POST"))
+                        if (dataString.contains("HTTP/1."))
+                            invalid = true;
+
+                    if (invalid) {
+                        Logger.getLogger(getClass().getName()).log(Level.INFO, "closed " + client + " with error invalid data/HTTP");
+                        os.write("Please use netcat");
+                        os.newLine();
+                        os.flush();
+                        socketOfServer.close();
+                        return;
+                    }
+
                     Termbin.FileType ft = Termbin.FileType.TXT;
 
                     if (ImageUtils.isValidPNG(data))
@@ -122,8 +130,8 @@ public class UploadServer {
                         os.write("Server Error");
                     }
                 } else {
-                    os.write("Invalid data");
+                    os.write("No Data Provided");
-                    Logger.getLogger(getClass().getName()).log(Level.INFO, "closed " + client + " with error invalid data");
+                    Logger.getLogger(getClass().getName()).log(Level.INFO, "closed " + client + " with error invalid data/length");
                 }
                 os.newLine();
                 os.flush();

src/main/java/de/gurkengewuerz/termbin/Termbin.java

@@ -37,7 +37,6 @@ public class Termbin {
         createDatabase();
 
         FileHandler fh = new FileHandler(conf.getString("accesslog") + ".%u.log", 5242880, 5, true);
-//        FileHandler fh = new FileHandler(conf.getString("accesslog") + ".log", true);
         SimpleFormatter sf = new SimpleFormatter();
         fh.setFormatter(sf);
         accesslog.setUseParentHandlers(false);