Skip to content

Centralize timeout budget handling#621

Merged
ryanfowler merged 1 commit into
mainfrom
centralize-timeout-handling
May 27, 2026
Merged

Centralize timeout budget handling#621
ryanfowler merged 1 commit into
mainfrom
centralize-timeout-handling

Conversation

@ryanfowler
Copy link
Copy Markdown
Owner

Summary

  • Move shared timeout parsing, Go-style duration formatting, remaining-budget calculation, and timeout error formatting into src/duration.rs
  • Reuse TimeoutBudget across HTTP client setup, WebSocket dialing, DNS inspection, and TLS inspection instead of duplicating elapsed-time logic
  • Update related duration validation call sites and document the new ownership in AGENTS.md

Testing

  • cargo fmt
  • cargo clippy --locked --all-targets --all-features -- -D warnings
  • cargo test --all-features

@ryanfowler ryanfowler force-pushed the centralize-timeout-handling branch from a5b24df to 9d19411 Compare May 27, 2026 06:55
@ryanfowler ryanfowler merged commit 1e74eee into main May 27, 2026
4 checks passed
@ryanfowler ryanfowler deleted the centralize-timeout-handling branch May 27, 2026 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant