Remove duplicate connect method from session

The Session.connect method is defined twice with minor differences. I
removed the first one to ensure the behavior remains the same, and
because I couldn't quite make sense of it. The first method depended
upon another method, Session#establish_connection, which doesn't appear
to be called anywhere else. I removed that method too.
Merge pull request #184 from devin-c/remove_duplicate_connect_method

Remove duplicate connect method from session