Skip to content

fix: token owner validation for unshield flow#121

Merged
bhflm merged 8 commits intomainfrom
fix/unshield-token-owner
Feb 4, 2026
Merged

fix: token owner validation for unshield flow#121
bhflm merged 8 commits intomainfrom
fix/unshield-token-owner

Conversation

@bhflm
Copy link
Contributor

@bhflm bhflm commented Jan 22, 2026

Fixes owner validation during the unshield process for delegated wallets. Previously, the unshield flow was incorrectly validating token ownership, causing failures when unshielding from wallets using EIP-7702 delegation.

Changes

  • Updated token owner validation logic to properly handle EIP-7702 delegated addresses
  • Fixed commitment event handling in unshield transactions
  • Increased expected railgun transaction length validation

@bhflm bhflm requested review from mesquka and zy0n January 22, 2026 00:21
@bhflm bhflm marked this pull request as draft January 22, 2026 01:40
@bhflm bhflm marked this pull request as ready for review January 23, 2026 10:36
@bhflm bhflm merged commit 452963d into main Feb 4, 2026
4 checks passed
@bhflm bhflm deleted the fix/unshield-token-owner branch February 4, 2026 06:27
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