[GitHub] [skywalking] HaojunRen commented on issue #5721: Too long Http GET url or parameter will cause span lost

2020-10-24 Thread GitBox


HaojunRen commented on issue #5721:
URL: https://github.com/apache/skywalking/issues/5721#issuecomment-716099606


   Based on Spring Cloud Hoxton.SR8 & Spring Boot 2.3.4.RELEASE



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




[GitHub] [skywalking] HaojunRen commented on issue #5721: Too long Http GET url or parameter will cause span lost

2020-10-24 Thread GitBox


HaojunRen commented on issue #5721:
URL: https://github.com/apache/skywalking/issues/5721#issuecomment-716099495


   这是测试结果,请尊重事实,如何?你可以不需要再恢复我,我不寄希望你能解决这个问题,谢谢



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




[GitHub] [skywalking] HaojunRen commented on issue #5721: Too long Http GET url or parameter will cause span lost

2020-10-24 Thread GitBox


HaojunRen commented on issue #5721:
URL: https://github.com/apache/skywalking/issues/5721#issuecomment-716099282


   If I reduce the url, 3 service node can display well
   
![图片](https://user-images.githubusercontent.com/42354711/9717-3eef3100-16ca-11eb-9957-7c7a5b90cabc.png)
   
   
![图片](https://user-images.githubusercontent.com/42354711/97100014-529a9780-16ca-11eb-9e02-aad106a36f51.png)
   



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




[GitHub] [skywalking] HaojunRen commented on issue #5721: Too long Http GET url or parameter will cause span lost

2020-10-24 Thread GitBox


HaojunRen commented on issue #5721:
URL: https://github.com/apache/skywalking/issues/5721#issuecomment-716098890


   
![图片](https://user-images.githubusercontent.com/42354711/97099901-10248b00-16c9-11eb-9ac1-d07de2547b49.png)
   
![图片](https://user-images.githubusercontent.com/42354711/97099924-495cfb00-16c9-11eb-9b13-236e71422345.png)
   
   There different is the url length
   
![图片](https://user-images.githubusercontent.com/42354711/97099945-81643e00-16c9-11eb-83eb-bfda31d6c151.png)
   



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




[GitHub] [skywalking] HaojunRen commented on issue #5721: Too long Http GET url or parameter will cause span lost

2020-10-24 Thread GitBox


HaojunRen commented on issue #5721:
URL: https://github.com/apache/skywalking/issues/5721#issuecomment-716097486


   Dis you do some tests to confirm that is not a real issue?



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




[GitHub] [skywalking] HaojunRen commented on issue #5721: Too long Http GET url or parameter will cause span lost

2020-10-24 Thread GitBox


HaojunRen commented on issue #5721:
URL: https://github.com/apache/skywalking/issues/5721#issuecomment-716096945


   It is not a good manner to close the issue before that issue is resolved



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




[GitHub] [skywalking] HaojunRen commented on issue #5721: Too long Http GET url or parameter will cause span lost

2020-10-24 Thread GitBox


HaojunRen commented on issue #5721:
URL: https://github.com/apache/skywalking/issues/5721#issuecomment-716085287


   可能还有个问题需要补充一下,这个issue似乎在Spring Boot 2.3.x才会出现



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




[GitHub] [skywalking] HaojunRen commented on issue #5721: Too long Http GET url or parameter will cause span lost

2020-10-24 Thread GitBox


HaojunRen commented on issue #5721:
URL: https://github.com/apache/skywalking/issues/5721#issuecomment-716085105


   
有些长url是因为parameter很多引起的,例如,doxx?a=123&b=23456&c=11&d=88,为了能输出埋点,需要削减掉参数?



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




[GitHub] [skywalking] HaojunRen commented on issue #5721: Too long Http GET url or parameter will cause span lost

2020-10-24 Thread GitBox


HaojunRen commented on issue #5721:
URL: https://github.com/apache/skywalking/issues/5721#issuecomment-716084850


   
很多业务系统(例如银行)里跟第三方系统做接口,url一般都很长,很长的url会导致埋点无法入库,在Skywalking界面上无法显示。你可以亲自测试一下,而Jaeger处理长url完全没问题



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




[GitHub] [skywalking] HaojunRen commented on issue #5721: Too long Http GET url or parameter will cause span lost

2020-10-24 Thread GitBox


HaojunRen commented on issue #5721:
URL: https://github.com/apache/skywalking/issues/5721#issuecomment-716083514


   Just lik Http GET 
htttp://ip:port/doxx/aaa, let the URL 
long enough. SkyWalking span limits the length to store. I use memory Mode to 
test the feature, but it works fine in 
   Http POST whatever Body length long or short



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