From cca77709e136a07798b90e4e4ee89229be0f5b33 Mon Sep 17 00:00:00 2001 From: cgoldfeder Date: Wed, 18 May 2016 10:33:39 -0700 Subject: [PATCH] Show the registrant in unauth'ed domain RFC5731 leaves the registrant field as optional. We currently don't show it on un-authed info, but we do show it in WHOIS, so there's no strong reason not to show it here. By doing so, we maintain the invariant that registrant must always be present on a DomainResource object. ------------- Created by MOE: https://github.com/google/moe MOE_MIGRATED_REVID=122645356 --- java/google/registry/flows/domain/DomainInfoFlow.java | 1 + .../flows/domain/testdata/domain_info_response_unauthorized.xml | 1 + 2 files changed, 2 insertions(+) diff --git a/java/google/registry/flows/domain/DomainInfoFlow.java b/java/google/registry/flows/domain/DomainInfoFlow.java index 189946bb1..4ae781d37 100644 --- a/java/google/registry/flows/domain/DomainInfoFlow.java +++ b/java/google/registry/flows/domain/DomainInfoFlow.java @@ -57,6 +57,7 @@ public class DomainInfoFlow extends BaseDomainInfoFlow .setFullyQualifiedDomainName(existingResource.getFullyQualifiedDomainName()) .setRepoId(existingResource.getRepoId()) .setCurrentSponsorClientId(existingResource.getCurrentSponsorClientId()) + .setRegistrant(existingResource.getRegistrant()) // If we didn't do this, we'd get implicit status values. .buildWithoutImplicitStatusValues(); } diff --git a/javatests/google/registry/flows/domain/testdata/domain_info_response_unauthorized.xml b/javatests/google/registry/flows/domain/testdata/domain_info_response_unauthorized.xml index d9a9b3830..c4058770e 100644 --- a/javatests/google/registry/flows/domain/testdata/domain_info_response_unauthorized.xml +++ b/javatests/google/registry/flows/domain/testdata/domain_info_response_unauthorized.xml @@ -8,6 +8,7 @@ xmlns:domain="urn:ietf:params:xml:ns:domain-1.0"> example.tld %ROID% + jd1234 NewRegistrar