scriptoriumadmin/fabro
1
0
Fork 0
mirror of https://github.com/fabro-sh/fabro.git synced 2026-08-28 05:27:41 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

fix(cli): raise doctor health check timeout to 1s

Browse source 
250ms was too aggressive: a server that was reachable but slightly slow
to answer /health made `fabro doctor` report a failed health check.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
Christophe de Carvalho 2026-08-04 16:43:42 +01:00
parent e7fc25f62b
commit 8b25001985
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/apps/fabro-cli/src/server_client.rs
View file

@ -443,7 +443,7 @@ mod tests {
let target = ServerTarget::http_url(format!("http://{addr}")).unwrap();
let client = connect_target_api_client_bundle(&target).await.unwrap();
let result = time::timeout(Duration::from_millis(750), client.get_health()).await;
let result = time::timeout(Duration::from_secs(5), client.get_health()).await;
server.abort();
assert!(

2
lib/foundation/fabro-client/src/client.rs
View file

@ -37,7 +37,7 @@ use crate::{AuthEntry, OAuthEntry, StoredSubject, sse};
const DEFAULT_CONTROL_PLANE_REQUEST_TIMEOUT: std::time::Duration =
std::time::Duration::from_secs(30);
const DEFAULT_HEALTH_REQUEST_TIMEOUT: std::time::Duration = std::time::Duration::from_millis(250);
const DEFAULT_HEALTH_REQUEST_TIMEOUT: std::time::Duration = std::time::Duration::from_secs(1);
type TransportFuture = BoxFuture<'static, Result<(fabro_http::HttpClient, String)>>;