Skip to content

fix: verify charge hashes by credential source#414

Merged
brendanjryan merged 3 commits intowevm:mainfrom
brendanjryan:brendan/session-push-hash
May 7, 2026
Merged

fix: verify charge hashes by credential source#414
brendanjryan merged 3 commits intowevm:mainfrom
brendanjryan:brendan/session-push-hash

Conversation

@brendanjryan
Copy link
Copy Markdown
Collaborator

@brendanjryan brendanjryan commented May 5, 2026

Summary

Charge-only fix for smart-account push payments (for example, Crossmint/ERC-4337):

  • uses Credential.source as the expected Transfer sender for Tempo charge hash credentials
  • keeps the existing receipt.from fallback for clients that do not send source
  • rejects malformed or wrong-chain hash credential sources instead of silently falling back
  • preserves existing challenge-bound MPP attribution memo verification for push hashes

This lets a smart wallet broadcast a transferWithMemo itself and submit a hash credential whose source is the smart wallet DID, even when the transaction's top-level sender is a bundler or EntryPoint path rather than the smart wallet.

@brendanjryan brendanjryan force-pushed the brendan/session-push-hash branch from ea79344 to 66fdac3 Compare May 5, 2026 17:29
@brendanjryan brendanjryan changed the title feat: support push session hashes fix: verify charge hashes by credential source May 5, 2026
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 5, 2026

Open in StackBlitz

npm i https://pkg.pr.new/mppx@414

commit: 21706e3

@brendanjryan brendanjryan force-pushed the brendan/session-push-hash branch from 66fdac3 to 1535430 Compare May 5, 2026 17:32
@brendanjryan brendanjryan marked this pull request as ready for review May 5, 2026 17:33
@brendanjryan brendanjryan merged commit 5059bfa into wevm:main May 7, 2026
8 checks passed
@brendanjryan brendanjryan deleted the brendan/session-push-hash branch May 7, 2026 17:50
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