bisq-network / bisq2

GNU Affero General Public License v3.0
140 stars 53 forks source link

Add @ExcludeForHash to inventory in InventoryRequest #2349

Closed HenrikJannsen closed 5 days ago

HenrikJannsen commented 5 days ago

Implements https://github.com/bisq-network/bisq2/issues/2346

When objects get an update we publish both the old version and the new one, where the new one is considered invalid for old nodes. The Invenotry contains all those data and would become itself invalid if we do not exclude the inventory from the hash creation for the pow check. Note that InventoryResponse is a direct message and its only the pow use case where the hash would cause issues.