diff --git a/java/google/registry/tools/CreateOrUpdateRegistrarCommand.java b/java/google/registry/tools/CreateOrUpdateRegistrarCommand.java index 4b2bc3028..d955ff4f0 100644 --- a/java/google/registry/tools/CreateOrUpdateRegistrarCommand.java +++ b/java/google/registry/tools/CreateOrUpdateRegistrarCommand.java @@ -154,7 +154,7 @@ abstract class CreateOrUpdateRegistrarCommand extends MutatingCommand { @Parameter( names = "--ip_whitelist", - description = "Comma-delimited list of IP ranges") + description = "Comma-delimited list of IP ranges. An empty string clears the whitelist.") List ipWhitelist = new ArrayList<>(); @Nullable @@ -332,7 +332,9 @@ abstract class CreateOrUpdateRegistrarCommand extends MutatingCommand { ImmutableList.Builder ipWhitelistBuilder = new ImmutableList.Builder<>(); if (!(ipWhitelist.size() == 1 && ipWhitelist.get(0).contains("null"))) { for (String ipRange : ipWhitelist) { - ipWhitelistBuilder.add(CidrAddressBlock.create(ipRange)); + if (!ipRange.isEmpty()) { + ipWhitelistBuilder.add(CidrAddressBlock.create(ipRange)); + } } } builder.setIpAddressWhitelist(ipWhitelistBuilder.build()); diff --git a/javatests/google/registry/tools/UpdateRegistrarCommandTest.java b/javatests/google/registry/tools/UpdateRegistrarCommandTest.java index 44458fa1f..a911156c5 100644 --- a/javatests/google/registry/tools/UpdateRegistrarCommandTest.java +++ b/javatests/google/registry/tools/UpdateRegistrarCommandTest.java @@ -134,7 +134,7 @@ public class UpdateRegistrarCommandTest extends CommandTestCase