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
UV: Set error status on connect failure

Change-Id: Ic663d241d2ab1852310c4c238060320b3269f3d9