Skip to content

BEP-402: Complete Missing Fields in Block Header to Generate Signature#2502

Merged
zzzckck merged 1 commit intobnb-chain:developfrom
buddh0:fix_SealSig
Jul 19, 2024
Merged

BEP-402: Complete Missing Fields in Block Header to Generate Signature#2502
zzzckck merged 1 commit intobnb-chain:developfrom
buddh0:fix_SealSig

Conversation

@buddh0
Copy link
Copy Markdown
Contributor

@buddh0 buddh0 commented May 31, 2024

Description

BEP-402: Complete Missing Fields in Block Header to Generate Signature

Rationale

tell us why we need these changes...

Example

add an example CLI or API response...

Changes

Notable changes:

  • add each change in a bullet point here
  • ...

galaio
galaio previously approved these changes Jun 5, 2024
@zzzckck
Copy link
Copy Markdown
Collaborator

zzzckck commented Jun 18, 2024

need a BEP for this PR.

@buddh0 buddh0 changed the title core/types: fix SealHash for Header BEP-402: Complete Missing Fields in Block Header to Generate Signature Jul 2, 2024
Copy link
Copy Markdown
Contributor

@MatusKysel MatusKysel left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Copy Markdown
Collaborator

@zzzckck zzzckck left a comment

Choose a reason for hiding this comment

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

Will not merge until the BEP-402 is fully accepted.

@buddh0
Copy link
Copy Markdown
Contributor Author

buddh0 commented Jul 11, 2024

rebase develop 07.11

@zzzckck zzzckck merged commit 4f38c78 into bnb-chain:develop Jul 19, 2024
@zzzckck
Copy link
Copy Markdown
Collaborator

zzzckck commented Jul 19, 2024

Will not merge until the BEP-402 is fully accepted.

With a second thought, it is ok to merge it. We can just not enable it, if the community reject this BEP.

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.

6 participants