apache / apache/arrow-rs-object-store

Crash in http in WASM due to hitting std::time::Instant::now()

Open
#624 1 comment 1 reaction 0 assignees View on GitHub
bug
Dominant language
Rust
Stars
322
Forks
212
Avg merge
5d 2h
Merged PRs (30d)
10

Description

**Describe the bug**
Calling `std::time::Instant::now()` crashes unconditionally on WASM. retry.rs uses tokio::time::sleep() in the retry case, so you may think things are working but see intermittent crashes.

**To Reproduce**
Any http request that is retried in WASM should display the problem.

**Expected behavior**
No crash.

**Additional context**
See for example https://github.com/apache/arrow-rs-object-store/blob/main/src/client/retry.rs#L389

Contributor guide

Open the contributing guide

Research direction

Start by reading src/client/retry.rs around line 389, where the issue identifies the problematic time call in the retry path. Reproduce an HTTP request retry under WASM and trace the timing behavior. Done means retried WASM requests no longer crash.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust, wasm
Domain
networking
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.