From 342f85c963fa15409e7a80e0bbbe0e42bc75bfe0 Mon Sep 17 00:00:00 2001
From: maskers <97827489+maskersss@users.noreply.github.com>
Date: Sun, 15 Sep 2024 13:59:46 +0300
Subject: [PATCH] Minor log parsing bug fixes (#402)

* fix java arg logic

* fix missing libraries detection for discord attached files

* change `\r\n` to `\n` in the entire log instead

---------

Co-authored-by: maskers <97827489+mskrss@users.noreply.github.com>
---
 src/handlers/event/analyze_logs/issues.rs | 4 ++--
 src/handlers/event/analyze_logs/mod.rs    | 2 ++
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/handlers/event/analyze_logs/issues.rs b/src/handlers/event/analyze_logs/issues.rs
index 26654c5..a5b258a 100644
--- a/src/handlers/event/analyze_logs/issues.rs
+++ b/src/handlers/event/analyze_logs/issues.rs
@@ -123,9 +123,9 @@ fn java_option(log: &str) -> Issue {
 
 	if let Some(captures) = vm_option.captures(log) {
 		let title = if &captures[1] == "UseShenandoahGC" {
-			"Wrong Java Arguments"
-		} else {
 			"Java 8 and below don't support ShenandoahGC"
+		} else {
+			"Wrong Java Arguments"
 		};
 		return Some((
 			title.to_string(),
diff --git a/src/handlers/event/analyze_logs/mod.rs b/src/handlers/event/analyze_logs/mod.rs
index b125b02..df89913 100644
--- a/src/handlers/event/analyze_logs/mod.rs
+++ b/src/handlers/event/analyze_logs/mod.rs
@@ -41,6 +41,8 @@ pub async fn handle(ctx: &Context, message: &Message, data: &Data) -> Result<()>
 		return Ok(());
 	};
 
+	let log = log.replace("\r\n", "\n");
+
 	let issues = issues::find(&log, data).await?;
 
 	let embed = {