Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

🐛 Fix invalid init response - Closes #7260 #7281

Merged
merged 2 commits into from
Jul 8, 2022

Conversation

has5aan
Copy link
Contributor

@has5aan has5aan commented Jul 6, 2022

What was the problem?

This PR resolves #7260

How was it solved?

Updated ABIHandler.init to:

  • Configure bftBatchSize from injected genesis configuration.
  • Configure rpc.ws from injected rpc.ws configuration.

Updated applicationConfigSchema to allow for a numeric bftBatchSize value with minimum value of 1 and default set to 103.
Updated GenesisConfig interface to allow for a numeric bftBatchSize.

Update configuration for dev_net and and dpos-mainchain, to contain bftBatchSize property set to 103.

Modified unit tests.

How was it tested?

Passing unit tests.

@CLAassistant
Copy link

CLAassistant commented Jul 6, 2022

CLA assistant check
All committers have signed the CLA.

@has5aan has5aan self-assigned this Jul 6, 2022
@has5aan has5aan requested review from ishantiw and mitsuaki-u July 6, 2022 16:02
Copy link
Contributor

@ishantiw ishantiw left a comment

Choose a reason for hiding this comment

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

Nice work @has5aan 🎉

@shuse2 shuse2 removed the request for review from mitsuaki-u July 7, 2022 18:14
@ishantiw ishantiw merged commit 7ebfa1a into development Jul 8, 2022
@ishantiw ishantiw deleted the 7260-invalid-init-response branch July 8, 2022 08:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Invalid init response
4 participants