Skip to content

Commit 1cbe34b

Browse files
authored
Fix time serialization issues (awslabs#707)
- Update Chrono to fix compilation issues. - Update leap second tests. Signed-off-by: David Calavera <[email protected]>
1 parent d1687e1 commit 1cbe34b

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

lambda-events/Cargo.toml

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,12 +25,15 @@ serde_with = { version = "^3", features = ["json"], optional = true }
2525
serde_json = "^1"
2626
serde_dynamo = { version = "^4.1", optional = true }
2727
bytes = { version = "1", features = ["serde"], optional = true }
28-
chrono = { version = "0.4.23", default-features = false, features = [
28+
chrono = { version = "0.4.31", default-features = false, features = [
2929
"clock",
3030
"serde",
3131
"std",
3232
], optional = true }
33-
query_map = { version = "^0.7", features = ["serde", "url-query"], optional = true }
33+
query_map = { version = "^0.7", features = [
34+
"serde",
35+
"url-query",
36+
], optional = true }
3437
flate2 = { version = "1.0.24", optional = true }
3538

3639
[features]

lambda-events/src/encodings/time.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -279,12 +279,12 @@ mod test {
279279
let encoded = serde_json::to_string(&instance).unwrap();
280280
assert_eq!(encoded, String::from(r#"{"v":"427683600.002"}"#));
281281

282-
// Make sure milliseconds are included.
282+
// Make sure leap seconds are included.
283283
let instance = Test {
284-
v: Utc.ymd(1983, 7, 22).and_hms_nano(1, 0, 0, 1_234_000_000),
284+
v: Utc.ymd(1983, 7, 22).and_hms_nano(23, 59, 59, 1_999_999_999),
285285
};
286286
let encoded = serde_json::to_string(&instance).unwrap();
287-
assert_eq!(encoded, String::from(r#"{"v":"427683601.234"}"#));
287+
assert_eq!(encoded, String::from(r#"{"v":"427766400.999"}"#));
288288
}
289289

290290
#[test]

0 commit comments

Comments
 (0)