From 62c2679da2992ec585efc3b269a91d849b75326b Mon Sep 17 00:00:00 2001 From: Quentin McGaw Date: Tue, 4 Nov 2025 14:42:54 +0000 Subject: [PATCH] fix(cyberghost): log warnings from updater resolver --- internal/provider/cyberghost/provider.go | 4 ++-- internal/provider/cyberghost/updater/servers.go | 5 ++++- internal/provider/cyberghost/updater/updater.go | 4 +++- internal/provider/providers.go | 2 +- 4 files changed, 10 insertions(+), 5 deletions(-) diff --git a/internal/provider/cyberghost/provider.go b/internal/provider/cyberghost/provider.go index f8bee0ea..f7cd49b4 100644 --- a/internal/provider/cyberghost/provider.go +++ b/internal/provider/cyberghost/provider.go @@ -15,12 +15,12 @@ type Provider struct { } func New(storage common.Storage, randSource rand.Source, - parallelResolver common.ParallelResolver, + updaterWarner common.Warner, parallelResolver common.ParallelResolver, ) *Provider { return &Provider{ storage: storage, randSource: randSource, - Fetcher: updater.New(parallelResolver), + Fetcher: updater.New(parallelResolver, updaterWarner), } } diff --git a/internal/provider/cyberghost/updater/servers.go b/internal/provider/cyberghost/updater/servers.go index d7ed0a24..44f86ed9 100644 --- a/internal/provider/cyberghost/updater/servers.go +++ b/internal/provider/cyberghost/updater/servers.go @@ -16,7 +16,10 @@ func (u *Updater) FetchServers(ctx context.Context, minServers int) ( possibleHosts := possibleServers.hostsSlice() resolveSettings := parallelResolverSettings(possibleHosts) - hostToIPs, _, err := u.parallelResolver.Resolve(ctx, resolveSettings) + hostToIPs, warnings, err := u.parallelResolver.Resolve(ctx, resolveSettings) + for _, warning := range warnings { + u.warner.Warn(warning) + } if err != nil { return nil, err } diff --git a/internal/provider/cyberghost/updater/updater.go b/internal/provider/cyberghost/updater/updater.go index 12134601..b6f3aae5 100644 --- a/internal/provider/cyberghost/updater/updater.go +++ b/internal/provider/cyberghost/updater/updater.go @@ -6,10 +6,12 @@ import ( type Updater struct { parallelResolver common.ParallelResolver + warner common.Warner } -func New(parallelResolver common.ParallelResolver) *Updater { +func New(parallelResolver common.ParallelResolver, warner common.Warner) *Updater { return &Updater{ parallelResolver: parallelResolver, + warner: warner, } } diff --git a/internal/provider/providers.go b/internal/provider/providers.go index 576669f9..3189058c 100644 --- a/internal/provider/providers.go +++ b/internal/provider/providers.go @@ -62,7 +62,7 @@ func NewProviders(storage Storage, timeNow func() time.Time, providerNameToProvider := map[string]Provider{ providers.Airvpn: airvpn.New(storage, randSource, client), providers.Custom: custom.New(extractor), - providers.Cyberghost: cyberghost.New(storage, randSource, parallelResolver), + providers.Cyberghost: cyberghost.New(storage, randSource, updaterWarner, parallelResolver), providers.Expressvpn: expressvpn.New(storage, randSource, unzipper, updaterWarner, parallelResolver), providers.Fastestvpn: fastestvpn.New(storage, randSource, client, updaterWarner, parallelResolver), providers.Giganews: giganews.New(storage, randSource, unzipper, updaterWarner, parallelResolver),