Prevent adding www when domain nameserver is unknown

Author: TrySound

apps/builder/app/builder/features/topbar/add-domain.tsx

@@ -54,7 +54,7 @@ export const AddDomain = ({
       });
       // enforce www subdomain when no support for cname flattening
       // and root cname can conflict with MX or NS
-      if (!registrar.cnameFlattening) {
+      if (registrar.known && !registrar.cnameFlattening) {
         domain = `www.${domain}`;
       }
     }

packages/domain/src/rdap.ts

@@ -91,15 +91,14 @@ export const isDomainUsingCloudflareNameservers = async (domain: string) => {
     console.error(
       "RDAP Server for the given top level domain could not be found."
     );
+    return undefined;
   }
 
-  if (rdapServer) {
-    const data = await fetchRdap(rdapServer, domain);
-    if (data) {
-      // detect by nameservers rather than registrar url
-      // sometimes stored as *.NS.CLOUDFLARE.COM
-      return data.toLowerCase().includes(".ns.cloudflare.com");
-    }
+  const data = await fetchRdap(rdapServer, domain);
+  if (data) {
+    // detect by nameservers rather than registrar url
+    // sometimes stored as *.NS.CLOUDFLARE.COM
+    return data.toLowerCase().includes(".ns.cloudflare.com");
   }
   return false;
 };

packages/domain/src/trpc/domain.ts

@@ -28,8 +28,12 @@ export const domainRouter = router({
   findDomainRegistrar: procedure
     .input(z.object({ domain: z.string() }))
     .query(async ({ input }) => {
+      const isCloudflare = await isDomainUsingCloudflareNameservers(
+        input.domain
+      );
       return {
-        cnameFlattening: await isDomainUsingCloudflareNameservers(input.domain),
+        known: isCloudflare !== undefined,
+        cnameFlattening: isCloudflare === true,
       };
     }),