Merge pull request #49 from omgapuppy/master

Remove LogTooLongException

Author: StephenHynes7

.travis.yml

@@ -4,3 +4,11 @@ jdk:
   - openjdk7
   - oraclejdk7
   - oraclejdk8
+before_install:
+  # override default MAVEN_OPTS
+  - echo "MAVEN_OPTS='-Xms1g -Xmx2g -XX:MaxPermSize=512m'" > ~/.mavenrc
+  # Fix OpenJDK buffer overflow
+  - cat /etc/hosts # optionally check the content *before*
+  - sudo hostname "$(hostname | cut -c1-63)"
+  - sed -e "s/^\\(127\\.0\\.0\\.1.*\\)/\\1 $(hostname | cut -c1-63)/" /etc/hosts | sudo tee /etc/hosts
+  - cat /etc/hosts # optionally check the content *after*

pom.xml

@@ -72,6 +72,16 @@
                     </execution>
                 </executions>
             </plugin>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-surefire-plugin</artifactId>
+                <version>2.19.1</version>
+                <configuration>
+                    <!-- Travis build workaround -->
+                    <forkCount>1</forkCount>
+                    <argLine>${surefire.jvm.params}</argLine>
+                </configuration>
+            </plugin>
         </plugins>
     </build>
     <profiles>

src/main/java/com/logentries/net/AsyncLogger.java

@@ -270,53 +270,53 @@ public int getDataHubPort() {
 	 *
 	 * @param logHostName
 	 */
-	public void setLogHostName(boolean logHostName) { 
-		this.logHostName = logHostName; 
+	public void setLogHostName(boolean logHostName) {
+		this.logHostName = logHostName;
 	}
 
 	/**
 	 * Gets value of the switch that determines whether to send HostName alongside with the log message
 	 *
 	 * @return logHostName switch value
 	 */
-	public boolean getLogHostName() { 
-		return this.logHostName; 
+	public boolean getLogHostName() {
+		return this.logHostName;
 	}
 
 	/**
 	 * Sets the HostName from configuration
 	 *
 	 * @param hostName
 	 */
-	public void setHostName(String hostName) { 
-		this.hostName = hostName; 
+	public void setHostName(String hostName) {
+		this.hostName = hostName;
 	}
 
 	/**
 	 * Gets HostName parameter
 	 *
 	 * @return Host name field value
 	 */
-	public String getHostName() { 
-		return this.hostName; 
+	public String getHostName() {
+		return this.hostName;
 	}
 
 	/**
 	 * Sets LogID parameter from config
 	 *
 	 * @param logID
 	 */
-	public void setLogID(String logID) { 
-		this.logID = logID; 
+	public void setLogID(String logID) {
+		this.logID = logID;
 	}
 
 	/**
 	 * Gets LogID parameter
 	 *
 	 * @return logID field value
 	 */
-	public String getLogID() { 
-		return this.logID; 
+	public String getLogID() {
+		return this.logID;
 	}
 
 	/**
@@ -419,7 +419,9 @@ public void addLineToQueue(String line) {
 	}
 
 	private void addLineToQueue (String line, int limit) {
-		if (limit == 0) { throw new LogTooLongException(); }
+		if (limit == 0) {
+            dbg("Message longer than " + RECURSION_LIMIT);
+            return; }
 
 		//// Check credentials only if logs are sent to LE directly.
 		// Check that we have all parameters set and socket appender running.

src/main/java/com/logentries/net/LogTooLongException.java

@@ -1,6 +1,9 @@
 package com.logentries.net;
 
 import org.junit.Test;
+
+import java.util.Random;
+
 import static org.junit.Assert.*;
 
 public class AsyncLoggerTest {
@@ -17,6 +20,21 @@ public void testGetAndSetToken()
 	}
 
 	@Test
+    public void testOversizeMessage()
+    {
+        AsyncLogger async = new AsyncLogger();
+        char[] chars = "abcdefghijklmnopqrstuvwxyz".toCharArray();
+        StringBuilder sb = new StringBuilder();
+        Random random = new Random();
+        for (int i = 0; i < 2100000; i++) {
+            char c = chars[random.nextInt(chars.length)];
+            sb.append(c);
+        }
+        String output = sb.toString();
+        async.addLineToQueue(output);
+    }
+
+    @Test
 	public void testGetAndSetHttpPut()
 	{
 		AsyncLogger async = new AsyncLogger();