refactor: Mark tips as beta (#12089)

## Description

Following our SDK call, we decided to mark the Tip decorator as beta for now, and not include it in the default posthandler chain.

This PR also fixes events not all included in the response in SimulateTx.



---

### Author Checklist

*All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.*

I have...

- [ ] included the correct [type prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json) in the PR title
- [ ] added `!` to the type prefix if API or client breaking change
- [ ] targeted the correct branch (see [PR Targeting](https://github.com/cosmos/cosmos-sdk/blob/main/CONTRIBUTING.md#pr-targeting))
- [ ] provided a link to the relevant issue or specification
- [ ] followed the guidelines for [building modules](https://github.com/cosmos/cosmos-sdk/blob/main/docs/building-modules)
- [ ] included the necessary unit and integration [tests](https://github.com/cosmos/cosmos-sdk/blob/main/CONTRIBUTING.md#testing)
- [ ] added a changelog entry to `CHANGELOG.md`
- [ ] included comments for [documenting Go code](https://blog.golang.org/godoc)
- [ ] updated the relevant documentation or specification
- [ ] reviewed "Files changed" and left comments if necessary
- [ ] confirmed all CI checks have passed

### Reviewers Checklist

*All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.*

I have...

- [ ] confirmed the correct [type prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json) in the PR title
- [ ] confirmed `!` in the type prefix if API or client breaking change
- [ ] confirmed all author checklist items have been addressed 
- [ ] reviewed state machine logic
- [ ] reviewed API design and naming
- [ ] reviewed documentation is accurate
- [ ] reviewed tests and test coverage
- [ ] manually tested (if applicable)
This commit is contained in:
Amaury
2022-05-31 14:05:16 +00:00
committed by GitHub
parent cabd3a9b4f
commit ece3d0e9fe
7 changed files with 51 additions and 45 deletions
+32 -17
View File
@@ -4,7 +4,7 @@ order: 14
# Transaction Tips
Transaction tips are a mechanism to pay for transaction fees using another denom than the native fee denom of the chain. {synopsis}
Transaction tips are a mechanism to pay for transaction fees using another denom than the native fee denom of the chain. They are still in beta, and are not included by default in the SDK. {synopsis}
## Context
@@ -72,27 +72,42 @@ In both cases, using `SIGN_MODE_LEGACY_AMINO_JSON` is recommended only if hardwa
## Enabling Tips on your Chain
The transaction tips functionality is introduced in Cosmos SDK v0.46, so earlier versions do not have support for tips. If you're using v0.46 or later, then enabling tips on your chain is as simple as adding the `TipMiddleware` in your middleware stack:
The transaction tips functionality is introduced in Cosmos SDK v0.46, so earlier versions do not have support for tips. It is however not included by default in a v0.46 app. Enabling tips on your chain is done by adding the `TipDecorator` in your posthandler chain:
```go
// NewTxHandler defines a TxHandler middleware stack.
func NewTxHandler(options TxHandlerOptions) (tx.Handler, error) {
// --snip--
// HandlerOptions are the options required for constructing a default SDK PostHandler.
type HandlerOptions struct {
BankKeeper types.BankKeeper
}
return ComposeMiddlewares(
// base tx handler that executes Msgs
NewRunMsgsTxHandler(options.MsgServiceRouter, options.LegacyRouter),
// --snip other middlewares--
// MyPostHandler returns a posthandler chain with the TipDecorator.
func MyPostHandler(options HandlerOptions) (sdk.AnteHandler, error) {
if options.BankKeeper == nil {
return nil, sdkerrors.Wrap(sdkerrors.ErrLogic, "bank keeper is required for posthandler")
}
// Add the TipMiddleware
NewTipMiddleware(options.BankKeeper),
)
postDecorators := []sdk.AnteDecorator{
posthandler.NewTipDecorator(options.bankKeeper),
}
return sdk.ChainAnteDecorators(postDecorators...), nil
}
func (app *SimApp) setPostHandler() {
postHandler, err := MyPostHandler(
HandlerOptions{
BankKeeper: app.BankKeeper,
},
)
if err != nil {
panic(err)
}
app.SetPostHandler(postHandler)
}
```
Notice that `NewTipMiddleware` needs a reference to the BankKeeper, for transferring the tip to the fee payer.
If you are using the Cosmos SDK's default middleware stack `NewDefaultTxHandler()`, then the tip middleware is included by default.
Notice that `NewTipDecorator` needs a reference to the BankKeeper, for transferring the tip to the fee payer.
## CLI Usage
@@ -170,7 +185,7 @@ For the fee payer, the SDK added a new method on the existing `TxBuilder` to imp
txBuilder := clientCtx.TxConfig.NewTxBuilder()
err := txBuilder.AddAuxSignerData(auxSignerData)
if err != nil {
return err
return err
}
// A lot of fields will be populated in txBuilder, such as its Msgs, tip
@@ -184,6 +199,6 @@ txBuilder.SetGasLimit(...)
// Usual signing code
err = authclient.SignTx(...)
if err != nil {
return err
return err
}
```