[GitHub] [apisix] yxudong opened a new issue #4301: bug: skywalking not work when upsteam timeout

2021-12-07 Thread GitBox


yxudong opened a new issue #4301:
URL: https://github.com/apache/apisix/issues/4301


   ### Issue description
   
   At present, sw_tracer:finish()  will be called in body_filter_by_lua phase.
   But in some case, the http request will not execute body_filter_by_lua 
phase. so this function will not be called.
   
   Detail: https://github.com/apache/skywalking/issues/7004
   
   Is this function should be called in log_by_lua phase?
   
   ### What's the actual result? (including assertion message & call stack if 
applicable)
   
   The upsteam timeout case not work with skywalking plugin.
   
   ### What's the expected result?
   
   The upsteam timeout case could also be work with skywalking plugin.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [apisix] yxudong opened a new issue #4301: bug: skywalking not work when upsteam timeout

2021-05-24 Thread GitBox


yxudong opened a new issue #4301:
URL: https://github.com/apache/apisix/issues/4301


   ### Issue description
   
   At presen, sw_tracer:finish()  will be called in body_filter_by_lua phase.
   But in some case, the http request will not execute body_filter_by_lua 
phase. so this function will not be called.
   
   Detail: https://github.com/apache/skywalking/issues/7004
   
   Is this function should be called in log_by_lua phase?
   
   ### What's the actual result? (including assertion message & call stack if 
applicable)
   
   The upsteam timeout case not work with skywalking plugin.
   
   ### What's the expected result?
   
   The upsteam timeout case could also be work with skywalking plugin.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org