[GitHub] [apisix] wu-sheng commented on issue #4301: bug: skywalking not work when upsteam timeout

2021-12-29 Thread GitBox
wu-sheng commented on issue #4301: URL: https://github.com/apache/apisix/issues/4301#issuecomment-1002896435 Should APISIX upgrade the dependency for the next release? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

[GitHub] [apisix] wu-sheng commented on issue #4301: bug: skywalking not work when upsteam timeout

2021-12-16 Thread GitBox
wu-sheng commented on issue #4301: URL: https://github.com/apache/apisix/issues/4301#issuecomment-996378124 @dmsolr Any update about this? -- 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 spe

[GitHub] [apisix] wu-sheng commented on issue #4301: bug: skywalking not work when upsteam timeout

2021-12-07 Thread GitBox
wu-sheng commented on issue #4301: URL: https://github.com/apache/apisix/issues/4301#issuecomment-988548156 ![image](https://user-images.githubusercontent.com/5441976/145162308-af4de142-2ccc-4581-b93d-1c6d15a01b3c.png) 2 fixes are code related. It is 7 months since last release, if y

[GitHub] [apisix] wu-sheng commented on issue #4301: bug: skywalking not work when upsteam timeout

2021-06-02 Thread GitBox
wu-sheng commented on issue #4301: URL: https://github.com/apache/apisix/issues/4301#issuecomment-853027025 https://github.com/apache/skywalking-nginx-lua/pull/85 merged. We will need skywalking's next release for APISIX fix. -- This is an automated message from the Apache Git Service. T

[GitHub] [apisix] wu-sheng commented on issue #4301: bug: skywalking not work when upsteam timeout

2021-05-25 Thread GitBox
wu-sheng commented on issue #4301: URL: https://github.com/apache/apisix/issues/4301#issuecomment-847842853 You should know, in other SkyWalking agents, there is a config to control collecting HTTP request and response bodies when necessary. It is better to keep `body_filter_by_lua` works

[GitHub] [apisix] wu-sheng commented on issue #4301: bug: skywalking not work when upsteam timeout

2021-05-25 Thread GitBox
wu-sheng commented on issue #4301: URL: https://github.com/apache/apisix/issues/4301#issuecomment-847842189 Does `header_filter_by_lua` work for timeout? -- 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 t

[GitHub] [apisix] wu-sheng commented on issue #4301: bug: skywalking not work when upsteam timeout

2021-05-24 Thread GitBox
wu-sheng commented on issue #4301: URL: https://github.com/apache/apisix/issues/4301#issuecomment-847576227 > My opinion: > > 1. normal response: keep the current behavior > 2. without response (the timed out case pointed by @yxudong): add `finish` call if we haven't called it yet

[GitHub] [apisix] wu-sheng commented on issue #4301: bug: skywalking not work when upsteam timeout

2021-05-24 Thread GitBox
wu-sheng commented on issue #4301: URL: https://github.com/apache/apisix/issues/4301#issuecomment-847536953 > Response without body also causes this problem, like HEAD request's response. Thanks for sharing more cases. What is the recommended fix? Do we have to move this into the `l

[GitHub] [apisix] wu-sheng commented on issue #4301: bug: skywalking not work when upsteam timeout

2021-05-24 Thread GitBox
wu-sheng commented on issue #4301: URL: https://github.com/apache/apisix/issues/4301#issuecomment-847469523 > We may add a flag to check whether the `sw_tracer:finish` was called in the `body_by_lua_filter*` phase, if not, just calling it in `log_by_lua*` phase. `log` method should n

[GitHub] [apisix] wu-sheng commented on issue #4301: bug: skywalking not work when upsteam timeout

2021-05-24 Thread GitBox
wu-sheng commented on issue #4301: URL: https://github.com/apache/apisix/issues/4301#issuecomment-847112213 Hi @apache/apisix-committers SkyWalking community needs verification of whether this is the right description, and a correct way to fix -- This is an automated message from the Ap