Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed Carlos review #411

Open
wants to merge 12 commits into
base: feature/ongoing-v0.3.0
Choose a base branch
from

Conversation

ignasirv
Copy link
Contributor

@ignasirv ignasirv commented Feb 25, 2025

Fixed from Carlos review issues:
https://github.com/orgs/0xPolygonHermez/projects/19/views/1?pane=issue&itemId=99101469&issue=0xPolygonHermez%7Cinternal-audit%7C93
https://github.com/orgs/0xPolygonHermez/projects/19/views/1?pane=issue&itemId=99109405&issue=0xPolygonHermez%7Cinternal-audit%7C94
https://github.com/orgs/0xPolygonHermez/projects/19/views/1?pane=issue&itemId=99120921&issue=0xPolygonHermez%7Cinternal-audit%7C95

  • Update aggchainECDSA to look like FEPV3
  • Dont user arrays at vkeys and selectors of initialize
  • Force 2 bytes for selector
  • Clean errors
  • Clean pending vkemanager after transfer role
  • Add non reentrancy to claimAsset at bridge
  • Replace createNewRollup function to attachAggchainToAL

@ignasirv ignasirv force-pushed the feature/carlos-review-1 branch from bd809aa to 255af03 Compare February 26, 2025 08:20
@ignasirv ignasirv force-pushed the feature/ongoing-v0.3.0 branch from 3659d96 to e505edf Compare February 26, 2025 10:19
@ignasirv ignasirv force-pushed the feature/carlos-review-1 branch 2 times, most recently from 5e6c1c6 to 088af5b Compare February 26, 2025 14:52
@ignasirv ignasirv force-pushed the feature/carlos-review-1 branch from 9e23741 to 9c60808 Compare February 27, 2025 16:14
@ignasirv ignasirv force-pushed the feature/carlos-review-1 branch 3 times, most recently from 5539f7d to 588679f Compare February 28, 2025 11:59
@ignasirv ignasirv force-pushed the feature/carlos-review-1 branch from 588679f to 1f0b3d6 Compare February 28, 2025 12:06
@ignasirv ignasirv force-pushed the feature/carlos-review-1 branch from 3dc8eb3 to 16b8904 Compare February 28, 2025 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants