Add req.Data to OnMessage panic error log

apache · Apr 20, 2022 · 7500e28 · 7500e28
1 parent 2373592
commit 7500e28
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/remoting/getty/listener.go b/remoting/getty/listener.go
@@ -269,13 +269,13 @@ func (h *RpcServerHandler) OnMessage(session getty.Session, pkg interface{}) {
 		if e := recover(); e != nil {
 			resp.Status = hessian.Response_SERVER_ERROR
 			if err, ok := e.(error); ok {
-				logger.Errorf("OnMessage panic: %+v", perrors.WithStack(err))
+				logger.Errorf("OnMessage panic: %+v, req: %#v", perrors.WithStack(err), req.Data)
 				resp.Error = perrors.WithStack(err)
 			} else if err, ok := e.(string); ok {
-				logger.Errorf("OnMessage panic: %+v", perrors.New(err))
+				logger.Errorf("OnMessage panic: %+v, req: %#v", perrors.New(err), req.Data)
 				resp.Error = perrors.New(err)
 			} else {
-				logger.Errorf("OnMessage panic: %+v, this is impossible.", e)
+				logger.Errorf("OnMessage panic: %+v, this is impossible. req: %#v", e, req.Data)
 				resp.Error = fmt.Errorf("OnMessage panic unknow exception. %+v", e)
 			}