Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Implement RESTRICT_ACTION_TO_SELF protocol feature #7088

Merged
merged 9 commits into from
Apr 10, 2019

Conversation

arhag
Copy link
Contributor

@arhag arhag commented Apr 8, 2019

Change Description

Resolves #6705.

This PR adds support for the RESTRICT_ACTION_TO_SELF protocol feature which disallows any authorization bypass for unprivileged contracts. Specifically, it closes the authorization bypass that exists in the original protocol rules when a contract sends actions only to itself (either via inline actions or deferred transactions).

Consensus Changes

  • Consensus Changes

Makes the RESTRICT_ACTION_TO_SELF protocol feature available. See above description and #6705 for details.

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

If any documentation out there describes the authorization bypass that some contracts relied on, and this protocol feature now closes, it needs to be updated to reflect the fact that the bypass will be unavailable after activation of RESTRICT_ACTION_TO_SELF.

@arhag arhag marked this pull request as ready for review April 10, 2019 00:01
unit test for RESTRICT_ACTION_TO_SELF protocol feature (6705 )
Copy link
Contributor

@johndebord johndebord left a comment

Choose a reason for hiding this comment

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

Looks good.

@arhag arhag mentioned this pull request Jul 3, 2019
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.

2 participants