From 5294f5be35a5541514e7ae4afbc5bf434c6911ce Mon Sep 17 00:00:00 2001 From: mlsmaycon Date: Thu, 27 Aug 2026 10:25:17 +0000 Subject: [PATCH] Reuse filterEnabledProviders in the authorized-provider selection --- .../internals/modules/agentnetwork/setup.go | 17 +++++------------ 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/management/internals/modules/agentnetwork/setup.go b/management/internals/modules/agentnetwork/setup.go index 53e7ae47d..282aa08df 100644 --- a/management/internals/modules/agentnetwork/setup.go +++ b/management/internals/modules/agentnetwork/setup.go @@ -3,7 +3,6 @@ package agentnetwork import ( "context" "fmt" - "sort" "github.com/netbirdio/netbird/management/internals/modules/agentnetwork/catalog" "github.com/netbirdio/netbird/management/internals/modules/agentnetwork/types" @@ -117,22 +116,16 @@ func (m *managerImpl) authorizedProvidersForGroups(ctx context.Context, accountI return nil, nil, fmt.Errorf("list account providers: %w", err) } - authorized := make([]*types.Provider, 0, len(providers)) - for _, p := range providers { - if p == nil || !p.Enabled { - continue - } + // filterEnabledProviders carries the enabled filter and the + // created_at/ID order shared with the router synthesizer. + enabled := filterEnabledProviders(providers) + authorized := make([]*types.Provider, 0, len(enabled)) + for _, p := range enabled { if len(policiesForProvider(applicable, p.ID)) == 0 { continue } authorized = append(authorized, p) } - sort.SliceStable(authorized, func(i, j int) bool { - if !authorized[i].CreatedAt.Equal(authorized[j].CreatedAt) { - return authorized[i].CreatedAt.Before(authorized[j].CreatedAt) - } - return authorized[i].ID < authorized[j].ID - }) return authorized, applicable, nil }