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.
This commit is contained in:
zzz
2015-02-01 19:53:11 +00:00
parent 7256096b8a
commit 5491287931
2 changed files with 5 additions and 3 deletions

View File

@ -18,7 +18,7 @@ public class RouterVersion {
/** deprecated */ /** deprecated */
public final static String ID = "Monotone"; public final static String ID = "Monotone";
public final static String VERSION = CoreVersion.VERSION; public final static String VERSION = CoreVersion.VERSION;
public final static long BUILD = 18; public final static long BUILD = 19;
/** for example "-test" */ /** for example "-test" */
public final static String EXTRA = ""; public final static String EXTRA = "";

View File

@ -727,8 +727,10 @@ public class OutboundClientMessageOneShotJob extends JobImpl {
* Anonymity issues? * Anonymity issues?
*/ */
private TunnelInfo selectOutboundTunnel() { private TunnelInfo selectOutboundTunnel() {
Hash gw = _lease.getGateway(); // hurts reliability? let's try picking at random again
return getContext().tunnelManager().selectOutboundTunnel(_from.calculateHash(), gw); //Hash gw = _lease.getGateway();
//return getContext().tunnelManager().selectOutboundTunnel(_from.calculateHash(), gw);
return getContext().tunnelManager().selectOutboundTunnel(_from.calculateHash());
} }
/** /**