update micro version and add gracefulTimeout && update log msg
This commit is contained in:
		
							
								
								
									
										6
									
								
								go.mod
									
									
									
									
									
								
							
							
						
						
									
										6
									
								
								go.mod
									
									
									
									
									
								
							| @@ -5,9 +5,9 @@ go 1.19 | ||||
| require ( | ||||
| 	go.unistack.org/micro-codec-yaml/v4 v4.0.0 | ||||
| 	go.unistack.org/micro-proto/v4 v4.0.1 | ||||
| 	go.unistack.org/micro/v4 v4.0.6 | ||||
| 	go.unistack.org/micro/v4 v4.0.17 | ||||
| 	go.unistack.org/protoc-gen-go-micro/v4 v4.0.7 | ||||
| 	golang.org/x/net v0.12.0 | ||||
| 	golang.org/x/net v0.15.0 | ||||
| ) | ||||
|  | ||||
| require ( | ||||
| @@ -15,7 +15,7 @@ require ( | ||||
| 	github.com/golang/protobuf v1.5.3 // indirect | ||||
| 	github.com/google/gnostic v0.6.9 // indirect | ||||
| 	golang.org/x/mod v0.12.0 // indirect | ||||
| 	golang.org/x/sys v0.10.0 // indirect | ||||
| 	golang.org/x/sys v0.12.0 // indirect | ||||
| 	golang.org/x/tools v0.11.0 // indirect | ||||
| 	google.golang.org/protobuf v1.31.0 // indirect | ||||
| 	gopkg.in/yaml.v2 v2.4.0 // indirect | ||||
|   | ||||
							
								
								
									
										12
									
								
								go.sum
									
									
									
									
									
								
							
							
						
						
									
										12
									
								
								go.sum
									
									
									
									
									
								
							| @@ -73,8 +73,8 @@ go.unistack.org/micro-codec-yaml/v4 v4.0.0 h1:DK/+fA69gmnvYmGHe3kqVR0pukS8ank5lP | ||||
| go.unistack.org/micro-codec-yaml/v4 v4.0.0/go.mod h1:UG4ouILbUfgNL7nvp7CpmD2IIzO8bQHjzDIahOvIAgM= | ||||
| go.unistack.org/micro-proto/v4 v4.0.1 h1:2RKHgtCOOcAFgKsnngGK5bqM/6MWXOjVCdw03dbuoF8= | ||||
| go.unistack.org/micro-proto/v4 v4.0.1/go.mod h1:ArmK7o+uFvxSY3dbJhKBBX4Pm1rhWdLEFf3LxBrMtec= | ||||
| go.unistack.org/micro/v4 v4.0.6 h1:YFWvTh3VwyOd6NHYTQcf47n2TF5+p/EhpnbuBQX3qhk= | ||||
| go.unistack.org/micro/v4 v4.0.6/go.mod h1:bVEYTlPi0EsdgZZt311bIroDg9ict7ky3C87dSCCAGk= | ||||
| go.unistack.org/micro/v4 v4.0.17 h1:mF7uM+J4ILdG+1fcwzKYCwDlxhdbF/e1WnGzKKLnIXc= | ||||
| go.unistack.org/micro/v4 v4.0.17/go.mod h1:ZDgU9931vm2l7X6RN/6UuwRIVp24GRdmQ7dKmegArk4= | ||||
| go.unistack.org/protoc-gen-go-micro/v4 v4.0.7 h1:c3CeBvjWV9MBrGGPaQCcHZrnuLY9sVJHv7vACI/dvuM= | ||||
| go.unistack.org/protoc-gen-go-micro/v4 v4.0.7/go.mod h1:9bsKAlESlPXPBSmY/NDLL//smZbhnEMrnWyG+M8zVFA= | ||||
| golang.org/x/crypto v0.0.0-20190308221718-c2843e01d9a2/go.mod h1:djNgcEr1/C05ACkg1iLfiJU5Ep61QUkGW8qpdssI0+w= | ||||
| @@ -94,8 +94,8 @@ golang.org/x/net v0.0.0-20190404232315-eb5bcb51f2a3/go.mod h1:t9HGtf8HONx5eT2rtn | ||||
| golang.org/x/net v0.0.0-20200822124328-c89045814202/go.mod h1:/O7V0waA8r7cgGh81Ro3o1hOxt32SMVPicZroKQ2sZA= | ||||
| golang.org/x/net v0.0.0-20210405180319-a5a99cb37ef4/go.mod h1:p54w0d4576C0XHj96bSt6lcn1PtDYWL6XObtHCRCNQM= | ||||
| golang.org/x/net v0.0.0-20210805182204-aaa1db679c0d/go.mod h1:9nx3DQGgdP8bBQD5qxJ1jj9UTztislL4KSBs9R2vV5Y= | ||||
| golang.org/x/net v0.12.0 h1:cfawfvKITfUsFCeJIHJrbSxpeu/E81khclypR0GVT50= | ||||
| golang.org/x/net v0.12.0/go.mod h1:zEVYFnQC7m/vmpQFELhcD1EWkZlX69l4oqgmer6hfKA= | ||||
| golang.org/x/net v0.15.0 h1:ugBLEUaxABaB5AJqW9enI0ACdci2RUd4eP51NTBvuJ8= | ||||
| golang.org/x/net v0.15.0/go.mod h1:idbUs1IY1+zTqbi8yxTbhexhEEk5ur9LInksu6HrEpk= | ||||
| golang.org/x/oauth2 v0.0.0-20180821212333-d2e6202438be/go.mod h1:N/0e6XlmueqKjAGxoOufVs8QHGRruUQn6yWY3a++T0U= | ||||
| golang.org/x/oauth2 v0.0.0-20200107190931-bf48bf16ab8d/go.mod h1:gOpvHmFTYa4IltrdGE7lF6nIHvwfUNPOp7c8zoXwtLw= | ||||
| golang.org/x/sync v0.0.0-20180314180146-1d60e4601c6f/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM= | ||||
| @@ -111,8 +111,8 @@ golang.org/x/sys v0.0.0-20201119102817-f84b799fce68/go.mod h1:h1NjWce9XRLGQEsW7w | ||||
| golang.org/x/sys v0.0.0-20210330210617-4fbd30eecc44/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs= | ||||
| golang.org/x/sys v0.0.0-20210423082822-04245dca01da/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs= | ||||
| golang.org/x/sys v0.0.0-20210510120138-977fb7262007/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg= | ||||
| golang.org/x/sys v0.10.0 h1:SqMFp9UcQJZa+pmYuAKjd9xq1f0j5rLcDIk0mj4qAsA= | ||||
| golang.org/x/sys v0.10.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg= | ||||
| golang.org/x/sys v0.12.0 h1:CM0HF96J0hcLAwsHPJZjfdNzs0gftsLfgKt57wWHJ0o= | ||||
| golang.org/x/sys v0.12.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg= | ||||
| golang.org/x/term v0.0.0-20201126162022-7de9c90e9dd1/go.mod h1:bj7SfCRtBDWHUb9snDiAeCFNEtKQo2Wmx5Cou7ajbmo= | ||||
| golang.org/x/text v0.3.0/go.mod h1:NqM8EUOU14njkJ3fqMW+pc6Ldnwhi/IjpwHt7yyuwOQ= | ||||
| golang.org/x/text v0.3.3/go.mod h1:5Zoc/QRtKVWzQhOtBMvqHzDpF6irO9z98xDceosuGiQ= | ||||
|   | ||||
							
								
								
									
										55
									
								
								http.go
									
									
									
									
									
								
							
							
						
						
									
										55
									
								
								http.go
									
									
									
									
									
								
							| @@ -198,16 +198,16 @@ func (h *Server) newHTTPHandler(handler interface{}, opts ...options.Option) *ht | ||||
| 		} | ||||
|  | ||||
| 		if method.Name == "" && h.opts.Logger.V(logger.ErrorLevel) { | ||||
| 			h.opts.Logger.Errorf(h.opts.Context, "nil method for %s", mname) | ||||
| 			h.opts.Logger.Error(h.opts.Context, fmt.Sprintf("nil method for %s", mname)) | ||||
| 			continue | ||||
| 		} | ||||
|  | ||||
| 		mtype, err := prepareEndpoint(method) | ||||
| 		if err != nil && h.opts.Logger.V(logger.ErrorLevel) { | ||||
| 			h.opts.Logger.Errorf(h.opts.Context, "%v", err) | ||||
| 			h.opts.Logger.Error(h.opts.Context, fmt.Sprintf("%v", err)) | ||||
| 			continue | ||||
| 		} else if mtype == nil { | ||||
| 			h.opts.Logger.Errorf(h.opts.Context, "nil mtype for %s", mname) | ||||
| 			h.opts.Logger.Error(h.opts.Context, fmt.Sprintf("nil mtype for %s", mname)) | ||||
| 			continue | ||||
| 		} | ||||
|  | ||||
| @@ -218,13 +218,13 @@ func (h *Server) newHTTPHandler(handler interface{}, opts ...options.Option) *ht | ||||
| 		hdlr.name = name | ||||
|  | ||||
| 		if err := hdlr.handlers.Insert([]string{md["Method"]}, md["Path"], pth); err != nil { | ||||
| 			h.opts.Logger.Errorf(h.opts.Context, "cant add handler for %s %s", md["Method"], md["Path"]) | ||||
| 			h.opts.Logger.Error(h.opts.Context, fmt.Sprintf("cant add handler for %s %s", md["Method"], md["Path"])) | ||||
| 		} | ||||
|  | ||||
| 		if h.registerRPC { | ||||
| 			h.opts.Logger.Infof(h.opts.Context, "register rpc handler for http.MethodPost %s /%s", hn, hn) | ||||
| 			h.opts.Logger.Info(h.opts.Context, fmt.Sprintf("register rpc handler for http.MethodPost %s /%s", hn, hn)) | ||||
| 			if err := hdlr.handlers.Insert([]string{http.MethodPost}, "/"+hn, pth); err != nil { | ||||
| 				h.opts.Logger.Errorf(h.opts.Context, "cant add rpc handler for http.MethodPost %s /%s", hn, hn) | ||||
| 				h.opts.Logger.Error(h.opts.Context, fmt.Sprintf("cant add rpc handler for http.MethodPost %s /%s", hn, hn)) | ||||
| 			} | ||||
| 		} | ||||
| 	} | ||||
| @@ -248,16 +248,16 @@ func (h *Server) newHTTPHandler(handler interface{}, opts ...options.Option) *ht | ||||
| 		} | ||||
|  | ||||
| 		if method.Name == "" && h.opts.Logger.V(logger.ErrorLevel) { | ||||
| 			h.opts.Logger.Errorf(h.opts.Context, "nil method for %s", mname) | ||||
| 			h.opts.Logger.Error(h.opts.Context, fmt.Sprintf("nil method for %s", mname)) | ||||
| 			continue | ||||
| 		} | ||||
|  | ||||
| 		mtype, err := prepareEndpoint(method) | ||||
| 		if err != nil && h.opts.Logger.V(logger.ErrorLevel) { | ||||
| 			h.opts.Logger.Errorf(h.opts.Context, "%v", err) | ||||
| 			h.opts.Logger.Error(h.opts.Context, fmt.Sprintf("%v", err)) | ||||
| 			continue | ||||
| 		} else if mtype == nil { | ||||
| 			h.opts.Logger.Errorf(h.opts.Context, "nil mtype for %s", mname) | ||||
| 			h.opts.Logger.Error(h.opts.Context, fmt.Sprintf("nil mtype for %s", mname)) | ||||
| 			continue | ||||
| 		} | ||||
|  | ||||
| @@ -268,13 +268,13 @@ func (h *Server) newHTTPHandler(handler interface{}, opts ...options.Option) *ht | ||||
| 		hdlr.name = name | ||||
|  | ||||
| 		if err := hdlr.handlers.Insert([]string{md.Method}, md.Path, pth); err != nil { | ||||
| 			h.opts.Logger.Errorf(h.opts.Context, "cant add handler for %s %s", md.Method, md.Path) | ||||
| 			h.opts.Logger.Error(h.opts.Context, fmt.Sprintf("cant add handler for %s %s", md.Method, md.Path)) | ||||
| 		} | ||||
|  | ||||
| 		if h.registerRPC { | ||||
| 			h.opts.Logger.Infof(h.opts.Context, "register rpc handler for http.MethodPost %s /%s", hn, hn) | ||||
| 			h.opts.Logger.Info(h.opts.Context, fmt.Sprintf("register rpc handler for http.MethodPost %s /%s", hn, hn)) | ||||
| 			if err := hdlr.handlers.Insert([]string{http.MethodPost}, "/"+hn, pth); err != nil { | ||||
| 				h.opts.Logger.Errorf(h.opts.Context, "cant add rpc handler for http.MethodPost %s /%s", hn, hn) | ||||
| 				h.opts.Logger.Error(h.opts.Context, fmt.Sprintf("cant add rpc handler for http.MethodPost %s /%s", hn, hn)) | ||||
| 			} | ||||
| 		} | ||||
| 	} | ||||
| @@ -317,7 +317,7 @@ func (h *Server) Register() error { | ||||
|  | ||||
| 	if !registered { | ||||
| 		if config.Logger.V(logger.InfoLevel) { | ||||
| 			config.Logger.Infof(config.Context, "Register [%s] Registering node: %s", config.Register.String(), service.Nodes[0].ID) | ||||
| 			config.Logger.Info(config.Context, fmt.Sprintf("Register [%s] Registering node: %s", config.Register.String(), service.Nodes[0].ID)) | ||||
| 		} | ||||
| 	} | ||||
|  | ||||
| @@ -350,7 +350,7 @@ func (h *Server) Deregister() error { | ||||
| 	} | ||||
|  | ||||
| 	if config.Logger.V(logger.InfoLevel) { | ||||
| 		config.Logger.Infof(config.Context, "Deregistering node: %s", service.Nodes[0].ID) | ||||
| 		config.Logger.Info(config.Context, fmt.Sprintf("Deregistering node: %s", service.Nodes[0].ID)) | ||||
| 	} | ||||
|  | ||||
| 	if err := server.DefaultDeregisterFunc(service, config); err != nil { | ||||
| @@ -400,7 +400,7 @@ func (h *Server) Start() error { | ||||
| 	} | ||||
|  | ||||
| 	if config.Logger.V(logger.InfoLevel) { | ||||
| 		config.Logger.Infof(config.Context, "Listening on %s", ts.Addr().String()) | ||||
| 		config.Logger.Info(config.Context, fmt.Sprintf("Listening on %s", ts.Addr().String())) | ||||
| 	} | ||||
|  | ||||
| 	h.Lock() | ||||
| @@ -440,7 +440,7 @@ func (h *Server) Start() error { | ||||
|  | ||||
| 	if err := config.RegisterCheck(h.opts.Context); err != nil { | ||||
| 		if config.Logger.V(logger.ErrorLevel) { | ||||
| 			config.Logger.Errorf(config.Context, "Server %s-%s register check error: %s", config.Name, config.ID, err) | ||||
| 			config.Logger.Error(config.Context, fmt.Sprintf("Server %s-%s register check error: %s", config.Name, config.ID, err)) | ||||
| 		} | ||||
| 	} else { | ||||
| 		if err = h.Register(); err != nil { | ||||
| @@ -468,7 +468,7 @@ func (h *Server) Start() error { | ||||
|  | ||||
| 	go func() { | ||||
| 		if cerr := hs.Serve(ts); cerr != nil && !errors.Is(cerr, net.ErrClosed) { | ||||
| 			h.opts.Logger.Error(h.opts.Context, cerr) | ||||
| 			h.opts.Logger.Error(h.opts.Context, fmt.Sprint(cerr)) | ||||
| 		} | ||||
| 	}() | ||||
|  | ||||
| @@ -496,28 +496,28 @@ func (h *Server) Start() error { | ||||
| 				// nolint: nestif | ||||
| 				if rerr != nil && registered { | ||||
| 					if config.Logger.V(logger.ErrorLevel) { | ||||
| 						config.Logger.Errorf(config.Context, "Server %s-%s register check error: %s, deregister it", config.Name, config.ID, rerr) | ||||
| 						config.Logger.Error(config.Context, fmt.Sprintf("Server %s-%s register check error: %s, deregister it", config.Name, config.ID, rerr)) | ||||
| 					} | ||||
| 					// deregister self in case of error | ||||
| 					if err := h.Deregister(); err != nil { | ||||
| 						if config.Logger.V(logger.ErrorLevel) { | ||||
| 							config.Logger.Errorf(config.Context, "Server %s-%s deregister error: %s", config.Name, config.ID, err) | ||||
| 							config.Logger.Error(config.Context, fmt.Sprintf("Server %s-%s deregister error: %s", config.Name, config.ID, err)) | ||||
| 						} | ||||
| 					} | ||||
| 				} else if rerr != nil && !registered { | ||||
| 					if config.Logger.V(logger.ErrorLevel) { | ||||
| 						config.Logger.Errorf(config.Context, "Server %s-%s register check error: %s", config.Name, config.ID, rerr) | ||||
| 						config.Logger.Error(config.Context, fmt.Sprintf("Server %s-%s register check error: %s", config.Name, config.ID, rerr)) | ||||
| 					} | ||||
| 					continue | ||||
| 				} | ||||
| 				if err := h.Register(); err != nil { | ||||
| 					if config.Logger.V(logger.ErrorLevel) { | ||||
| 						config.Logger.Errorf(config.Context, "Server %s-%s register error: %s", config.Name, config.ID, err) | ||||
| 						config.Logger.Error(config.Context, fmt.Sprintf("Server %s-%s register error: %s", config.Name, config.ID, err)) | ||||
| 					} | ||||
| 				} | ||||
|  | ||||
| 				if err := h.Register(); err != nil { | ||||
| 					config.Logger.Errorf(config.Context, "Server register error: %s", err) | ||||
| 					config.Logger.Error(config.Context, fmt.Sprintf("Server register error: %s", err)) | ||||
| 				} | ||||
| 			// wait for exit | ||||
| 			case ch = <-h.exit: | ||||
| @@ -527,12 +527,15 @@ func (h *Server) Start() error { | ||||
|  | ||||
| 		// deregister | ||||
| 		if err := h.Deregister(); err != nil { | ||||
| 			config.Logger.Errorf(config.Context, "Server deregister error: %s", err) | ||||
| 			config.Logger.Error(config.Context, fmt.Sprintf("Server deregister error: %s", err)) | ||||
| 		} | ||||
|  | ||||
| 		//  empty context | ||||
| 		// TODO ts.Listener is sopped? | ||||
| 		ch <- hs.Shutdown(context.Background()) | ||||
| 		ctx, cancel := context.WithTimeout(context.Background(), h.opts.GracefulTimeout) | ||||
| 		defer cancel() | ||||
|  | ||||
| 		if err := hs.Shutdown(ctx); err != nil { | ||||
| 			ch <- hs.Close() | ||||
| 		} | ||||
| 	}() | ||||
|  | ||||
| 	return nil | ||||
|   | ||||
		Reference in New Issue
	
	Block a user