From 37e749b8a57dcad813e8084ddc7a773b1dd1b730 Mon Sep 17 00:00:00 2001 From: jgjesdal Date: Thu, 17 Sep 2026 09:41:36 +0200 Subject: [PATCH] fix: binary ingest builds against ResponseError::content_type #123 was branched before #128 gave ResponseError a content_type field and merged after it, so main stopped compiling at the three places the binary path builds one. All three are raised by the SDK with no server response behind them (a 204 that fails to deserialize, a value refused locally, a series /byids cannot find), so each carries None. Co-Authored-By: Claude Opus 5 (1M context) Signed-off-by: jgjesdal --- src/generic.rs | 1 + src/timeseries/binary.rs | 2 ++ 2 files changed, 3 insertions(+) diff --git a/src/generic.rs b/src/generic.rs index 56ed379..feddc72 100644 --- a/src/generic.rs +++ b/src/generic.rs @@ -865,6 +865,7 @@ pub trait ApiServiceProvider { ResponseError { status: response.status(), message: err.to_string(), + content_type: None, } }); } diff --git a/src/timeseries/binary.rs b/src/timeseries/binary.rs index 0ebea83..fee9834 100644 --- a/src/timeseries/binary.rs +++ b/src/timeseries/binary.rs @@ -737,6 +737,7 @@ fn unprocessable(message: String) -> ResponseError { ResponseError { status: StatusCode::UNPROCESSABLE_ENTITY, message, + content_type: None, } } @@ -931,6 +932,7 @@ impl TimeSeriesService { return Err(ResponseError { status: StatusCode::NOT_FOUND, message: format!("Could not find following timeseries: {}", missing.join(", ")), + content_type: None, }); } Ok(resolved)