add comments

CHANGELOG.md

@@ -57,6 +57,7 @@ Ref: https://keepachangelog.com/en/1.0.0/
 * (eth) [\#845](https://github.com/cosmos/ethermint/pull/845) The `eth` namespace must be included in the list of API's as default to run the rpc server without error.
 * (evm) [#202](https://github.com/tharsis/ethermint/pull/202) Web3 api `SendTransaction`/`SendRawTransaction` returns ethereum compatible transaction hash, and query api `GetTransaction*` also accept that.
 * (rpc) [tharsis#258](https://github.com/tharsis/ethermint/pull/258) Return empty `BloomFilter` instead of throwing an error when it cannot be found (`nil` or empty).
+* (rpc) [tharsis#277](https://github.com/tharsis/ethermint/pull/321) Fix `BloomFilter` response.
 
 ### Improvements
 

docs/core/proto-docs.md

@@ -570,7 +570,7 @@ method.
 
 | Field | Type | Label | Description |
 | ----- | ---- | ----- | ----------- |
-| `height` | [int64](#int64) | | height of the block which we want to query the bloom filter |
+| `height` | [int64](#int64) | | height of the block which we want to query the bloom filter. Tendermint always replace the query request header by the current context header, height cannot be extracted from there, so we need to explicitly pass it in parameter. |
 
 
 

ethereum/rpc/types/block.go

@@ -30,6 +30,7 @@ func NewBlockNumber(n *big.Int) BlockNumber {
 
 // ContextWithHeight wraps a context with the a gRPC block height header. If the provided height is
 // 0, it will return an empty context and the gRPC query will use the latest block height for querying.
+// Note that all metadata are processed and removed by tendermint layer, so it wont be accessible at gRPC server level.
 func ContextWithHeight(height int64) context.Context {
  if height == 0 {
  return context.Background()

proto/ethermint/evm/v1alpha1/query.proto

@@ -223,7 +223,10 @@ message QueryBlockLogsResponse {
 // QueryBlockBloomRequest is the request type for the Query/BlockBloom RPC
 // method.
 message QueryBlockBloomRequest {
- // height of the block which we want to query the bloom filter
+ // height of the block which we want to query the bloom filter.
+ // Tendermint always replace the query request header by the current context
+ // header, height cannot be extracted from there, so we need to explicitly pass
+ // it in parameter.
  int64 height = 1;
 }