Fixes sanitazation of date and time values in path/query parameters

CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.6.1] - 2023-11-02
+
+### Changed
+
+- Fixes sanitization of Date and Time values in query and path parameters
+
 ## [1.6.0] - 2023-10-31
 
 ### Added

Microsoft.Kiota.Abstractions.Tests/RequestInformationTests.cs

@@ -195,7 +195,50 @@ public void SetsPathParametersOfGuidType()
             // Assert
             Assert.Contains($"%24requestId=6d320a89-2d8f-4204-855d-b98a1bc176d4", requestInfo.URI.OriginalString);
         }
+        [Fact]
+        public void SetsPathParametersOfDateType()
+        {
+            // Arrange as the request builders would
+            var requestInfo = new RequestInformation
+            {
+                HttpMethod = Method.GET,
+                UrlTemplate = "http://localhost/users{?%24date}"
+            };
 
+            // Act
+            var date = new Date(2023,10,26);
+            var pathParameters = new Dictionary<string, object>
+            {
+                { "%24date", date }
+            };
+
+            requestInfo.PathParameters = pathParameters;
+
+            // Assert
+            Assert.Contains($"%24date=2023-10-26", requestInfo.URI.OriginalString);
+        }
+        [Fact]
+        public void SetsPathParametersOfTimeType()
+        {
+            // Arrange as the request builders would
+            var requestInfo = new RequestInformation
+            {
+                HttpMethod = Method.GET,
+                UrlTemplate = "http://localhost/users{?%24time}"
+            };
+
+            // Act
+            var time = new Time(6,0,0);
+            var pathParameters = new Dictionary<string, object>
+            {
+                { "%24time", time }
+            };
+
+            requestInfo.PathParameters = pathParameters;
+
+            // Assert
+            Assert.Contains($"%24time=06%3A00%3A00", requestInfo.URI.OriginalString);
+        }
         [Fact]
         public void ThrowsInvalidOperationExceptionWhenBaseUrlNotSet()
         {

src/Microsoft.Kiota.Abstractions.csproj

@@ -14,7 +14,7 @@
     <PackageProjectUrl>https://aka.ms/kiota/docs</PackageProjectUrl>
     <EmbedUntrackedSources>true</EmbedUntrackedSources>
     <Deterministic>true</Deterministic>
-    <VersionPrefix>1.6.0</VersionPrefix>
+    <VersionPrefix>1.6.1</VersionPrefix>
     <VersionSuffix></VersionSuffix>
     <GeneratePackageOnBuild>true</GeneratePackageOnBuild>
     <SignAssembly>false</SignAssembly>

src/RequestInformation.cs

@@ -81,6 +81,8 @@
             DateTimeOffset dateTimeOffset => dateTimeOffset.ToString("o"),// Default to ISO 8601 for datetimeoffsets in the url.
             DateTime dateTime => dateTime.ToString("o"),// Default to ISO 8601 for datetimes in the url.
             Guid guid => guid.ToString("D"),// Default of 32 digits separated by hyphens
+            Date date => date.ToString(), //Default to string format of the custom date object
+            Time time => time.ToString(), //Default to string format of the custom time object
             _ => value,//return object as is as the ToString method is good enough.
         };
 
@@ -197,7 +199,7 @@
         /// Sets the request body to a binary stream.
         /// </summary>
         /// <param name="content">The binary stream to set as a body.</param>
         [Obsolete("Use SetStreamContent and pass the content type instead")]
         public void SetStreamContent(Stream content) => SetStreamContent(content, BinaryContentType);
         /// <summary>
         /// Sets the request body to a binary stream.
@@ -335,7 +337,7 @@
                     writer.WriteNullValue(null);
                     break;
                 default:
                     throw new InvalidOperationException($"error serialization data value with unknown type {item?.GetType()}");
             }
             Headers.TryAdd(ContentTypeHeader, contentType);
             Content = writer.GetSerializedContent();