moonming commented on issue #1614:
URL: 
https://github.com/apache/incubator-apisix/issues/1614#issuecomment-634550451


   good catch.
   `upstram_uri` was set in 
https://github.com/apache/incubator-apisix/blob/master/bin/apisix#L397, and 
will be rewrite if enable `proxy-rewrite` 
plugin:https://github.com/apache/incubator-apisix/blob/master/apisix/plugins/proxy-rewrite.lua#L156.
   
   so the line 54 of the file of apisix/plugins/skywalking/tracer.lua should be:
   ```ctx.var.upstream_uri or ctx.var.uri```
   
   welcome PR to fix it.


----------------------------------------------------------------
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


Reply via email to