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

refactor(protocol): improve block liveness bond handling #17010

Merged
merged 18 commits into from
May 7, 2024

Conversation

dantaik
Copy link
Contributor

@dantaik dantaik commented May 6, 2024

This PR is based on some feedback from OZ.

In this PR, the liveness bond handling logic is unified into one single section of code. When the block is proven for the first time, the liveness bond value associated with the block will be set to 0, regardless if it is returned to the assigned prover.

@dantaik dantaik marked this pull request as ready for review May 6, 2024 09:27
@dantaik dantaik requested review from Brechtpd and adaki2004 May 6, 2024 09:27
@dantaik dantaik marked this pull request as draft May 6, 2024 09:31
@dantaik dantaik changed the title refactor(protocol): improve lib proving logics refactor(protocol): improve lib proving logics (to be tested) May 6, 2024
@dantaik dantaik changed the title refactor(protocol): improve lib proving logics (to be tested) refactor(protocol): improve lib proving logics -1 (to be tested) May 6, 2024
Copy link

openzeppelin-code bot commented May 6, 2024

refactor(protocol): improve liveness bond return logics

Generated at commit: 4488874731f1d37edf8aa6305d6b1535e5a42ed1

🚨 Report Summary

Severity Level Results
Contracts Critical
High
Medium
Low
Note
Total
2
2
0
5
41
50
Dependencies Critical
High
Medium
Low
Note
Total
0
0
0
0
0
0

For more details view the full report in OpenZeppelin Code Inspector

@dantaik dantaik changed the title refactor(protocol): improve lib proving logics -1 (to be tested) refactor(protocol): improve lib proving logics - 1 (to be tested) May 6, 2024
@dantaik dantaik removed request for Brechtpd and adaki2004 May 6, 2024 14:41
@dantaik dantaik changed the title refactor(protocol): improve lib proving logics - 1 (to be tested) refactor(protocol): improve proving logics May 7, 2024
@dantaik dantaik changed the title refactor(protocol): improve proving logics refactor(protocol): improve liveness bond check/return logic into one section of code May 7, 2024
@dantaik dantaik marked this pull request as ready for review May 7, 2024 02:03
@dantaik dantaik requested review from adaki2004 and Brechtpd May 7, 2024 02:03
@dantaik dantaik changed the title refactor(protocol): improve liveness bond check/return logic into one section of code refactor(protocol): improve liveness bond return logics May 7, 2024
@dantaik dantaik enabled auto-merge May 7, 2024 02:30
@dantaik dantaik changed the title refactor(protocol): improve liveness bond return logics refactor(protocol): improve blok liveness bond handling May 7, 2024
@dantaik dantaik changed the title refactor(protocol): improve blok liveness bond handling refactor(protocol): improve block liveness bond handling May 7, 2024
@dantaik dantaik requested a review from davidtaikocha May 7, 2024 06:16
@dantaik dantaik added this pull request to the merge queue May 7, 2024
Merged via the queue into main with commit 23bfeb6 May 7, 2024
8 checks passed
@dantaik dantaik deleted the lib_proving_improve_1 branch May 7, 2024 06:20
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.

3 participants