This is an automated email from the ASF dual-hosted git repository.

alexstocks pushed a commit to branch 3.0
in repository https://gitbox.apache.org/repos/asf/dubbo-go.git


The following commit(s) were added to refs/heads/3.0 by this push:
     new 3be2d9806 Add req.Data to OnMessage panic error log (#1847)
3be2d9806 is described below

commit 3be2d98061f80c18975289c7694f194aa6b5f2b7
Author: shenchao861129 <[email protected]>
AuthorDate: Wed Apr 20 13:54:03 2022 +0800

    Add req.Data to OnMessage panic error log (#1847)
    
    Co-authored-by: shenchao <[email protected]>
---
 remoting/getty/listener.go | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/remoting/getty/listener.go b/remoting/getty/listener.go
index c79f24299..f9d510766 100644
--- 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)
                        }
 

Reply via email to