Skip to content

Fix duration overflow crashes and bound retry attempts#590

Merged
ryanfowler merged 1 commit into
mainfrom
fix-large-duration-crashes
May 26, 2026
Merged

Fix duration overflow crashes and bound retry attempts#590
ryanfowler merged 1 commit into
mainfrom
fix-large-duration-crashes

Conversation

@ryanfowler
Copy link
Copy Markdown
Owner

Summary

  • Reject oversized duration values before converting to Duration, using the same maximum bound across CLI and config parsing.
  • Validate --retry with checked addition so the total attempt count cannot overflow.
  • Add unit coverage for oversized durations and retry overflow handling.

Testing

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

@ryanfowler ryanfowler force-pushed the fix-large-duration-crashes branch from ec92fd4 to c774b82 Compare May 26, 2026 02:35
@ryanfowler ryanfowler enabled auto-merge May 26, 2026 02:37
@ryanfowler ryanfowler merged commit 1dd4627 into main May 26, 2026
4 checks passed
@ryanfowler ryanfowler deleted the fix-large-duration-crashes branch May 26, 2026 02:39
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