Merge pull request #155 from Flagsmith/fix/request-timeout

fix: Fix default request timeout being 0

Author: rolodato

Flagsmith.Client.Test/FlagsmithTest.cs

@@ -833,5 +833,11 @@ public void TestRequestTimeoutInterpretsSecondsCorrectly()
             };
             Assert.Equal(100, config.RequestTimeout);
         }
+
+        [Fact]
+        public void TestRequestTimeoutHasReasonableDefault()
+        {
+            Assert.True(new FlagsmithConfiguration().RequestTimeout > 1);
+        }
     }
 }

Flagsmith.FlagsmithClient/Flagsmith.FlagsmithClient.csproj

@@ -1,13 +1,13 @@
 <Project Sdk="Microsoft.NET.Sdk">
-  
+
     <PropertyGroup>
         <TargetFramework>netstandard2.0</TargetFramework>
         <LangVersion>8</LangVersion>
         <DefaultItemExcludes>$(DefaultItemExcludes);example/**;</DefaultItemExcludes>
         <CopyLocalLockFileAssemblies>true</CopyLocalLockFileAssemblies>
         <PackageId>Flagsmith</PackageId>
         <Title>Flagsmith</Title>
-        <Version>8.0.0</Version>
+        <Version>8.0.1</Version>
         <Authors>flagsmith</Authors>
         <Company>Flagsmith</Company>
         <PackageDescription>Client SDK for Flagsmith. Ship features with confidence using feature flags and remote config. Host yourself or use our hosted version at https://flagsmith.com/</PackageDescription>

Flagsmith.FlagsmithClient/FlagsmithConfiguration.cs

@@ -9,7 +9,7 @@ namespace Flagsmith
     public class FlagsmithConfiguration
     {
         private static readonly Uri DefaultApiUri = new Uri("https://edge.api.flagsmith.com/api/v1/");
-        private TimeSpan _timeout;
+        private TimeSpan _timeout = TimeSpan.FromSeconds(5);
 
         /// <summary>
         /// Versioned base Flagsmith API URI to use for all requests. Defaults to
@@ -54,7 +54,7 @@ public class FlagsmithConfiguration
         public Double? RequestTimeout
         {
             get => _timeout.TotalSeconds;
-            set => _timeout = TimeSpan.FromSeconds(value ?? 100);
+            set => _timeout = TimeSpan.FromSeconds(value ?? 5);
         }
 
         /// <summary>