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

fix 7600 double confirm after changing sign key #7601

Merged
merged 4 commits into from
Jul 19, 2019
Merged

Conversation

taokayan
Copy link
Contributor

@taokayan taokayan commented Jul 5, 2019

Change Description

This fix #7600 double confirm bug after changing signing key which results in producing by different nodeos

Consensus Changes

No. Changes are subjective.

API Changes

No

Documentation Additions

No

Copy link
Contributor

@arhag arhag left a comment

Choose a reason for hiding this comment

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

The PR also needs a Python test to ensure changing block signing keys back and forth doesn't not lead to the problem described in issue #7600.

plugins/producer_plugin/producer_plugin.cpp Outdated Show resolved Hide resolved
programs/eosio-launcher/main.cpp Outdated Show resolved Hide resolved
plugins/producer_plugin/producer_plugin.cpp Outdated Show resolved Hide resolved
tests/nodeos_producer_watermark_test.py Outdated Show resolved Hide resolved
tests/nodeos_producer_watermark_test.py Show resolved Hide resolved
tests/nodeos_producer_watermark_test.py Outdated Show resolved Hide resolved
tests/nodeos_producer_watermark_test.py Outdated Show resolved Hide resolved
tests/nodeos_producer_watermark_test.py Outdated Show resolved Hide resolved
tests/nodeos_producer_watermark_test.py Outdated Show resolved Hide resolved
Copy link
Contributor

@arhag arhag left a comment

Choose a reason for hiding this comment

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

Also make a version of this PR for develop.

@arhag arhag merged commit de18961 into release/1.8.x Jul 19, 2019
@arhag arhag deleted the kayan_signkey branch July 19, 2019 18:23
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