webRTCConn close as part of its reset, instead of depending on external deferred close after receiving reset channel, allows valid disconnections even before use

client/snowflake.go

@@ -21,7 +21,7 @@ import (
 var ptInfo pt.ClientInfo
 
 const (
-	ReconnectTimeout  = 5
+	ReconnectTimeout  = 10
 	SnowflakeCapacity = 3
 )
 
@@ -109,7 +109,6 @@ func handler(conn *pt.SocksConn) error {
 		conn.Reject()
 		return errors.New("handler: Received invalid Snowflake")
 	}
-	defer remote.Close()
 	defer conn.Close()
 	log.Println("handler: Snowflake assigned.")
 

client/webrtc.go

@@ -291,13 +291,14 @@ func (c *webRTCConn) Reset() {
 		c.reset <- struct{}{}
 		log.Println("WebRTC resetting...")
 	}()
+	c.Close()
 }
 
 func (c *webRTCConn) cleanup() {
 	if nil != c.snowflake {
 		s := c.snowflake
 		log.Printf("WebRTC: closing DataChannel")
-		// Setting snowflak to nil *before* Close indicates to OnClose that it
+		// Setting snowflake to nil *before* Close indicates to OnClose that it
 		// was locally triggered.
 		c.snowflake = nil
 		s.Close()