Merge pull request #59 from kuzzleio/1.0.3-proposal

# [1.0.3](https://github.com/kuzzleio/sdk-csharp/releases/tag/1.0.3) (2020-09-18)


#### Bug fixes

- [ [#58](#58) ] Fix expiresIn option handling   ([scottinet](https://github.com/scottinet))

---

Author: scottinet

Kuzzle.Tests/API/Controllers/AuthControllerTest.cs

@@ -192,13 +192,18 @@ public async void LoginAsyncTestSuccess(TimeSpan? expiresIn) {
         credentials,
         expiresIn);
 
-      _api.Verify(new JObject {
+      var expectedQuery = new JObject {
         { "controller", "auth" },
         { "action", "login" },
         { "strategy", "foostrategy" },
-        { "expiresIn", expiresIn?.TotalMilliseconds },
-        { "body", credentials }
-      });
+        { "body", credentials },
+      };
+
+      if (expiresIn != null) {
+        expectedQuery["expiresIn"] = $"{Math.Floor((double)expiresIn?.TotalSeconds)}s";
+      }
+
+      _api.Verify(expectedQuery);
 
       Assert.Equal<JObject>(
         expected,
@@ -250,11 +255,16 @@ public async void RefreshTokenTestSuccess(TimeSpan? expiresIn) {
 
       JObject result = await _authController.RefreshTokenAsync(expiresIn);
 
-      _api.Verify(new JObject {
+      var expectedQuery = new JObject {
         { "controller", "auth" },
         { "action", "refreshToken" },
-        { "expiresIn", expiresIn?.TotalMilliseconds }
-      });
+      };
+
+      if (expiresIn != null) {
+        expectedQuery["expiresIn"] = $"{Math.Floor((double)expiresIn?.TotalSeconds)}s";
+      }
+
+      _api.Verify(expectedQuery);
 
       Assert.Equal<JObject>(
         expected,
@@ -338,4 +348,4 @@ public async void ValidateMyCredentialsAsyncTest(bool value) {
       Assert.Equal(value, result);
     }
   }
-}
+}

Kuzzle/API/Controllers/AuthController.cs

@@ -144,13 +144,19 @@ public async Task<JObject> LoginAsync(
       JObject credentials,
       TimeSpan? expiresIn = null
     ) {
-      Response response = await api.QueryAsync(new JObject {
+      var query = new JObject {
         { "controller", "auth" },
         { "action", "login" },
         { "strategy", strategy },
         { "body", credentials },
-        { "expiresIn", expiresIn?.TotalMilliseconds }
-      });
+      };
+
+      if (expiresIn != null) {
+        // workaround to https://github.com/kuzzleio/kuzzle/issues/1784
+        query["expiresIn"] = $"{Math.Floor((double)expiresIn?.TotalSeconds)}s";
+      }
+
+      Response response = await api.QueryAsync(query);
 
       api.AuthenticationToken = (string)response.Result["jwt"];
 
@@ -176,11 +182,17 @@ await api.QueryAsync(new JObject {
     /// Refreshes an authentication token.
     /// </summary>
     public async Task<JObject> RefreshTokenAsync(TimeSpan? expiresIn = null) {
-      Response response = await api.QueryAsync(new JObject {
+      var query = new JObject {
         { "controller", "auth" },
         { "action", "refreshToken" },
-        { "expiresIn", expiresIn?.TotalMilliseconds }
-      });
+      };
+
+      if (expiresIn != null) {
+        // workaround to https://github.com/kuzzleio/kuzzle/issues/1784
+        query["expiresIn"] = $"{Math.Floor((double)expiresIn?.TotalSeconds)}s";
+      }
+
+      Response response = await api.QueryAsync(query);
 
       api.AuthenticationToken = (string)response.Result["jwt"];
 

Kuzzle/Kuzzle.nuspec

@@ -3,7 +3,7 @@
   <metadata>
     <id>kuzzlesdk</id>
     <title>Kuzzle SDK</title>
-    <version>1.0.2</version>
+    <version>1.0.3</version>
     <authors>Kuzzle Team</authors>
     <owners>Kuzzle Team</owners>
     <license type="expression">Apache-2.0</license>