Use the browser-only js-sys workaround only when actually targeting a…

… browser (#1005) (#1008) wasi runtimes do provide a SystemTime implementation, see https://github.com/bytecodealliance/wasmtime/blob/main/crates/wasi-common/src/clocks.rs
open-telemetry · Mar 29, 2023 · 3597479 · 3597479
1 parent 3b273fa
commit 3597479
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/opentelemetry-api/src/lib.rs b/opentelemetry-api/src/lib.rs
@@ -73,13 +73,16 @@ pub mod time {
     use std::time::SystemTime;
 
     #[doc(hidden)]
-    #[cfg(not(target_arch = "wasm32"))]
+    #[cfg(any(
+        not(target_arch = "wasm32"),
+        all(target_arch = "wasm32", target_os = "wasi")
+    ))]
     pub fn now() -> SystemTime {
         SystemTime::now()
     }
 
     #[doc(hidden)]
-    #[cfg(target_arch = "wasm32")]
+    #[cfg(all(target_arch = "wasm32", not(target_os = "wasi")))]
     pub fn now() -> SystemTime {
         SystemTime::UNIX_EPOCH + std::time::Duration::from_millis(js_sys::Date::now() as u64)
     }