Adding lower default timeout for requests

DrEsteban · Nov 24, 2024 · f142d9d · f142d9d
1 parent ccc35b1
commit f142d9d
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 0 deletions.
diff --git a/src/Models/EnphaseConfiguration.cs b/src/Models/EnphaseConfiguration.cs
@@ -8,4 +8,9 @@ public class EnphaseConfiguration
 
     [RequiredIf(nameof(Enabled), true)]
     public string Host { get; set; } = string.Empty;
+
+    /// <summary>
+    /// Timeout for requests to the gateway.
+    /// </summary>
+    public int RequestTimeoutSeconds { get; set; } = 10;
 }
diff --git a/src/Models/TeslaConfiguration.cs b/src/Models/TeslaConfiguration.cs
@@ -38,6 +38,11 @@ public class TeslaConfiguration
     [RequiredIf(nameof(Enabled), true)]
     public string RequestHost { get; set; } = "powerwall";
 
+    /// <summary>
+    /// Timeout for requests to the Tesla Gateway.
+    /// </summary>
+    public int RequestTimeoutSeconds { get; set; } = 10;
+
     public TeslaLoginRequest GetTeslaLoginRequest()
         => new()
         {

diff --git a/src/Program.cs b/src/Program.cs
@@ -134,6 +134,7 @@
             client.BaseAddress = new Uri($"https://{teslaConfig.Host}");
             // The Tesla Gateway only accepts a certain set of Host header values
             client.DefaultRequestHeaders.Host = teslaConfig.RequestHost;
+            client.Timeout = TimeSpan.FromSeconds(teslaConfig.RequestTimeoutSeconds);
         })
         .ConfigurePrimaryHttpMessageHandler(_ =>
         {
@@ -160,6 +161,7 @@
         {
             var enphaseConfig = c.GetRequiredService<IOptions<EnphaseConfiguration>>().Value;
             client.BaseAddress = new Uri($"http://{enphaseConfig.Host}");
+            client.Timeout = TimeSpan.FromSeconds(enphaseConfig.RequestTimeoutSeconds);
         })
         .UseHttpClientMetrics()
         .RemoveAllLoggers()