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

fix(runtime) DeleteAccount action must be the last action #2829

Merged
merged 5 commits into from
Jun 10, 2020

Conversation

lexfrl
Copy link
Contributor

@lexfrl lexfrl commented Jun 10, 2020

In order to avoid dDos we disallow to have DeleteAccount to be not a last action in Receipt.

@vgrichina this contains a new rpc error type to handle

fixes #2599

@gitpod-io
Copy link

gitpod-io bot commented Jun 10, 2020

@lexfrl lexfrl force-pushed the fckt/delete-last-action branch from 344820b to 8eb3437 Compare June 10, 2020 14:43
@lexfrl lexfrl force-pushed the fckt/delete-last-action branch from 8eb3437 to 1dcda25 Compare June 10, 2020 14:44
Copy link
Collaborator

@bowenwang1996 bowenwang1996 left a comment

Choose a reason for hiding this comment

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

Why did you change Cargo.lock?

core/primitives/src/errors.rs Outdated Show resolved Hide resolved
Co-authored-by: Vlad Frolov <frolvlad@gmail.com>
@lexfrl lexfrl force-pushed the fckt/delete-last-action branch from cf25f32 to a3ffe63 Compare June 10, 2020 18:17
@lexfrl
Copy link
Contributor Author

lexfrl commented Jun 10, 2020

Why did you change Cargo.lock?

@bowenwang1996
Maybe cargo build changed it.. I reverted it back.

@lexfrl lexfrl changed the title fix(runtime) DeleteAccount action must be a last action fix(runtime) DeleteAccount action must be the last action Jun 10, 2020
@nearprotocol-bulldozer nearprotocol-bulldozer bot merged commit fe67f8f into master Jun 10, 2020
@nearprotocol-bulldozer nearprotocol-bulldozer bot deleted the fckt/delete-last-action branch June 10, 2020 18:45
@vgrichina
Copy link
Collaborator

@fckt I plan to remove need to update near-api-js for new error types:
near/near-api-js#331

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.

[Runtime] Delete Account action consumes too much CPU
5 participants