From 54912879310dea1ad4be204b7e1f598c22e2be7d Mon Sep 17 00:00:00 2001 From: zzz Date: Sun, 1 Feb 2015 19:53:11 +0000 Subject: [PATCH] OCMOSJ: Pick a OB tunnel at random, not with the OBEP closest to the lease, as that may be reducing the odds of trying a different path and hurting connection reliability. While the change may slightly increase connection congestion, if it helps with reliability then it's worth it. --- router/java/src/net/i2p/router/RouterVersion.java | 2 +- .../i2p/router/message/OutboundClientMessageOneShotJob.java | 6 ++++-- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/router/java/src/net/i2p/router/RouterVersion.java b/router/java/src/net/i2p/router/RouterVersion.java index 80cfb9c84..5b1ba79e3 100644 --- a/router/java/src/net/i2p/router/RouterVersion.java +++ b/router/java/src/net/i2p/router/RouterVersion.java @@ -18,7 +18,7 @@ public class RouterVersion { /** deprecated */ public final static String ID = "Monotone"; public final static String VERSION = CoreVersion.VERSION; - public final static long BUILD = 18; + public final static long BUILD = 19; /** for example "-test" */ public final static String EXTRA = ""; diff --git a/router/java/src/net/i2p/router/message/OutboundClientMessageOneShotJob.java b/router/java/src/net/i2p/router/message/OutboundClientMessageOneShotJob.java index d9f7a0a51..8a517a806 100644 --- a/router/java/src/net/i2p/router/message/OutboundClientMessageOneShotJob.java +++ b/router/java/src/net/i2p/router/message/OutboundClientMessageOneShotJob.java @@ -727,8 +727,10 @@ public class OutboundClientMessageOneShotJob extends JobImpl { * Anonymity issues? */ private TunnelInfo selectOutboundTunnel() { - Hash gw = _lease.getGateway(); - return getContext().tunnelManager().selectOutboundTunnel(_from.calculateHash(), gw); + // hurts reliability? let's try picking at random again + //Hash gw = _lease.getGateway(); + //return getContext().tunnelManager().selectOutboundTunnel(_from.calculateHash(), gw); + return getContext().tunnelManager().selectOutboundTunnel(_from.calculateHash()); } /**