refactor: drop time crate in favor of std::time::SystemTime (#2059)

* refactor: drop time crate in favor of std::time::SystemTime

* use as_secs_f64

* don't panic, but return None

Author: mxinden

neqo-common/Cargo.toml

@@ -22,7 +22,6 @@ env_logger = { version = "0.10", default-features = false }
 hex = { version = "0.4", default-features = false, features = ["alloc"], optional = true }
 log = { workspace = true }
 qlog = { workspace = true }
-time = { version = "0.3", default-features = false, features = ["formatting"] }
 
 [dev-dependencies]
 test-fixture = { path = "../test-fixture" }

neqo-common/src/qlog.rs

@@ -11,6 +11,7 @@ use std::{
     io::BufWriter,
     path::PathBuf,
     rc::Rc,
+    time::SystemTime,
 };
 
 use qlog::{
@@ -172,13 +173,10 @@ pub fn new_trace(role: Role) -> qlog::TraceSeq {
         common_fields: Some(CommonFields {
             group_id: None,
             protocol_type: None,
-            reference_time: {
-                // It is better to allow this than deal with a conversion from i64 to f64.
-                // We can't do the obvious two-step conversion with f64::from(i32::try_from(...)),
-                // because that overflows earlier than is ideal.  This should be fine for a while.
-                #[allow(clippy::cast_precision_loss)]
-                Some((time::OffsetDateTime::now_utc().unix_timestamp() * 1000) as f64)
-            },
+            reference_time: SystemTime::now()
+                .duration_since(SystemTime::UNIX_EPOCH)
+                .map(|d| d.as_secs_f64() * 1_000.0)
+                .ok(),
             time_format: Some("relative".to_string()),
         }),
     }