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

Properly guard read_write chain APIs during read-only and irreversible read modes #7671

Closed

Description

@arhag

Both chain_apis::read_write::validate() and controller::push_transaction currently only validate that the controller read-mode is not chain::db_read_mode::READ_ONLY when it should also ensure it is not chain::db_read_mode::IRREVERSIBLE either. Furthermore, controller::push_scheduled_transaction does do any validation on the read-mode even though calls to it shouldn't be allowed during irreversible or read-only modes similar to controller::push_transaction.

Activity

  1. arhag commented on Jul 25, 2019

    @arhag
    ContributorAuthor

    Also ensure producer plugin is in start_block_result::waiting mode when the controller read-mode is chain::db_read_mode::IRREVERSIBLE.

    Relevant code:

    if( chain.get_read_mode() == chain::db_read_mode::READ_ONLY )
    return start_block_result::waiting;

  2. added a commit that references this issue on Oct 16, 2019
  3. arhag commented on Oct 16, 2019

    @arhag
    ContributorAuthor

    Resolved by #8056.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions