diff --git a/core/README.md b/core/README.md index 187685be4..762ee8091 100644 --- a/core/README.md +++ b/core/README.md @@ -62,7 +62,7 @@ | [CAP-0021](cap-0021.md) | Generalized transaction preconditions | David Mazières | Draft | | [CAP-0022](cap-0022.md) | Invalid transactions must have no effects | David Mazières | Draft | | [CAP-0023](cap-0023.md) | Two-Part Payments with BalanceEntry | Jonathan Jove | Draft | -| [cap_orbitlens_41fc89b6](cap_orbitlens_41fc89b6.md) | Disable Inflation Mechanism | OrbitLens | Draft | +| [CAP-0026](cap-0026.md) | Disable Inflation Mechanism | OrbitLens | Draft | ### Rejected Proposals | Number | Title | Author | Status | diff --git a/core/cap_orbitlens_41fc89b6.md b/core/cap-0026.md similarity index 66% rename from core/cap_orbitlens_41fc89b6.md rename to core/cap-0026.md index a9d753180..240fcda6b 100644 --- a/core/cap_orbitlens_41fc89b6.md +++ b/core/cap-0026.md @@ -6,7 +6,7 @@ Title: Disable Inflation Mechanism Author: OrbitLens Status: Draft Created: 2019-07-10 -Updated: 2019-08-08 +Updated: 2019-09-24 Discussion: https://groups.google.com/forum/#!topic/stellar-dev/LIFvbMi9jPo Protocol version: TBD ``` @@ -36,25 +36,21 @@ distributions that require complex logic, like stimulating DEX liquidity provide ## Abstract -Turning off inflation requires several changes in Stellar Core, namely `Inflation` and -`SetOptions` operations behavior as well as fees processing routine. -At the same time, it can be implemented without XDR changes and breaking protocol changes. +Turning off inflation requires a minor change in Stellar Core, namely `Inflation` +operation behavior. At the same time, it can be implemented without XDR changes and +breaking protocol changes. ## Specification -This proposal requires the following Core behavior changes: - -1. Inflation operation always returns `INFLATION_NOT_TIME` result code. -2. SetOptions operation returns `SET_OPTIONS_INVALID_INFLATION` result code when a users -tries to change `inflationDest`. -3. Fee processing routine discards paid transaction fees instead of adding them to the fee pool. -4. LedgerHeader properties `feePool` and `inflationSeq` for a newly created ledger are set to zero. +This proposal requires a single Core behavior change. +Transaction containing `Inflation` operation should always return +`opNOT_SUPPORTED` result code. ## Design Rationale The proposed approach does not require breaking protocol changes and allows turning on the inflation mechanism in the future if needed. Due to the simplicity of proposed changes, -the implementation potentially should require minimum efforts. +the implementation potentially requires minimum efforts. ## Security Concerns @@ -62,9 +58,4 @@ None. ## Backwards Incompatibilities -This CAP contains no breaking changes and is fully backward compatible. - -## Questions - -- Is it possible (and does it makes sense) to remove `inflationDest` field from -the `Account` entry, as well as `feePool` and `inflationSeq` from `LedgerHeader`? \ No newline at end of file +This CAP contains no breaking changes and is fully backward compatible. \ No newline at end of file