Fixes classification of errors in frontend

This fixes errors being classified as server errors (HTTP 5XX errors) when a domain is deprecated and correctly reclassifies them as application/ 4XX errors
agautam478 · May 13, 2024 · 3c59510 · 3c59510
1 parent ad0c027
commit 3c59510
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/service/frontend/wrappers/clusterredirection/policy.go b/service/frontend/wrappers/clusterredirection/policy.go
@@ -195,7 +195,7 @@ func (policy *selectedOrAllAPIsForwardingRedirectionPolicy) WithDomainIDRedirect
 		return err
 	}
 	if domainEntry.IsDeprecatedOrDeleted() {
-		return types.DomainNotActiveError{
+		return &types.DomainNotActiveError{
 			Message:        "domain is deprecated.",
 			DomainName:     domainEntry.GetInfo().Name,
 			CurrentCluster: policy.currentClusterName,
@@ -212,7 +212,7 @@ func (policy *selectedOrAllAPIsForwardingRedirectionPolicy) WithDomainNameRedire
 		return err
 	}
 	if domainEntry.IsDeprecatedOrDeleted() {
-		return types.DomainNotActiveError{
+		return &types.DomainNotActiveError{
 			Message:        "domain is deprecated or deleted",
 			DomainName:     domainName,
 			CurrentCluster: policy.currentClusterName,