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

Consolidated Security Updates for 2.0.12 - 2.1.x #10273

Merged
merged 3 commits into from
Apr 20, 2021

Conversation

nickjjzhao
Copy link
Contributor

@nickjjzhao nickjjzhao commented Apr 20, 2021

Consolidated Security Updates for 2.0.12

  • Apply three-strikes rule to all transaction failures
  • Apply unconditional subjective CPU check along with some additional logging
  • Provide options to enable subjective CPU billing for P2P and API transactions ,and provide an option to disable it for individual accounts

v2.0.x - #10264

Co-Authored-By:Kevin Heifner [email protected]
Co-Authored-By:Lin Huang [email protected]

Change Type

Select ONE:

  • Documentation
  • Stability bug fix
  • Other
  • Other - special case

Testing Changes

Select ANY that apply:

  • New Tests
  • Existing Tests
  • Test Framework
  • CI System

- Apply three-strikes rule to all transaction failures
- Apply unconditional subjective CPU check along with some additional logging
- Provide options to enable subjective CPU billing for P2P and API transactions ,and provide an option to disable it for individual accounts
@heifner heifner changed the title Merge security patch PR#371 into release/2.1.x Consolidated Security Updates for 2.0.12 - 2.1.x Apr 20, 2021
heifner
heifner previously approved these changes Apr 20, 2021
@heifner heifner dismissed their stale review April 20, 2021 12:54

Sorry, this needs a bit more work

plugins/producer_plugin/producer_plugin.cpp Outdated Show resolved Hide resolved
plugins/producer_plugin/producer_plugin.cpp Outdated Show resolved Hide resolved
plugins/producer_plugin/producer_plugin.cpp Outdated Show resolved Hide resolved
@nickjjzhao nickjjzhao merged commit df34a98 into release/2.1.x Apr 20, 2021
@nickjjzhao nickjjzhao deleted the release-2.1.x-security-fixes branch April 20, 2021 22:31
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