Merge pull request #788 from OffchainLabs/panic-on-compress-failure

Panic if tx compression fails in PosterDataCost
OffchainLabs · Jul 13, 2022 · 9bdfeaf · 9bdfeaf
2 parents f97f49c + 12a4442
commit 9bdfeaf
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/arbos/l1pricing/l1pricing.go b/arbos/l1pricing/l1pricing.go
@@ -20,7 +20,6 @@ import (
 	"github.com/ethereum/go-ethereum/common"
 	"github.com/ethereum/go-ethereum/core"
 	"github.com/ethereum/go-ethereum/core/types"
-	"github.com/ethereum/go-ethereum/log"
 	"github.com/offchainlabs/nitro/arbos/storage"
 	"github.com/offchainlabs/nitro/arbos/util"
 )
@@ -622,8 +621,7 @@ func (ps *L1PricingState) PosterDataCost(message core.Message, poster common.Add
 
 	byteCount, err := byteCountAfterBrotli0(message.Data())
 	if err != nil {
-		log.Error("failed to compress tx", "err", err)
-		return common.Big0, 0
+		panic(fmt.Sprintf("failed to compress tx: %v", err))
 	}
 
 	// Approximate the l1 fee charged for posting this tx's calldata