twitter / CocoaSPDY / 240

76%

master: %

Pull #99

travis-ci

Fix proxy when config is invalid.

If an auto-config .pac file specifies a SOCKS proxy, or has a typo,
or a manually configured proxy has no values, or any other case where
we end up with no available proxies, we should fall back to using a
direct connection.

In addition, within the connectToOrigin code in SPDYSocket, if the
connection fails on the first call (using a manually configured proxy
or something invalid happens setting up the streams), then it's
important to return NO to the session. Since the delegate hasn't been
set yet in the session, always returning YES could result in a zombie
session being inserted into the pool and remaining there indefinitely.

More unit tests for endpoint manager and socket, and some refactoring.
Fixed some test logic.

28 of 28 new or added lines in 2 files covered. (100.0%)

2178 of 3565 relevant lines covered (61.09%)

30.47 hits per line