From fe420f43e5205d421cbdcf05fc9e9172ea2ed465 Mon Sep 17 00:00:00 2001 From: Diana Tuck Date: Mon, 13 Jan 2025 12:37:25 -0800 Subject: [PATCH] Move block outside of parent if/else --- source/service.go | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/source/service.go b/source/service.go index c75c7f57b..368c2474b 100644 --- a/source/service.go +++ b/source/service.go @@ -227,12 +227,14 @@ func (sc *serviceSource) Endpoints(ctx context.Context) ([]*endpoint.Endpoint, e mergedEndpoints[lastMergedEndpoint].SetIdentifier == endpoints[i].SetIdentifier && mergedEndpoints[lastMergedEndpoint].RecordTTL == endpoints[i].RecordTTL { mergedEndpoints[lastMergedEndpoint].Targets = append(mergedEndpoints[lastMergedEndpoint].Targets, endpoints[i].Targets[0]) - } else if mergedEndpoints[lastMergedEndpoint].DNSName == endpoints[i].DNSName && + } else { + mergedEndpoints = append(mergedEndpoints, endpoints[i]) + } + + if mergedEndpoints[lastMergedEndpoint].DNSName == endpoints[i].DNSName && mergedEndpoints[lastMergedEndpoint].RecordType == endpoints[i].RecordType && mergedEndpoints[lastMergedEndpoint].RecordType == endpoint.RecordTypeCNAME { log.Debugf("CNAME %s with multiple targets found", endpoints[i].DNSName) - } else { - mergedEndpoints = append(mergedEndpoints, endpoints[i]) } } endpoints = mergedEndpoints