all: update outdated comments about OptionsUpdater interface (#1207)

In #1088, OptionsUpdater was removed, but current code still mention it.
This commit updates all comments which still mention about that
interface (authorize is exlcuded, and will be updated in #1206).
This commit is contained in:
Cuong Manh Le 2020-08-05 21:39:24 +07:00 committed by GitHub
parent 633c25feb7
commit 73abed0d21
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 4 additions and 6 deletions

View file

@ -226,8 +226,7 @@ func (a *Authenticate) setAdminUsers(opts *config.Options) {
}
}
// OnConfigChange implements the OptionsUpdater interface and updates internal
// structures based on config.Options
// OnConfigChange updates internal structures based on config.Options
func (a *Authenticate) OnConfigChange(cfg *config.Config) {
if a == nil {
return

View file

@ -716,8 +716,8 @@ func (o *Options) Checksum() uint64 {
}
// handleConfigUpdate takes configuration file, an existing options struct, and
// updates each service in the services slice OptionsUpdater with a new set of
// options if any change is detected.
// returns new options if any change is detected. If no change was detected, the
// existing option will be returned.
func handleConfigUpdate(configFile string, opt *Options) *Options {
serviceName := telemetry.ServiceName(opt.Services)

View file

@ -169,8 +169,7 @@ func New(opts config.Options) (*Proxy, error) {
return p, nil
}
// UpdateOptions implements the OptionsUpdater interface and updates internal
// structures based on config.Options
// UpdateOptions updates internal structures based on config.Options
func (p *Proxy) UpdateOptions(o config.Options) error {
if p == nil {
return nil