Remove fee handler and fix post-handler error handling. (#618)

* Remove the fee handler customization since the SDK added a post handler that does pretty much the same thing.

* Make DefaultGasLimit a variable instead of constant.

* Fix the post handler response code thing.

* Bring in SDK changes up to v0.50.12 and Mark v0.50.12-pio-1. (#614)

* build(deps): Bump github.com/cosmos/cosmos-db from 1.1.0 to 1.1.1 (cosmos#23030)

* build(deps): Bump github.com/cosmos/ledger-cosmos-go from 0.13.3 to 0.14.0 (cosmos#23029)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Julien Robert <julien@rbrt.fr>

* feat(x/tx): add an option to encode maps using amino json (backport cosmos#23539) (cosmos#23540)

Co-authored-by: Julien Robert <julien@rbrt.fr>

* feat(client/v2): add map support (backport cosmos#23544) (cosmos#23554)

Co-authored-by: Julien Robert <julien@rbrt.fr>

* docs(keyring): add keyctl docs (backport cosmos#23563) (cosmos#23566)

Co-authored-by: Alessio Treglia <al@essio.dev>

* docs: correct explanation on how to set custom signer via depinject (backport cosmos#23654) (cosmos#23663)

Co-authored-by: Julien Robert <julien@rbrt.fr>

* chore: remove unused orm module (backport cosmos#23633) (cosmos#23637)

Co-authored-by: Zachary Becker <zrbecker@gmail.com>

* Merge commit from fork

* Prevent empty groups

(cherry picked from commit 95090a0ec1f193104ba7d6d033a490d1515e54d8)

* Handle inflight proposals

(cherry picked from commit 855983471882068894ed7952b592ab3e61b464b9)

* No empty group with simulations

* Update changelog

* Set release date

* updates

* Update RELEASE_NOTES.md

* Update RELEASE_NOTES.md

---------

Co-authored-by: Alex Peters <alpe@users.noreply.github.com>
Co-authored-by: Julien Robert <julien@rbrt.fr>

* Mark v0.50.12-pio-1 in the changelog.

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Julien Robert <julien@rbrt.fr>
Co-authored-by: Alessio Treglia <al@essio.dev>
Co-authored-by: Zachary Becker <zrbecker@gmail.com>

* Prov: Improve performance of GetAllBalances and GetAccountsBalances. (#616)

* Improve performance of GetAllBalances and GetAccountsBalances.

* Add changelog entry.

* Bring in SDK changes up to v0.50.13 and Mark v0.50.13-pio-1. (#615)

* Merge commit from fork

* Fix ABS061

* Use typed event

* Add changelog

* Review feedback

* Check for events

* Review feedback

* Remove unecessary code comments

* Set release date

* fix changelog

---------

Co-authored-by: Alex | Interchain Labs <alex@interchainlabs.io>

* Add changelog entry.

* Put nothing back at the top of the changelog.

---------

Co-authored-by: Alexander Peters <alpe@users.noreply.github.com>
Co-authored-by: Alex | Interchain Labs <alex@interchainlabs.io>

* Add a bunch of logging.

* Revert "Add a bunch of logging."

This reverts commit 3baf4d3.

* Make the DefaultGasLimit a constant again (back to how the SDK has it).

* Add changelog entry.

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Julien Robert <julien@rbrt.fr>
Co-authored-by: Alessio Treglia <al@essio.dev>
Co-authored-by: Zachary Becker <zrbecker@gmail.com>
Co-authored-by: Alexander Peters <alpe@users.noreply.github.com>
Co-authored-by: Alex | Interchain Labs <alex@interchainlabs.io>

Author: 7 people

CHANGELOG.md

@@ -38,7 +38,9 @@ Ref: https://keepachangelog.com/en/1.0.0/
 
 ## [Unreleased Provenance]
 
-* nothing
+### Improvments
+
+* [618](https://github.com/provenance-io/cosmos-sdk/pull/618) Provenance: Remove the fee handler (now using the post handler).
 
 ---
 

baseapp/baseapp.go

@@ -172,8 +172,6 @@ type BaseApp struct {
 	// trace set will return full stack traces for errors in ABCI Log field
 	trace bool
 
-	feeHandler sdk.FeeHandler
-
 	aggregateEventsFunc func(anteEvents []abci.Event, resultEvents []abci.Event) ([]abci.Event, []abci.Event)
 
 	// indexEvents defines the set of events in the form {eventType}.{attributeKey},
@@ -984,7 +982,13 @@ func (app *BaseApp) runTxProv(mode execMode, txBytes []byte) (gInfo sdk.GasInfo,
 
 		newCtx, errPostHandler := app.postHandler(postCtx, tx, mode == execModeSimulate, err == nil)
 		if errPostHandler != nil {
-			return gInfo, nil, anteEvents, ctx, errors.Join(err, errPostHandler)
+			// The result of errors.Join breaks the response code stuff, resulting in code 1 (logic error) always.
+			// So if there was also an error running the msgs, we'll use the error code from that, but include
+			// the post handler error too. Otherwise, we just return the post handler error.
+			if err != nil {
+				errPostHandler = errorsmod.Wrap(err, errPostHandler.Error())
+			}
+			return gInfo, nil, anteEvents, ctx, errPostHandler
 		}
 
 		// we don't want runTx to panic if runMsgs has failed earlier
@@ -995,16 +999,10 @@ func (app *BaseApp) runTxProv(mode execMode, txBytes []byte) (gInfo sdk.GasInfo,
 	}
 
 	if err == nil {
-		var feeEvents sdk.Events
 		if mode == execModeFinalize {
 			// When block gas exceeds, it'll panic and won't commit the cached store.
 			consumeBlockGas()
 
-			feeEvents, err = FeeInvoke(mode, app, runMsgCtx)
-			if err != nil {
-				return gInfo, nil, nil, ctx, err
-			}
-
 			msCache.Write()
 			app.finalizeBlockState.eventHistory = append(app.finalizeBlockState.eventHistory, result.Events...)
 		}
@@ -1013,12 +1011,6 @@ func (app *BaseApp) runTxProv(mode execMode, txBytes []byte) (gInfo sdk.GasInfo,
 			// append the events in the order of occurrence
 			result.Events = append(anteEvents, result.Events...)
 		}
-
-		// additional fee events
-		if len(feeEvents) > 0 {
-			// append the fee events at the end of the other events, since they get charged at the end of the Tx
-			result.Events = append(result.Events, feeEvents.ToABCIEvents()...)
-		}
 	}
 
 	if result != nil { // tx was successful run aggregator for ante and result events
@@ -1038,19 +1030,6 @@ func AggregateEvents(app *BaseApp, anteEvents []abci.Event, resultEvents []abci.
 	return anteEvents, resultEvents
 }
 
-// FeeInvoke apply fee logic and append events
-func FeeInvoke(mode execMode, app *BaseApp, runMsgCtx sdk.Context) (sdk.Events, error) {
-	if app.feeHandler != nil {
-		// call the msgFee
-		_, eventsFromFeeHandler, err := app.feeHandler(runMsgCtx, mode == execModeSimulate)
-		if err != nil {
-			return nil, err
-		}
-		return eventsFromFeeHandler, nil
-	}
-	return nil, nil
-}
-
 // runMsgs iterates through a list of messages and executes them with the provided
 // Context and execution mode. Messages will only be executed during simulation
 // and DeliverTx. An error is returned if any single message fails or if a

baseapp/options.go

@@ -6,6 +6,7 @@ import (
 	"math"
 
 	abci "github.com/cometbft/cometbft/abci/types"
+
 	dbm "github.com/cosmos/cosmos-db"
 
 	"cosmossdk.io/store/metrics"
@@ -319,15 +320,6 @@ func (app *BaseApp) SetQueryMultiStore(ms storetypes.MultiStore) {
 	app.qms = ms
 }
 
-// SetFeeHandler sets the FeeHandler which if set will change the behavior of fee handling
-func (app *BaseApp) SetFeeHandler(feeHandler sdk.FeeHandler) {
-	if app.sealed {
-		panic("SetFeeHandler() on sealed BaseApp")
-	}
-
-	app.feeHandler = feeHandler
-}
-
 // SetAggregateEventsFunc sets the function that aggregates events from baseapp result events and feehandler events
 func (app *BaseApp) SetAggregateEventsFunc(aggregateEventsFunc func(resultEvents []abci.Event, feeEvents []abci.Event) ([]abci.Event, []abci.Event)) {
 	if app.sealed {