Merge pull request #209 from amirkaws/amirkaws-metrics-timestamp-fix

sliedig · web-flow · commit 135050efec70 · 2023-02-15T06:23:59.000+08:00
fix: Wrong metrics timestamp
diff --git a/libraries/src/AWS.Lambda.Powertools.Metrics/Model/Metadata.cs b/libraries/src/AWS.Lambda.Powertools.Metrics/Model/Metadata.cs
@@ -30,7 +30,6 @@ public class Metadata
     public Metadata()
     {
         CloudWatchMetrics = new List<MetricDirective> {new()};
-        Timestamp = DateTime.Now;
         CustomMetadata = new Dictionary<string, object>();
     }
 
@@ -39,7 +38,7 @@ public Metadata()
     /// </summary>
     /// <value>The timestamp.</value>
     [JsonConverter(typeof(UnixMillisecondDateTimeConverter))]
-    public DateTime Timestamp { get; }
+    public DateTime Timestamp => DateTime.Now;
 
     /// <summary>
     ///     Gets the cloud watch metrics.

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,6 @@ public class Metadata`
`30`	`30`	`public Metadata()`
`31`	`31`	`{`
`32`	`32`	`CloudWatchMetrics = new List<MetricDirective> {new()};`
`33`		`- Timestamp = DateTime.Now;`
`34`	`33`	`CustomMetadata = new Dictionary<string, object>();`
`35`	`34`	`}`
`36`	`35`
`@@ -39,7 +38,7 @@ public Metadata()`
`39`	`38`	`/// </summary>`
`40`	`39`	`/// <value>The timestamp.</value>`
`41`	`40`	`[JsonConverter(typeof(UnixMillisecondDateTimeConverter))]`
`42`		`- public DateTime Timestamp { get; }`
	`41`	`+ public DateTime Timestamp => DateTime.Now;`
`43`	`42`
`44`	`43`	`/// <summary>`
`45`	`44`	`/// Gets the cloud watch metrics.`