CompactBlock in GetBlockRange reports different version from LightWalletD

[pacu]

Security check

• This issue is not a security vulnerability.

Usage mode

zainod (running the daemon)

Describe the bug

the problem is this field specifically.

message CompactBlock {
uint32 protoVersion = 1; // the version of this wire format, for storage

lightwalletd returns 0, Zaino 4. Neither of those numbers make much sense but given there is no versioning for storage it makes more sense that it's zero instead of of 4

Steps to reproduce

run integration tests grpc comparison tests. zcash/integration-tests#84

Expected behavior

this should be equivalent

Environment

all platforms

Configuration

Logs

Additional context

No response

[zancas]

I'm AFK, can you please assign me a follow up issue?

[daira]

Zaino probably interpreted the field, contrary to the comment, as the block version.

[pacu]

We discussed this in LCWG and it appears that 1 could be the value that it should have in both servers