Merge pull request #1094 from HenrikJannsen/bugfixes_with_network_dat…

…a_validation Bugfixes with network data validation
bisq-network · Aug 7, 2023 · 306f567 · 306f567
2 parents 3999587 + 20553de
commit 306f567
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/contract/src/main/java/bisq/contract/ContractService.java b/contract/src/main/java/bisq/contract/ContractService.java
@@ -77,7 +77,7 @@ public CompletableFuture<Boolean> shutdown() {
     }
 
     private <T extends Offer<?, ?>> byte[] getContractHash(Contract<T> contract) {
-        return DigestUtil.sha256(contract.getHashForSignature());
+        return DigestUtil.hash(contract.getHashForSignature());
     }
 
 }
diff --git a/trade/src/main/java/bisq/trade/protocol/messages/TradeMessage.java b/trade/src/main/java/bisq/trade/protocol/messages/TradeMessage.java
@@ -46,7 +46,7 @@ protected TradeMessage(String tradeId, NetworkId sender) {
         this.tradeId = tradeId;
         this.sender = sender;
 
-        NetworkDataValidation.validateId(tradeId);
+        NetworkDataValidation.validateText(tradeId, 200); // For private channels we combine user profile IDs for channelId
     }
 
     public bisq.trade.protobuf.TradeMessage.Builder getTradeMessageBuilder() {