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

feat: validate predicates using the VM before sending out transaction #1286

Merged
merged 11 commits into from
Mar 19, 2024

Conversation

iqdecay
Copy link
Contributor

@iqdecay iqdecay commented Feb 29, 2024

This PR normally closes (correctly this time I hope) #1195 by adding predicates
validation within the SDK.

  • This means that now the SDK can detect if an invalid predicate transaction is
    about to be sent to the node and prevents it.

segfault-magnet
segfault-magnet previously approved these changes Mar 8, 2024
Copy link
Contributor

@segfault-magnet segfault-magnet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A lot smaller than expected.

Nicely done :)

MujkicA
MujkicA previously approved these changes Mar 10, 2024
Copy link
Contributor

@hal3e hal3e left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Left a nit regarding code duplication.

@hal3e hal3e dismissed stale reviews from segfault-magnet and MujkicA via 9558e0b March 14, 2024 09:25
Br1ght0ne
Br1ght0ne previously approved these changes Mar 15, 2024
@hal3e hal3e dismissed stale reviews from Br1ght0ne and segfault-magnet via de9f931 March 15, 2024 11:01
@hal3e hal3e merged commit df7ba2d into master Mar 19, 2024
41 checks passed
@hal3e hal3e deleted the iqdecay/feat-vm-predicates-validation branch March 19, 2024 15:22
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.

7 participants