From 8c7e9df6d52d0a95502bb4199defd5b74155ee99 Mon Sep 17 00:00:00 2001
From: Evan Forbes <42654277+evan-forbes@users.noreply.github.com>
Date: Wed, 25 Oct 2023 06:46:57 -0500
Subject: [PATCH] chore: discard abci responses by default (#2764)

## Overview

sets the default to discard abci responses

closes #

## Checklist

- [x] New and updated code has appropriate documentation
- [x] New and updated code has new and/or updated testing
- [x] Required CI checks are passing
- [x] Visual proof for any user facing features like CLI or
documentation updates
- [x] Linked issues closed with keywords

(cherry picked from commit 3677fa962c6fe37457f308c14e541cbe8b85afc0)
---
 app/default_overrides.go | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/app/default_overrides.go b/app/default_overrides.go
index 1462d80cc5..81086ab4ec 100644
--- a/app/default_overrides.go
+++ b/app/default_overrides.go
@@ -233,6 +233,7 @@ func DefaultConsensusConfig() *tmcfg.Config {
 	// TODO: make TimeoutBroadcastTx configurable per https://github.com/celestiaorg/celestia-app/issues/1034
 	cfg.RPC.TimeoutBroadcastTxCommit = 50 * time.Second
 	cfg.RPC.MaxBodyBytes = int64(8388608) // 8 MiB
+
 	cfg.Mempool.TTLNumBlocks = 5
 	cfg.Mempool.TTLDuration = time.Duration(cfg.Mempool.TTLNumBlocks) * appconsts.GoalBlockTime
 	// Given that there is a stateful transaction size check in CheckTx,
@@ -242,12 +243,15 @@ func DefaultConsensusConfig() *tmcfg.Config {
 	upperBoundBytes := appconsts.DefaultSquareSizeUpperBound * appconsts.DefaultSquareSizeUpperBound * appconsts.ContinuationSparseShareContentSize
 	cfg.Mempool.MaxTxBytes = upperBoundBytes
 	cfg.Mempool.MaxTxsBytes = int64(upperBoundBytes) * cfg.Mempool.TTLNumBlocks
-
 	cfg.Mempool.Version = "v1" // prioritized mempool
+
 	cfg.Consensus.TimeoutPropose = appconsts.TimeoutPropose
 	cfg.Consensus.TimeoutCommit = appconsts.TimeoutCommit
 	cfg.Consensus.SkipTimeoutCommit = false
+
 	cfg.TxIndex.Indexer = "null"
+	cfg.Storage.DiscardABCIResponses = true
+
 	return cfg
 }