search

six-ddc / plow

A high-performance HTTP benchmarking tool that includes a real-time web UI and terminal display
Apache License 2.0
4.11k stars 140 forks source link

optimize link address setting for direct connect to remote plow #8

Closed wweir closed 3 years ago

wweir commented 3 years ago

在服务器(远程)执行 plow 的时候，原先 --link=XXX 是必须设置，才能正常展示图标的。 现在不用设置也可以正常查看了

six-ddc commented 3 years ago

感谢 pr，我理解这样是改成相对路径是吧，但是发现有个新的问题，就是终端上的展示变成了 

@ Real-time charts is listening on http://./

没有办法去直观的展示这个图标页面的地址了

wweir commented 3 years ago

@six-ddc 改了一下 main.go 不知道原先在 main 里面那么写写的目的是什么，不过，现在看起来应该没问题了

six-ddc commented 3 years ago

又看了下，这里的的 linkAddr 在 charts 页面貌似是多余的，涉及到资源的地方，都配置成 '/path' 就行，不需要设置带上 scheme 和 host，浏览器会自动带上地址的

wweir commented 3 years ago

现在 link 相关的东西全删了。 原来在担心 link 是给代理、跳板机之类场景用的。现在看来，并没什么用，就全给删了