Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add Exact Calldata Enforcer #51

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.

Sign up for GitHub

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

Jump to bottom
Merged
hanzel98 merged 1 commit into from
Mar 6, 2025
Merged

Add Exact Calldata Enforcer #51

hanzel98 merged 1 commit into from
Mar 6, 2025

Conversation

@hanzel98
Copy link
Contributor

@hanzel98 hanzel98 commented Feb 25, 2025

What?

  • Implemented a new Exact calldata enforcer that can be used to compare and validate an exact match between the terms calldata specified by the delegator match the execution calldata specified by the redeemer.

Why?

  • This enforcer would be useful to match an empty calldata, or any calldata that is expected to be an exact match.

How?

  • It compares the terms calldata against the execution calldata.

@hanzel98 hanzel98 requested a review from a team as a code owner February 25, 2025 00:40
@hanzel98 hanzel98 marked this pull request as draft February 25, 2025 00:40
@hanzel98 hanzel98 self-assigned this Feb 25, 2025
pegahcarter
pegahcarter reviewed Feb 28, 2025
View reviewed changes
@McOso McOso mentioned this pull request Mar 5, 2025
Merged
@hanzel98 hanzel98 marked this pull request as ready for review March 6, 2025 21:53
@hanzel98 hanzel98 merged commit db11bf7 into main Mar 6, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@McOso McOso McOso approved these changes

+1 more reviewer

@pegahcarter pegahcarter pegahcarter left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@hanzel98 hanzel98

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@hanzel98 @pegahcarter @McOso