Skip to content

fix: harden idle timeout parsing#1398

Open
hiSandog wants to merge 1 commit into
vercel-labs:mainfrom
hiSandog:fix/small-cleanup-20260529
Open

fix: harden idle timeout parsing#1398
hiSandog wants to merge 1 commit into
vercel-labs:mainfrom
hiSandog:fix/small-cleanup-20260529

Conversation

@hiSandog
Copy link
Copy Markdown

Summary

  • avoid byte-splitting the idle timeout suffix so non-ASCII units return an error instead of panicking
  • use checked multiplication when converting s/m/h values to milliseconds
  • add parser regression coverage for Unicode units and overflow

Validation

  • git diff --check
  • cargo test parse_idle_timeout --manifest-path cli/Cargo.toml (not run: cargo is not installed in this local environment)

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 29, 2026

@hiSandog is attempting to deploy a commit to the Vercel Labs Team on Vercel.

A member of the Team first needs to authorize it.

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