commit 5c185a06eb766055211a942cc3e3c80deb677a38 Author: David Fifield david@bamsoftware.com Date: Sun May 25 11:18:33 2014 -0700
Add some commented test cases to think about.
It may make sense for use to ensure that the proxy URL has a separable host name and port. On the other hand, that's one of the error conditions the application is going to have to check for anyway when it goes to connect to the proxy. --- meek-client/proxy_test.go | 10 ++++++++++ 1 file changed, 10 insertions(+)
diff --git a/meek-client/proxy_test.go b/meek-client/proxy_test.go index d26362d..77123b9 100644 --- a/meek-client/proxy_test.go +++ b/meek-client/proxy_test.go @@ -40,6 +40,16 @@ func TestGetProxyURL(t *testing.T) { {"socks4a://localhost:1080", "socks4a://localhost:1080"}, {"unknown://localhost/whatever", "unknown://localhost/whatever"}, } + /* + No port: reject; or infer from scheme? + http://localhost + socks4a://localhost + socks5://localhost + Port without host: probably reject? + http://:8080 + socks4a://:1080 + socks5://:1080 + */
os.Clearenv() u, err := PtGetProxyURL()