From 1299e8769d0dde9439536687bd14244415bd1639 Mon Sep 17 00:00:00 2001
From: James <38226001+james-munson@users.noreply.github.com>
Date: Fri, 31 May 2019 01:15:09 -0700
Subject: [PATCH] Add sentry log support (#454)

* Add sentry log support

* Add sentry log support

* Update
---
 runelite-client/pom.xml                       |  5 +++++
 .../runeliteplus/RuneLitePlusConfig.java      | 11 ++++++++++
 .../runeliteplus/RuneLitePlusPlugin.java      |  6 +++++
 .../src/main/resources/logback.xml            | 22 +++++++++++++++++++
 4 files changed, 44 insertions(+)

diff --git a/runelite-client/pom.xml b/runelite-client/pom.xml
index 85061e5131..8d50b6f66f 100644
--- a/runelite-client/pom.xml
+++ b/runelite-client/pom.xml
@@ -275,6 +275,11 @@
 			<artifactId>asm-all</artifactId>
 			<version>6.0_BETA</version>
 		</dependency>
+		<dependency>
+			<groupId>io.sentry</groupId>
+			<artifactId>sentry-logback</artifactId>
+			<version>1.7.16</version>
+		</dependency>
     </dependencies>
 
 	<build>
diff --git a/runelite-client/src/main/java/net/runelite/client/plugins/runeliteplus/RuneLitePlusConfig.java b/runelite-client/src/main/java/net/runelite/client/plugins/runeliteplus/RuneLitePlusConfig.java
index 8db739148c..8eda3a582e 100644
--- a/runelite-client/src/main/java/net/runelite/client/plugins/runeliteplus/RuneLitePlusConfig.java
+++ b/runelite-client/src/main/java/net/runelite/client/plugins/runeliteplus/RuneLitePlusConfig.java
@@ -81,4 +81,15 @@ public interface RuneLitePlusConfig extends Config
 	{
 		return false;
 	}
+
+	@ConfigItem(
+			keyName = "logOpt",
+			name = "Send logs",
+			description = "Send logs to help us analyze errors",
+			position = 4
+	)
+	default boolean logOpt()
+	{
+		return false;
+	}
 }
diff --git a/runelite-client/src/main/java/net/runelite/client/plugins/runeliteplus/RuneLitePlusPlugin.java b/runelite-client/src/main/java/net/runelite/client/plugins/runeliteplus/RuneLitePlusPlugin.java
index a074c17410..6214ceeb8d 100644
--- a/runelite-client/src/main/java/net/runelite/client/plugins/runeliteplus/RuneLitePlusPlugin.java
+++ b/runelite-client/src/main/java/net/runelite/client/plugins/runeliteplus/RuneLitePlusPlugin.java
@@ -33,6 +33,7 @@ import java.awt.event.KeyEvent;
 import java.util.HashMap;
 import java.util.Map;
 import javax.inject.Inject;
+import io.sentry.Sentry;
 import lombok.extern.slf4j.Slf4j;
 import net.runelite.api.Client;
 import net.runelite.api.events.ConfigChanged;
@@ -157,6 +158,11 @@ public class RuneLitePlusPlugin extends Plugin
 			ClientUI.frame.setTitle(ClientUI.currentPresenceName);
 		}
 
+		if (config.logOpt())
+		{
+			Sentry.init("https://f0ed76be2fe847f8b9eb3620fa55d729@sentry.io/1468399?stacktrace.app.packages=net.runelite.client");
+		}
+
 		if (config.customPresence())
 		{
 			RuneLiteProperties.discordAppID = rlPlusDiscordApp;
diff --git a/runelite-client/src/main/resources/logback.xml b/runelite-client/src/main/resources/logback.xml
index d3a5ffbcad..b4adf47b85 100644
--- a/runelite-client/src/main/resources/logback.xml
+++ b/runelite-client/src/main/resources/logback.xml
@@ -51,8 +51,30 @@
         </encoder>
     </appender>
 
+    <!-- Configure the Sentry appender, overriding the logging threshold to the WARN level -->
+    <appender name="SENTRY_ERROR" class="io.sentry.logback.SentryAppender">
+        <filter class="ch.qos.logback.classic.filter.ThresholdFilter">
+            <level>ERROR</level>
+        </filter>
+    </appender>
+
+    <appender name="SENTRY_WARN" class="io.sentry.logback.SentryAppender">
+        <filter class="ch.qos.logback.classic.filter.ThresholdFilter">
+            <level>WARN</level>
+        </filter>
+    </appender>
+
     <root level="INFO">
         <appender-ref ref="STDOUT"/>
         <appender-ref ref="FILE"/>
     </root>
+
+    <root level="ERROR">
+        <appender-ref ref="SENTRY_ERROR" />
+    </root>
+
+    <root level="WARN">
+        <appender-ref ref="SENTRY_WARN" />
+    </root>
+
 </configuration>
\ No newline at end of file