fix(updater): do not allow or set custom provider

This commit is contained in:
Quentin McGaw (desktop)
2022-01-22 22:23:14 +00:00
parent 779cb18590
commit fdc0db07e0
3 changed files with 16 additions and 2 deletions

View File

@@ -65,7 +65,12 @@ func (c *CLI) Update(ctx context.Context, args []string, logger UpdaterLogger) e
} }
if updateAll { if updateAll {
options.Providers = constants.AllProviders() for _, provider := range constants.AllProviders() {
if provider == constants.Custom {
continue
}
options.Providers = append(options.Providers, provider)
}
} else { } else {
if csvProviders == "" { if csvProviders == "" {
return ErrNoProviderSpecified return ErrNoProviderSpecified

View File

@@ -43,6 +43,10 @@ func (u Updater) Validate() (err error) {
for i, provider := range u.Providers { for i, provider := range u.Providers {
valid := false valid := false
for _, validProvider := range constants.AllProviders() { for _, validProvider := range constants.AllProviders() {
if validProvider == constants.Custom {
continue
}
if provider == validProvider { if provider == validProvider {
valid = true valid = true
break break

View File

@@ -22,7 +22,12 @@ func readUpdater() (updater settings.Updater, err error) {
} }
// TODO use current provider being used // TODO use current provider being used
updater.Providers = constants.AllProviders() for _, provider := range constants.AllProviders() {
if provider == constants.Custom {
continue
}
updater.Providers = append(updater.Providers, provider)
}
return updater, nil return updater, nil
} }