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

BEP126 #14

Closed
wants to merge 16 commits into from
Closed

BEP126 #14

wants to merge 16 commits into from

Conversation

yoshidan
Copy link
Contributor

@yoshidan yoshidan commented Sep 4, 2023

yoshidan and others added 2 commits August 31, 2023 16:28
Signed-off-by: Naohiro Yoshida <[email protected]>
@yoshidan yoshidan marked this pull request as ready for review September 7, 2023 00:00
@yoshidan yoshidan requested a review from 3100 September 7, 2023 00:00
Signed-off-by: Naohiro Yoshida <[email protected]>
Naohiro Yoshida added 3 commits September 13, 2023 21:57
Signed-off-by: Naohiro Yoshida <[email protected]>
Signed-off-by: Naohiro Yoshida <[email protected]>
Signed-off-by: Naohiro Yoshida <[email protected]>
@@ -0,0 +1,1078 @@
pragma solidity 0.6.4;
Copy link
Member

Choose a reason for hiding this comment

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

Please assign a license identifier:

// SPDX-License-Identifier: Apache-2.0
// Copyright 2020 All BNB Chain Developers
// Modified by Datachain, Inc. on 2023

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@@ -0,0 +1,86 @@
pragma solidity 0.6.4;
Copy link
Member

Choose a reason for hiding this comment

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

Same as ./BSCValidatorSet.template.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done


func getVoteAttestationFromHeader(header *types.Header) (*VoteAttestation, error) {
if len(header.Extra) <= extraVanity+extraSeal {
return nil, nil
Copy link
Member

Choose a reason for hiding this comment

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

Could you add a comment explaining under what circumstances this occurs?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added link to bsc source.

func createMainnetProver() (*module.Prover, core.Chain, error) {
chain, err := ethereum.NewChain(ethereum.ChainConfig{
EthChainId: 56,
RpcAddr: "https://bsc-mainnet-rpc.allthatnode.com",
Copy link
Member

Choose a reason for hiding this comment

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

This doesn't seem to be an official RPC. Is there a particular reason for choosing this?
Additionally, what do you think about making it configurable from the outside?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The official node was not able to obtain account proof.

2023/09/15 16:17:46 Failed to run command : failed to get account proof : height = 31757489, failed to get account proof limit exceeded

I made RPC address configurable now.

yoshidan and others added 10 commits September 15, 2023 16:12
* update lcp

Signed-off-by: Naohiro Yoshida <[email protected]>
Signed-off-by: Naohiro Yoshida <[email protected]>
Signed-off-by: Naohiro Yoshida <[email protected]>
Signed-off-by: Naohiro Yoshida <[email protected]>
Signed-off-by: Naohiro Yoshida <[email protected]>
Signed-off-by: Naohiro Yoshida <[email protected]>
Signed-off-by: Naohiro Yoshida <[email protected]>
Signed-off-by: Naohiro Yoshida <[email protected]>
Signed-off-by: Naohiro Yoshida <[email protected]>
Signed-off-by: Naohiro Yoshida <[email protected]>
@yoshidan yoshidan closed this Sep 27, 2023
@yoshidan yoshidan deleted the bep126 branch May 16, 2024 00:07
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.

2 participants