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

Fix JIT reward #279

Open

Philogy wants to merge 3 commits into main from feat/fix-jit-reward

Contributor

Philogy commented Nov 27, 2024

adds a reward_checksum: u80 field to the MultiTick reward variant to ensure that any JIT liquidity change reverts the bundle
adds the security requirement that some non-negligible amount is swapped in the end current tick, because even with the new mechanism if there is no swapping going through the final tick a reward to it could be JITed without consequences.
how to compute the checksum is documented under Reward Update Internals

Philogy added 3 commits

November 27, 2024 17:59


          🔒 Fix jit reward

b11fc93


          🥢 Fix poolgate

08adae6


          📝 Document checksum calculation

cee97d3

Philogy requested a review from gravaton

November 27, 2024 11:29

Will-Smith11 reviewed

View reviewed changes

contracts/docs/payload-types.md Show resolved Hide resolved

Philogy removed the request for review from gravaton

December 6, 2024 05:22

Collaborator

Will-Smith11 commented Jan 16, 2025

Circling back here. We need this fix for launch. Lets get this merged in and going once we get the current testnet contracts working.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet