diff --git a/client/cmd/service_controller.go b/client/cmd/service_controller.go index fea3eb39c..75207309f 100644 --- a/client/cmd/service_controller.go +++ b/client/cmd/service_controller.go @@ -20,7 +20,7 @@ import ( func (p *program) Start(svc service.Service) error { // Start should not block. Do the actual work async. - log.Info("starting service") //nolint + log.Info("starting Netbird service") //nolint // in any case, even if configuration does not exists we run daemon to serve CLI gRPC API. p.serv = grpc.NewServer() diff --git a/client/internal/connect.go b/client/internal/connect.go index f83505b24..fd6ed1992 100644 --- a/client/internal/connect.go +++ b/client/internal/connect.go @@ -63,8 +63,9 @@ func RunClient(ctx context.Context, config *Config) error { // connect (just a connection, no stream yet) and login to Management Service to get an initial global Wiretrustee config mgmClient, loginResp, err := connectToManagement(ctx, config.ManagementURL.Host, myPrivateKey, mgmTlsEnabled) if err != nil { - log.Warn(err) + log.Debug(err) if s, ok := status.FromError(err); ok && s.Code() == codes.PermissionDenied { + log.Info("peer registration required. Please run `netbird status` for details") state.Set(StatusNeedsLogin) return nil } @@ -92,11 +93,11 @@ func RunClient(ctx context.Context, config *Config) error { engine := NewEngine(engineCtx, cancel, signalClient, mgmClient, engineConfig) err = engine.Start() if err != nil { - log.Errorf("error while starting Wiretrustee Connection Engine: %s", err) + log.Errorf("error while starting Netbird Connection Engine: %s", err) return wrapErr(err) } - log.Print("Wiretrustee engine started, my IP is: ", peerConfig.Address) + log.Print("Netbird engine started, my IP is: ", peerConfig.Address) state.Set(StatusConnected) <-engineCtx.Done() @@ -120,7 +121,7 @@ func RunClient(ctx context.Context, config *Config) error { return wrapErr(err) } - log.Info("stopped Wiretrustee client") + log.Info("stopped Netbird client") if _, err := state.Status(); err == ErrResetConnection { return err @@ -183,7 +184,7 @@ func connectToSignal(ctx context.Context, wtConfig *mgmProto.WiretrusteeConfig, // connectToManagement creates Management Services client, establishes a connection, logs-in and gets a global Wiretrustee config (signal, turn, stun hosts, etc) func connectToManagement(ctx context.Context, managementAddr string, ourPrivateKey wgtypes.Key, tlsEnabled bool) (*mgm.GrpcClient, *mgmProto.LoginResponse, error) { - log.Debugf("connecting to management server %s", managementAddr) + log.Debugf("connecting to Management Service %s", managementAddr) client, err := mgm.NewClient(ctx, managementAddr, ourPrivateKey, tlsEnabled) if err != nil { return nil, nil, status.Errorf(codes.FailedPrecondition, "failed connecting to Management Service : %s", err) @@ -198,12 +199,7 @@ func connectToManagement(ctx context.Context, managementAddr string, ourPrivateK sysInfo := system.GetInfo(ctx) loginResp, err := client.Login(*serverPublicKey, sysInfo) if err != nil { - if s, ok := status.FromError(err); ok && s.Code() == codes.PermissionDenied { - log.Error("peer registration required. Please run wiretrustee login command first") - return nil, nil, err - } else { - return nil, nil, err - } + return nil, nil, err } log.Debugf("peer logged in to Management Service %s", managementAddr) diff --git a/client/internal/engine.go b/client/internal/engine.go index 361fe3843..b2801d29d 100644 --- a/client/internal/engine.go +++ b/client/internal/engine.go @@ -38,7 +38,7 @@ type EngineConfig struct { WgPort int WgIfaceName string - // WgAddr is a Wireguard local address (Wiretrustee Network IP) + // WgAddr is a Wireguard local address (Netbird Network IP) WgAddr string // WgPrivateKey is a Wireguard private key of our peer (it MUST never leave the machine) @@ -127,11 +127,11 @@ func (e *Engine) Stop() error { // Removing peers happens in the conn.CLose() asynchronously time.Sleep(500 * time.Millisecond) - log.Debugf("removing Wiretrustee interface %s", e.config.WgIfaceName) + log.Debugf("removing Netbird interface %s", e.config.WgIfaceName) if e.wgInterface.Interface != nil { err = e.wgInterface.Close() if err != nil { - log.Errorf("failed closing Wiretrustee interface %s %v", e.config.WgIfaceName, err) + log.Errorf("failed closing Netbird interface %s %v", e.config.WgIfaceName, err) return err } } @@ -160,7 +160,7 @@ func (e *Engine) Stop() error { } } - log.Infof("stopped Wiretrustee Engine") + log.Infof("stopped Netbird Engine") return nil }