Safe check condition patterns

Author: NotRyken

changelog.md

@@ -1,2 +1 @@
-- Support conditional reconnecting
-- Use YACL in place of Cloth Config
+- Safe check condition patterns

common/src/main/java/dev/terminalmc/autoreconnectrf/mixin/MixinDisconnectedScreens.java

@@ -26,6 +26,7 @@
 import java.util.NoSuchElementException;
 import java.util.concurrent.TimeUnit;
 import java.util.regex.Pattern;
+import java.util.regex.PatternSyntaxException;
 
 import net.minecraft.ChatFormatting;
 import net.minecraft.client.Minecraft;
@@ -64,11 +65,16 @@ private void init(CallbackInfo info) {
             boolean match = false;
 
             for (String condition : Config.get().options.conditionPatterns) {
-                if (Pattern.compile(condition).matcher(reasonStr).find()) {
-                    AutoReconnect.LOG.info("Matched pattern '{}' against reason '{}'",
-                            condition, reasonStr);
-                    match = true;
-                    break;
+                try {
+                    if (Pattern.compile(condition).matcher(reasonStr).find()) {
+                        AutoReconnect.LOG.info("Matched pattern '{}' against reason '{}'",
+                                condition, reasonStr);
+                        match = true;
+                        break;
+                    }
+                } catch (PatternSyntaxException e) {
+                    AutoReconnect.LOG.error(String.format("Invalid pattern: %s\n%s", condition,
+                            e.getMessage()));
                 }
             }
             if (!match && reason.getContents() instanceof TranslatableContents tc) {

gradle.properties

@@ -2,7 +2,7 @@
 # Neo/Forge version ranges: https://maven.apache.org/enforcer/enforcer-rules/versionRanges.html
 
 # Project
-mod_version=0.2.0
+mod_version=0.3.0
 mod_group=dev.terminalmc
 mod_id=autoreconnectrf
 mod_name=AutoReconnect-Reforged