Skip to content

Commit

Permalink
fix comments
Browse files Browse the repository at this point in the history
  • Loading branch information
paulyufan2 committed Apr 29, 2024
1 parent 20a1c09 commit 54d6c61
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 9 deletions.
7 changes: 1 addition & 6 deletions cns/restserver/v2/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -47,10 +47,5 @@ func (s Server) Start(ctx context.Context, addr string) error {

// after context is done, shutdown local server
<-ctx.Done()
if err := e.Shutdown(ctx); err != nil {
logger.Errorf("failed to shutdown echo server due to %+v", err)
return errors.Wrap(err, "failed to shutdown echo server")
}

return nil
return e.Shutdown(ctx)
}
4 changes: 1 addition & 3 deletions cns/service.go
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ func (service *Service) AddListener(config *common.ServiceConfig) error {
}
} else {
// use the URL that customer provides by -c
logger.Warnf("Do not specify cns-url by -c option, this option will be deprecated!")
logger.Printf("CNS remote server url: %+v", nodeURL)

// do not enable local server if customer uses -c option
config.Server.EnableLocalServer = false
Expand All @@ -86,8 +86,6 @@ func (service *Service) AddListener(config *common.ServiceConfig) error {
}
}

logger.Debugf("CNS remote server url: %+v", nodeURL)

nodeListener, err := acn.NewListener(nodeURL)
if err != nil {
return errors.Wrap(err, "Failed to construct url for node listener")
Expand Down

0 comments on commit 54d6c61

Please sign in to comment.