Fix TNT-40917 Zero numeric values not included in serialized Delivery request (#18)

* Fix TNT-40917 Zero numeric values not included in serialized Delivery request

* Fix security vulnerabilities in dev dependencies

Author: XDex

Source/Adobe.Target.Delivery/Adobe.Target.Delivery.csproj

@@ -16,7 +16,7 @@
         <PackageReference Include="Newtonsoft.Json" Version="12.0.3" />
         <PackageReference Include="RestSharp" Version="106.11.4" />
         <PackageReference Include="Polly" Version="7.2.0" />
-            <PackageReference Include="System.ComponentModel.Annotations" Version="4.7.0" />
+        <PackageReference Include="System.ComponentModel.Annotations" Version="4.7.0" />
     </ItemGroup>
 
 </Project>

Source/Adobe.Target.Delivery/Client/ClientUtils.cs

@@ -56,6 +56,21 @@ public static Multimap<string, string> ParameterToMultiMap(string collectionForm
                     parameters.Add(name, ParameterToString(item));
                 }
             }
+            else if (value is IDictionary dictionary)
+            {
+                if(collectionFormat == "deepObject") {
+                    foreach (DictionaryEntry entry in dictionary)
+                    {
+                        parameters.Add(name + "[" + entry.Key + "]", ParameterToString(entry.Value));
+                    }
+                }
+                else {
+                    foreach (DictionaryEntry entry in dictionary)
+                    {
+                        parameters.Add(entry.Key.ToString(), ParameterToString(entry.Value));
+                    }
+                }
+            }
             else
             {
                 parameters.Add(name, ParameterToString(value));

Source/Adobe.Target.Delivery/Model/AnalyticsRequest.cs

@@ -34,6 +34,7 @@ namespace Adobe.Target.Delivery.Model
     [DataContract(Name = "AnalyticsRequest")]
     public partial class AnalyticsRequest : IEquatable<AnalyticsRequest>, IValidatableObject
     {
+
         /// <summary>
         /// Gets or Sets Logging
         /// </summary>

Source/Adobe.Target.Delivery/Model/AudienceManager.cs

@@ -39,7 +39,7 @@ public partial class AudienceManager : IEquatable<AudienceManager>, IValidatable
         /// </summary>
         /// <param name="locationHint">DCS location hint. Used to determine which AAM DCS Endpoint to hit in order to retrieve the profile. .</param>
         /// <param name="blob">AAM Blob. Used to send additional data to AAM. Validation   * Cannot be blank. .</param>
-        public AudienceManager(int locationHint = default(int), string blob = default(string))
+        public AudienceManager(int? locationHint = default(int?), string blob = default(string))
         {
             this.LocationHint = locationHint;
             this.Blob = blob;
@@ -50,7 +50,7 @@ public partial class AudienceManager : IEquatable<AudienceManager>, IValidatable
         /// </summary>
         /// <value>DCS location hint. Used to determine which AAM DCS Endpoint to hit in order to retrieve the profile. </value>
         [DataMember(Name = "locationHint", EmitDefaultValue = false)]
-        public int LocationHint { get; set; }
+        public int? LocationHint { get; set; }
 
         /// <summary>
         /// AAM Blob. Used to send additional data to AAM. Validation   * Cannot be blank. 

Source/Adobe.Target.Delivery/Model/AuthenticatedState.cs

@@ -31,9 +31,7 @@ namespace Adobe.Target.Delivery.Model
     /// <summary>
     /// Defines AuthenticatedState
     /// </summary>
-    
     [JsonConverter(typeof(StringEnumConverter))]
-    
     public enum AuthenticatedState
     {
         /// <summary>

Source/Adobe.Target.Delivery/Model/ChannelType.cs

@@ -32,9 +32,7 @@ namespace Adobe.Target.Delivery.Model
     /// Specifies the channel for the request. Only activities with the matching channel will be evaluated. 
     /// </summary>
     /// <value>Specifies the channel for the request. Only activities with the matching channel will be evaluated. </value>
-    
     [JsonConverter(typeof(StringEnumConverter))]
-    
     public enum ChannelType
     {
         /// <summary>

Source/Adobe.Target.Delivery/Model/Context.cs

@@ -34,6 +34,7 @@ namespace Adobe.Target.Delivery.Model
     [DataContract(Name = "Context")]
     public partial class Context : IEquatable<Context>, IValidatableObject
     {
+
         /// <summary>
         /// Gets or Sets Channel
         /// </summary>
@@ -58,7 +59,7 @@ protected Context() { }
         /// <param name="timeOffsetInMinutes">Specifies minutes from UTC for specific client.</param>
         /// <param name="userAgent">User-Agent should be sent only via this property. HTTP header User-Agent is ignored..</param>
         /// <param name="beacon">In case beacon &#x3D; true is provided in the request, the server will return a 204 No Content response with no response body.  (default to false).</param>
-        public Context(ChannelType channel = default(ChannelType), MobilePlatform mobilePlatform = default(MobilePlatform), Application application = default(Application), Screen screen = default(Screen), Window window = default(Window), Browser browser = default(Browser), Address address = default(Address), Geo geo = default(Geo), double timeOffsetInMinutes = default(double), string userAgent = default(string), bool beacon = false)
+        public Context(ChannelType channel = default(ChannelType), MobilePlatform mobilePlatform = default(MobilePlatform), Application application = default(Application), Screen screen = default(Screen), Window window = default(Window), Browser browser = default(Browser), Address address = default(Address), Geo geo = default(Geo), double? timeOffsetInMinutes = default(double?), string userAgent = default(string), bool beacon = false)
         {
             this.Channel = channel;
             this.MobilePlatform = mobilePlatform;
@@ -120,7 +121,7 @@ protected Context() { }
         /// </summary>
         /// <value>Specifies minutes from UTC for specific client</value>
         [DataMember(Name = "timeOffsetInMinutes", EmitDefaultValue = false)]
-        public double TimeOffsetInMinutes { get; set; }
+        public double? TimeOffsetInMinutes { get; set; }
 
         /// <summary>
         /// User-Agent should be sent only via this property. HTTP header User-Agent is ignored.
@@ -133,7 +134,7 @@ protected Context() { }
         /// In case beacon &#x3D; true is provided in the request, the server will return a 204 No Content response with no response body. 
         /// </summary>
         /// <value>In case beacon &#x3D; true is provided in the request, the server will return a 204 No Content response with no response body. </value>
-        [DataMember(Name = "beacon", EmitDefaultValue = false)]
+        [DataMember(Name = "beacon", EmitDefaultValue = true)]
         public bool Beacon { get; set; }
 
         /// <summary>

Source/Adobe.Target.Delivery/Model/CustomerId.cs

@@ -34,6 +34,7 @@ namespace Adobe.Target.Delivery.Model
     [DataContract(Name = "CustomerId")]
     public partial class CustomerId : IEquatable<CustomerId>, IValidatableObject
     {
+
         /// <summary>
         /// Gets or Sets AuthenticatedState
         /// </summary>

Source/Adobe.Target.Delivery/Model/DecisioningMethod.cs

@@ -31,9 +31,7 @@ namespace Adobe.Target.Delivery.Model
     /// <summary>
     /// Defines DecisioningMethod
     /// </summary>
-    
     [JsonConverter(typeof(StringEnumConverter))]
-    
     public enum DecisioningMethod
     {
         /// <summary>

Source/Adobe.Target.Delivery/Model/DeliveryRequest.cs

@@ -56,7 +56,7 @@ protected DeliveryRequest() { }
         /// <param name="notifications">Notifications for the displayed content, clicked selectors, and/or visited views or mboxes..</param>
         /// <param name="qaMode">qaMode.</param>
         /// <param name="preview">preview.</param>
-        public DeliveryRequest(string requestId = default(string), string impressionId = default(string), VisitorId id = default(VisitorId), long environmentId = default(long), Property property = default(Property), Trace trace = default(Trace), Context context = default(Context), ExperienceCloud experienceCloud = default(ExperienceCloud), ExecuteRequest execute = default(ExecuteRequest), PrefetchRequest prefetch = default(PrefetchRequest), Telemetry telemetry = default(Telemetry), List<Notification> notifications = default(List<Notification>), QAMode qaMode = default(QAMode), Preview preview = default(Preview))
+        public DeliveryRequest(string requestId = default(string), string impressionId = default(string), VisitorId id = default(VisitorId), long? environmentId = default(long?), Property property = default(Property), Trace trace = default(Trace), Context context = default(Context), ExperienceCloud experienceCloud = default(ExperienceCloud), ExecuteRequest execute = default(ExecuteRequest), PrefetchRequest prefetch = default(PrefetchRequest), Telemetry telemetry = default(Telemetry), List<Notification> notifications = default(List<Notification>), QAMode qaMode = default(QAMode), Preview preview = default(Preview))
         {
             // to ensure "context" is required (not null)
             this.Context = context ?? throw new ArgumentNullException("context is a required property for DeliveryRequest and cannot be null");
@@ -100,7 +100,7 @@ protected DeliveryRequest() { }
         /// </summary>
         /// <value>Valid client environment id. If not specified host will be determined base on the provided host.</value>
         [DataMember(Name = "environmentId", EmitDefaultValue = false)]
-        public long EnvironmentId { get; set; }
+        public long? EnvironmentId { get; set; }
 
         /// <summary>
         /// Gets or Sets Property