Skip to content

fix(tempo): preserve filled transaction validity bounds - #5232

Merged
struong merged 1 commit into
wevm:mainfrom
mattsse:mattsse/fix-tempo-filled-validity
Oct 14, 2026
Merged

struong merged 1 commit into
wevm:mainfrom
mattsse:mattsse/fix-tempo-filled-validity

Conversation

@mattsse

@mattsse mattsse commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

Preserves validBefore and validAfter returned by eth_fillTransaction when preparing Tempo transactions. Sponsored sequential-nonce transactions were copying the fee-payer signature while dropping relay-added expiry bounds, so the signature recovered to the wrong payer and broadcasts failed with insufficient funds.

@vercel

vercel Bot commented Oct 14, 2026

Copy link
Copy Markdown

@mattsse is attempting to deploy a commit to the Wevm Team on Vercel.

A member of the Team first needs to authorize it.

@changeset-bot

changeset-bot Bot commented Oct 14, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 4ae9dde

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
viem Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@pkg-pr-new

pkg-pr-new Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/viem@5232

commit: 4ae9dde

@struong
struong merged commit d302ad0 into wevm:main Oct 14, 2026
28 of 35 checks passed
@github-actions github-actions Bot mentioned this pull request Oct 14, 2026
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.

2 participants