Browse Source

Add Governor Timelock Control Warning (#3032)

Co-authored-by: Francisco Giordano <frangio.1@gmail.com>
JulissaDantes 3 years ago
parent
commit
e667ec9533
1 changed files with 4 additions and 0 deletions
  1. 4 0
      contracts/governance/extensions/GovernorTimelockControl.sol

+ 4 - 0
contracts/governance/extensions/GovernorTimelockControl.sol

@@ -16,6 +16,10 @@ import "../TimelockController.sol";
  * the assets and permissions must be attached to the {TimelockController}. Any asset sent to the {Governor} will be
  * the assets and permissions must be attached to the {TimelockController}. Any asset sent to the {Governor} will be
  * inaccessible.
  * inaccessible.
  *
  *
+ * WARNING: Setting up the TimelockController to have additional proposers besides the governor introduces the risk that
+ * approved governance proposals could be blocked by the other proposers, effectively executing a Denial of Service attack,
+ * and therefore blocking access to governance-controlled assets.
+ *
  * _Available since v4.3._
  * _Available since v4.3._
  */
  */
 abstract contract GovernorTimelockControl is IGovernorTimelock, Governor {
 abstract contract GovernorTimelockControl is IGovernorTimelock, Governor {