Always set OnlineStatus::Offline if dir fails

We don't really care about the exact reason, and the codes can't be trusted
anyway (some brokers return 8).

Author: syyyr

Cargo.lock

@@ -3,7 +3,7 @@ name = "historyprovider"
 description = "historyprovider-rs"
 license = "MIT"
 repository = "https://github.com/silicon-heaven/historyprovider-rs"
-version = "2.3.7"
+version = "2.4.0"
 edition = "2024"
 
 [[bin]]

Cargo.toml

@@ -300,10 +300,7 @@ fn online_status_worker(
                         .await;
                     if dir_result.is_ok() {
                         set_online_status(&site, SiteOnlineStatus::Online, &client_commands, &app_state).await;
-                    } else if let Err(err) = dir_result
-                        && let CallRpcMethodErrorKind::RpcError(RpcError { code, .. }) = err.error()
-                        && (*code == RpcErrorCode::MethodCallTimeout.into() || *code == RpcErrorCode::MethodNotFound.into())
-                    {
+                    } else if let Err(err) = dir_result && matches!(err.error(), CallRpcMethodErrorKind::RpcError(RpcError { .. })) {
                         set_online_status(&site, SiteOnlineStatus::Offline, &client_commands, &app_state).await;
                     }
                 }