Fixing panic caused by assignment happening before err check (#263)

otterize · Jan 5, 2025 · 89ba1b1 · 89ba1b1
1 parent e832117
commit 89ba1b1
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/mapper/pkg/resolvers/schema.helpers.resolvers.go b/src/mapper/pkg/resolvers/schema.helpers.resolvers.go
@@ -446,13 +446,13 @@ func (r *Resolver) resolveOtterizeIdentityForExternalAccessDestination(ctx conte
 	}
 
 	dstSvcIdentity, ok, err := r.kubeFinder.ResolveOtterizeIdentityForService(ctx, destService, dest.LastSeen)
-	dstSvcIdentity.ResolutionData.Host = lo.ToPtr(destIP)
 	if err != nil {
 		return model.OtterizeServiceIdentity{}, false, errors.Wrap(err)
 	}
 	if !ok {
 		return model.OtterizeServiceIdentity{}, false, nil
 	}
+	dstSvcIdentity.ResolutionData.Host = lo.ToPtr(destIP)
 
 	return dstSvcIdentity, true, nil
 }