Bump RestSharp from 108.0.1 to 112.0.0 in /src/GregClient (#110)

dependabot[bot] · QilongTang · web-flow · commit 64c476549eb6 · 2024-09-20T11:24:20.000-04:00
* Bump RestSharp from 108.0.1 to 112.0.0 in /src/GregClient Bumps [RestSharp](https://github.com/restsharp/RestSharp) from 108.0.1 to 112.0.0. - [Release notes](https://github.com/restsharp/RestSharp/releases) - [Commits](restsharp/RestSharp@108.0.1...112.0.0) --- updated-dependencies: - dependency-name: RestSharp dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> * Update TimeOut assignment --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Aaron (Qilong) <aaron.tang@autodesk.com>
diff --git a/src/GregClient/GregClient.cs b/src/GregClient/GregClient.cs
@@ -54,12 +54,12 @@ private RestResponse ExecuteInternal(Request m)
                 if (timeoutSetting != null)
                 {
                     // Timeout App Settings is in seconds.
-                    // RestRequest.Timeout is in milliseconds.
+                    // RestRequest.Timeout is now timespan object for clarity from RestSharp 112.0.0
                     var userVal = Convert.ToInt32(timeoutSetting);
                     // Sanity check.
                     if (userVal >= 0 && userVal < 86400/*24 hours*/)
                     {
-                        req.Timeout = Convert.ToInt32(timeoutSetting) * 1000;// get milliseconds
+                        req.Timeout = TimeSpan.FromSeconds(userVal);
                     }
                 }
             }
diff --git a/src/GregClient/GregClient.csproj b/src/GregClient/GregClient.csproj
@@ -26,7 +26,7 @@
   </PropertyGroup>
 
   <ItemGroup Label="Common package dependencies">
-    <PackageReference Include="RestSharp" Version="108.0.1" />
+    <PackageReference Include="RestSharp" Version="112.0.0" />
   </ItemGroup>
   <ItemGroup>
     <None Include="..\..\CHANGELOG.md">
diff --git a/src/GregClient/Requests/JSONRequest.cs b/src/GregClient/Requests/JSONRequest.cs
@@ -14,7 +14,7 @@ public abstract class JsonRequest : Request
         // Ex: Timeout property
         internal void InitReqParams(ref RestRequest request)
         {
-            request.Timeout = Timeout;
+            request.Timeout = TimeSpan.FromMilliseconds(Timeout);
         }
 
         internal override void Build(ref RestRequest request)

Original file line number	Diff line number	Diff line change
`@@ -54,12 +54,12 @@ private RestResponse ExecuteInternal(Request m)`
`54`	`54`	`if (timeoutSetting != null)`
`55`	`55`	`{`
`56`	`56`	`// Timeout App Settings is in seconds.`
`57`		`- // RestRequest.Timeout is in milliseconds.`
	`57`	`+ // RestRequest.Timeout is now timespan object for clarity from RestSharp 112.0.0`
`58`	`58`	`var userVal = Convert.ToInt32(timeoutSetting);`
`59`	`59`	`// Sanity check.`
`60`	`60`	`if (userVal >= 0 && userVal < 86400/24 hours/)`
`61`	`61`	`{`
`62`		`- req.Timeout = Convert.ToInt32(timeoutSetting) * 1000;// get milliseconds`
	`62`	`+ req.Timeout = TimeSpan.FromSeconds(userVal);`
`63`	`63`	`}`
`64`	`64`	`}`
`65`	`65`	`}`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@ public abstract class JsonRequest : Request`
`14`	`14`	`// Ex: Timeout property`
`15`	`15`	`internal void InitReqParams(ref RestRequest request)`
`16`	`16`	`{`
`17`		`- request.Timeout = Timeout;`
	`17`	`+ request.Timeout = TimeSpan.FromMilliseconds(Timeout);`
`18`	`18`	`}`
`19`	`19`
`20`	`20`	`internal override void Build(ref RestRequest request)`