From 60022ae3156daa80e8322345b587b83ddcc85a10 Mon Sep 17 00:00:00 2001 From: nickfelt Date: Mon, 5 Jun 2017 14:40:02 -0700 Subject: [PATCH] Remove outdated comment in RdeUploadActionTest I think this comment was meant to be the justification for not using "localhost" (aka a hostname) in the URLs, because jsch would mangle it. However, we already cut over to using "localhost" in [] to avoid a dependency on IPv4, and it's been fine. So this comment no longer makes sense. ------------- Created by MOE: https://github.com/google/moe MOE_MIGRATED_REVID=158063880 --- javatests/google/registry/rde/RdeUploadActionTest.java | 2 -- 1 file changed, 2 deletions(-) diff --git a/javatests/google/registry/rde/RdeUploadActionTest.java b/javatests/google/registry/rde/RdeUploadActionTest.java index bb4b238e4..81b788f33 100644 --- a/javatests/google/registry/rde/RdeUploadActionTest.java +++ b/javatests/google/registry/rde/RdeUploadActionTest.java @@ -278,8 +278,6 @@ public class RdeUploadActionTest { @Test public void testRunWithLock_succeedsOnThirdTry() throws Exception { - // XXX: For any port other than 22, JSch will reformat the hostname IPv6 style which causes - // known host matching to fail. int port = sftpd.serve("user", "password", folder.getRoot()); URI uploadUrl = URI.create(String.format("sftp://user:password@localhost:%d/", port)); DateTime stagingCursor = DateTime.parse("2010-10-18TZ");