From 082c8c42906df96bafbae7e1195c024be4a2c7f9 Mon Sep 17 00:00:00 2001 From: woodser Date: Sat, 13 Jul 2024 07:37:48 -0400 Subject: [PATCH] Revert "randomize completed trade info, fixes #1099" This reverts commit db6cb237bff8435cb156a344b7a554b5bde772a5. --- .../trade/statistics/TradeStatistics3.java | 20 ++----------------- 1 file changed, 2 insertions(+), 18 deletions(-) diff --git a/core/src/main/java/haveno/core/trade/statistics/TradeStatistics3.java b/core/src/main/java/haveno/core/trade/statistics/TradeStatistics3.java index dbaa880027..75db58808a 100644 --- a/core/src/main/java/haveno/core/trade/statistics/TradeStatistics3.java +++ b/core/src/main/java/haveno/core/trade/statistics/TradeStatistics3.java @@ -54,8 +54,6 @@ import java.util.Date; import java.util.HashMap; import java.util.Map; import java.util.Optional; -import java.util.concurrent.ThreadLocalRandom; -import java.util.concurrent.TimeUnit; import static com.google.common.base.Preconditions.checkNotNull; @@ -90,27 +88,13 @@ public final class TradeStatistics3 implements ProcessOncePersistableNetworkPayl Offer offer = checkNotNull(trade.getOffer()); return new TradeStatistics3(offer.getCurrencyCode(), trade.getPrice().getValue(), - fuzzTradeAmount(trade.getAmount().longValueExact()), + trade.getAmount().longValueExact(), offer.getPaymentMethod().getId(), - fuzzTradeDate(trade.getTakeOfferDate().getTime()), + trade.getTakeOfferDate().getTime(), truncatedArbitratorNodeAddress, extraDataMap); } - private static long fuzzTradeAmount(long exactAmount) { // randomize completed trade info #1099 - long adjustedAmount = (long) ThreadLocalRandom.current().nextDouble( - exactAmount * 0.95, exactAmount * 1.05); - log.info("fuzzed trade amount from {} to {}", exactAmount, adjustedAmount); - return adjustedAmount; - } - - private static long fuzzTradeDate(long originalTimestamp) { // randomize completed trade info #1099 - long adjustedTimestamp = ThreadLocalRandom.current().nextLong( - originalTimestamp-TimeUnit.HOURS.toMillis(24), originalTimestamp); - log.info("fuzzed trade datestamp from {} to {}", new Date(originalTimestamp), new Date(adjustedTimestamp)); - return adjustedTimestamp; - } - // This enum must not change the order as we use the ordinal for storage to reduce data size. // The payment method string can be quite long and would consume 15% more space. // When we get a new payment method we can add it to the enum at the end. Old users would add it as string if not