From 7ec599f8499c7d2f6a211a05b85ded389aa4fa54 Mon Sep 17 00:00:00 2001 From: Weimin Yu Date: Tue, 3 Jun 2025 11:17:43 -0400 Subject: [PATCH] Fix create_cdns_tld command (#2760) The Cloud DNS rest api is now case-sensitive about enum names (must be lower case, counterintuitively). --- core/src/main/java/google/registry/tools/CreateCdnsTld.java | 6 +++--- .../test/java/google/registry/tools/CreateCdnsTldTest.java | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/core/src/main/java/google/registry/tools/CreateCdnsTld.java b/core/src/main/java/google/registry/tools/CreateCdnsTld.java index a3aff7db3..cfa8203ff 100644 --- a/core/src/main/java/google/registry/tools/CreateCdnsTld.java +++ b/core/src/main/java/google/registry/tools/CreateCdnsTld.java @@ -72,11 +72,11 @@ final class CreateCdnsTld extends ConfirmingCommand { .setDescription(description) .setNameServerSet( RegistryToolEnvironment.get() == RegistryToolEnvironment.PRODUCTION - ? "cloud-dns-registry" - : "cloud-dns-registry-test") + ? "cloud-dns-registry" + : "cloud-dns-registry-test") .setDnsName(dnsName) .setName((name != null) ? name : dnsName) - .setDnssecConfig(new ManagedZoneDnsSecConfig().setNonExistence("NSEC").setState("ON")); + .setDnssecConfig(new ManagedZoneDnsSecConfig().setNonExistence("nsec").setState("on")); } @Override diff --git a/core/src/test/java/google/registry/tools/CreateCdnsTldTest.java b/core/src/test/java/google/registry/tools/CreateCdnsTldTest.java index bb6f0867e..c4e797dbb 100644 --- a/core/src/test/java/google/registry/tools/CreateCdnsTldTest.java +++ b/core/src/test/java/google/registry/tools/CreateCdnsTldTest.java @@ -55,7 +55,7 @@ class CreateCdnsTldTest extends CommandTestCase { .setDnsName(dnsName) .setDescription(description) .setName(name) - .setDnssecConfig(new ManagedZoneDnsSecConfig().setState("ON").setNonExistence("NSEC")); + .setDnssecConfig(new ManagedZoneDnsSecConfig().setState("on").setNonExistence("nsec")); } @Test