fix(core): http api connect_timeout deserialization, closes #4004 (#4006)

lucasfernog · web-flow · commit f3c5ca89e79d · 2022-04-29T21:26:45.000-03:00
diff --git a/.changes/fix-api-timeout-type.md b/.changes/fix-api-timeout-type.md
@@ -0,0 +1,5 @@
+---
+"api": patch
+---
+
+Fixes the type of `http > connectTimeout`.
diff --git a/.changes/http-timeout-serde-fix.md b/.changes/http-timeout-serde-fix.md
@@ -0,0 +1,5 @@
+---
+"tauri": patch
+---
+
+Deserialize numeric values (seconds) in the http API `ClientBuilder.connect_timeout` and `HttpRequestBuilder.timeout` fields.
diff --git a/core/tauri/src/api/http.rs b/core/tauri/src/api/http.rs
@@ -21,13 +21,34 @@ pub use attohttpc::header;
 
 use header::{HeaderName, HeaderValue};
 
+#[derive(Deserialize)]
+#[serde(untagged)]
+enum SerdeDuration {
+  Seconds(u64),
+  Duration(Duration),
+}
+
+fn deserialize_duration<'de, D: Deserializer<'de>>(
+  deserializer: D,
+) -> Result<Option<Duration>, D::Error> {
+  if let Some(duration) = Option::<SerdeDuration>::deserialize(deserializer)? {
+    Ok(Some(match duration {
+      SerdeDuration::Seconds(s) => Duration::from_secs(s),
+      SerdeDuration::Duration(d) => d,
+    }))
+  } else {
+    Ok(None)
+  }
+}
+
 /// The builder of [`Client`].
 #[derive(Debug, Clone, Default, Deserialize)]
 #[serde(rename_all = "camelCase")]
 pub struct ClientBuilder {
   /// Max number of redirections to follow.
   pub max_redirections: Option<usize>,
   /// Connect timeout for the request.
+  #[serde(deserialize_with = "deserialize_duration")]
   pub connect_timeout: Option<Duration>,
 }
 
@@ -448,6 +469,7 @@ pub struct HttpRequestBuilder {
   /// The request body
   pub body: Option<Body>,
   /// Timeout for the whole request
+  #[serde(deserialize_with = "deserialize_duration")]
   pub timeout: Option<Duration>,
   /// The response type (defaults to Json)
   pub response_type: Option<ResponseType>,
diff --git a/tooling/api/src/http.ts b/tooling/api/src/http.ts
@@ -45,9 +45,14 @@
 
 import { invokeTauriCommand } from './helpers/tauri'
 
+interface Duration {
+  secs: number
+  nanos: number
+}
+
 interface ClientOptions {
   maxRedirections: number
-  connectTimeout: number
+  connectTimeout: number | Duration
 }
 
 enum ResponseType {
@@ -177,7 +182,7 @@ interface HttpOptions {
   headers?: Record<string, any>
   query?: Record<string, any>
   body?: Body
-  timeout?: number
+  timeout?: number | Duration
   responseType?: ResponseType
 }
 
@@ -417,6 +422,7 @@ async function fetch<T>(
 }
 
 export type {
+  Duration,
   ClientOptions,
   Part,
   HttpVerb,

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +"api": patch
 +---
++
 +Fixes the type of `http > connectTimeout`.