commit 40007e4fa55f77e4657bbc8931f8d540c002057d
Author: David Fifield <da...@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()
_______________________________________________
tor-commits mailing list
tor-commits@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
