From 2f39574123c352198e33a7b5ff009f83ba4f961e Mon Sep 17 00:00:00 2001
From: zzz <zzz@mail.i2p>
Date: Tue, 21 May 2013 13:39:43 +0000
Subject: [PATCH] log tweaks

---
 .../i2p/router/tunnel/pool/BuildRequestor.java   | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/router/java/src/net/i2p/router/tunnel/pool/BuildRequestor.java b/router/java/src/net/i2p/router/tunnel/pool/BuildRequestor.java
index 06bbaf4905..80bd1803af 100644
--- a/router/java/src/net/i2p/router/tunnel/pool/BuildRequestor.java
+++ b/router/java/src/net/i2p/router/tunnel/pool/BuildRequestor.java
@@ -160,7 +160,7 @@ abstract class BuildRequestor {
         if (cfg.isInbound()) {
             if (log.shouldLog(Log.INFO))
                 log.info("Sending the tunnel build request " + msg.getUniqueId() + " out the tunnel " + pairedTunnel + " to " 
-                          + cfg.getPeer(0).toBase64() + " for " + cfg + " waiting for the reply of "
+                          + cfg.getPeer(0) + " for " + cfg + " waiting for the reply of "
                           + cfg.getReplyMessageId());
             // send it out a tunnel targetting the first hop
             // TODO - would be nice to have a TunnelBuildFirstHopFailJob queued if the
@@ -168,7 +168,7 @@ abstract class BuildRequestor {
             ctx.tunnelDispatcher().dispatchOutbound(msg, pairedTunnel.getSendTunnelId(0), cfg.getPeer(0));
         } else {
             if (log.shouldLog(Log.INFO))
-                log.info("Sending the tunnel build request directly to " + cfg.getPeer(1).toBase64() 
+                log.info("Sending the tunnel build request directly to " + cfg.getPeer(1)
                           + " for " + cfg + " waiting for the reply of " + cfg.getReplyMessageId() 
                           + " with msgId=" + msg.getUniqueId());
             // send it directly to the first hop
@@ -183,8 +183,8 @@ abstract class BuildRequestor {
             outMsg.setPriority(PRIORITY);
             RouterInfo peer = ctx.netDb().lookupRouterInfoLocally(cfg.getPeer(1));
             if (peer == null) {
-                if (log.shouldLog(Log.ERROR))
-                    log.error("Could not find the next hop to send the outbound request to: " + cfg);
+                if (log.shouldLog(Log.WARN))
+                    log.warn("Could not find the next hop to send the outbound request to: " + cfg);
                 exec.buildComplete(cfg, pool);
                 return;
             }
@@ -300,7 +300,7 @@ abstract class BuildRequestor {
                 if (peerInfo == null) {
                     if (log.shouldLog(Log.WARN))
                         log.warn("Peer selected for hop " + i + "/" + hop + " was not found locally: " 
-                                  + peer.toBase64() + " for " + cfg);
+                                  + peer + " for " + cfg);
                     return null;
                 } else {
                     key = peerInfo.getIdentity().getPublicKey();
@@ -343,9 +343,9 @@ abstract class BuildRequestor {
      *  Can't do this for inbound tunnels since the msg goes out an expl. tunnel.
      */
     private static class TunnelBuildFirstHopFailJob extends JobImpl {
-        final TunnelPool _pool;
-        final PooledTunnelCreatorConfig _cfg;
-        final BuildExecutor _exec;
+        private final TunnelPool _pool;
+        private final PooledTunnelCreatorConfig _cfg;
+        private final BuildExecutor _exec;
         private TunnelBuildFirstHopFailJob(RouterContext ctx, TunnelPool pool, PooledTunnelCreatorConfig cfg, BuildExecutor exec) {
             super(ctx);
             _cfg = cfg;
-- 
GitLab