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

Update chain loader to call action with correct param - Closes #4501 #4502

Merged
merged 3 commits into from
Nov 14, 2019

Conversation

shuse2
Copy link
Collaborator

@shuse2 shuse2 commented Nov 14, 2019

What was the problem?

getTransactions was called with invalid params, and causing banning

How did I solve it?

Call with correct parameter.

How to manually test it?

Start 2 nodes and and observe it will not get banned

Review checklist

framework/src/modules/chain/loader.js Outdated Show resolved Hide resolved
@shuse2 shuse2 force-pushed the 4501-fix_rpc_call_from_loader branch from aba6161 to fefecec Compare November 14, 2019 15:41
Copy link
Contributor

@ManuGowda ManuGowda left a comment

Choose a reason for hiding this comment

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

no unit test required?

@shuse2
Copy link
Collaborator Author

shuse2 commented Nov 14, 2019

@ManuGowda Test are added

@sridharmeganathan sridharmeganathan added this to the Sprint 10 milestone Nov 14, 2019
@shuse2 shuse2 removed this from the Sprint 10 milestone Nov 14, 2019
@shuse2 shuse2 force-pushed the 4501-fix_rpc_call_from_loader branch from f8f0f7d to 0b84c88 Compare November 14, 2019 19:49
@shuse2 shuse2 force-pushed the 4501-fix_rpc_call_from_loader branch from 0b84c88 to aa18580 Compare November 14, 2019 19:51
@shuse2 shuse2 merged commit e3136da into development Nov 14, 2019
@shuse2 shuse2 deleted the 4501-fix_rpc_call_from_loader branch November 14, 2019 20:58
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 transaction payload during RPC causes network isolation
4 participants