Skip to content

Commit

Permalink
Merge pull request #139 from rootulp/rp/cip-20-review
Browse files Browse the repository at this point in the history
docs(cip-20): move to review
  • Loading branch information
jcstein authored Apr 23, 2024
2 parents bbb9d3a + 9cb584b commit 6e3e53d
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions cips/cip-20.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ title: Disable Blobstream module
description: Disable the Blobstream state machine module
author: Rootul Patel (@rootulp)
discussions-to: https://forum.celestia.org/t/cip-disable-blobstream-module/1693
status: Draft
status: Review
type: Standards Track
category: Core
created: 2024-04-16
Expand Down Expand Up @@ -68,7 +68,9 @@ Additionally, a store migration needs to be registered during the upgrade proces

## Security Considerations

TBD
This CIP generally reduces the surface area of celestia-app because it removes functionality that is no longer deemed necessary. Therefore, it could be argued that this CIP reduces the surface area for bugs or security vulnerabilities.

However, there is a slim chance that this CIP uncoveres a bug because the Celestia state machine hasn't disabled a module before. Executing this CIP involves using new components (i.e. a versioned module manager and configurator) which may uncover bugs in software outside the scope of this CIP.

## Copyright

Expand Down

0 comments on commit 6e3e53d

Please sign in to comment.