From f84d29a9dbbad6dd2d7b52dc3718ea961dca9d7a Mon Sep 17 00:00:00 2001
From: yijiem <yijiem@google.com>
Date: Tue, 19 Dec 2023 13:14:17 -0800
Subject: [PATCH] return error instead of assertion when hostname is empty in
 cf_engine/dns_service_resolver

---
 .../lib/event_engine/cf_engine/dns_service_resolver.cc   | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/src/core/lib/event_engine/cf_engine/dns_service_resolver.cc b/src/core/lib/event_engine/cf_engine/dns_service_resolver.cc
index a202e815abe..fc02cbd566a 100644
--- a/src/core/lib/event_engine/cf_engine/dns_service_resolver.cc
+++ b/src/core/lib/event_engine/cf_engine/dns_service_resolver.cc
@@ -50,7 +50,14 @@ void DNSServiceResolverImpl::LookupHostname(
     });
     return;
   }
-  GPR_ASSERT(!host.empty());
+  if (host.empty()) {
+    engine_->Run([on_resolve = std::move(on_resolve),
+                  status = absl::InvalidArgumentError(absl::StrCat(
+                      "host must not be empty in name: ", name))]() mutable {
+      on_resolve(status);
+    });
+    return;
+  }
   if (port_string.empty()) {
     if (default_port.empty()) {
       engine_->Run([on_resolve = std::move(on_resolve),