Merge pull request #112 from joesturge/error-handling

fix: dont park thread when error occurs
joesturge · Sep 19, 2024 · dcb47ac · dcb47ac
2 parents ba03563 + b2ddaa8
commit dcb47ac
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Changed
+
+- Error handling no longer pauses logging thread, just sets unhealthy
+
+### Security 
+
+- Dependabot updates
+
 ## [2.5.1] - 2024-09-03
 
 ### Security 

diff --git a/src/health.rs b/src/health.rs
@@ -51,11 +51,7 @@ pub fn healthy() {
 
 pub fn unhealthy() {
     set(Status::UNHEALTHY);
-    error!(target: "lazymc-docker-proxy::health", "Error detected. Pausing application...");
-    // wait indefinitely
-    loop {
-        std::thread::park();
-    }
+    error!(target: "lazymc-docker-proxy::health", "Application is unhealthy.");
 }
 
 pub fn run() {