internal/ethapi: add comment explaining return of nil instead of error (ethereum#25097)

kosuke-taniguchi · fjl · cp-wjhan · commit 3dc5492dbf36 · 2023-06-26T11:31:36.000+09:00
Co-authored-by: Felix Lange &lt;fjl@twurst.com&gt;
diff --git a/internal/ethapi/api.go b/internal/ethapi/api.go
@@ -1752,6 +1752,8 @@ func (s *PublicTransactionPoolAPI) GetRawTransactionByHash(ctx context.Context,
 func (s *PublicTransactionPoolAPI) GetTransactionReceipt(ctx context.Context, hash common.Hash) (map[string]interface{}, error) {
 	tx, blockHash, blockNumber, index, err := s.b.GetTransaction(ctx, hash)
 	if err != nil {
+		// When the transaction doesn't exist, the RPC method should return JSON null
+		// as per specification.
 		return nil, nil
 	}
 	receipts, err := s.b.GetReceipts(ctx, blockHash)

Original file line number	Diff line number	Diff line change
`@@ -1752,6 +1752,8 @@ func (s *PublicTransactionPoolAPI) GetRawTransactionByHash(ctx context.Context,`
`1752`	`1752`	`func (s *PublicTransactionPoolAPI) GetTransactionReceipt(ctx context.Context, hash common.Hash) (map[string]interface{}, error) {`
`1753`	`1753`	`tx, blockHash, blockNumber, index, err := s.b.GetTransaction(ctx, hash)`
`1754`	`1754`	`if err != nil {`
	`1755`	`+ // When the transaction doesn't exist, the RPC method should return JSON null`
	`1756`	`+ // as per specification.`
`1755`	`1757`	`return nil, nil`
`1756`	`1758`	`}`
`1757`	`1759`	`receipts, err := s.b.GetReceipts(ctx, blockHash)`