Don't call connection_success from v1_connect

Status is _always_ indicated from within the callback and never from
within the return of the connect function itself

Change-Id: I43c90d91b3161d5154e759b198550d64272ed8e0
Reviewed-on: http://review.couchbase.org/33389
Reviewed-by: Brett Lawson <brett19@gmail.com>
Tested-by: Mark Nunberg <mnunberg@haskalah.org>