handler.go: Send NewIncomingConnection before any user packets.

dial.go

@@ -267,7 +267,7 @@ func (dialer Dialer) clientListen(rakConn *Conn, conn net.Conn) {
 			continue
 		}
 		if err := rakConn.receive(b[:n]); err != nil {
-			dialer.ErrorLog.Error("client: handle packet: " + err.Error())
+			dialer.ErrorLog.Error("client: " + err.Error())
 		}
 	}
 }

handler.go

@@ -222,14 +222,15 @@ func handleConnectionRequestAccepted(conn *Conn, b []byte) error {
 	if err := pk.UnmarshalBinary(b); err != nil {
 		return fmt.Errorf("read CONNECTION_REQUEST_ACCEPTED: %w", err)
 	}
-
 	select {
 	case <-conn.connected:
 		return errUnexpectedAdditionalCRA
 	default:
+		// Make sure to send NewIncomingConnection before closing conn.connected.
+		err := conn.send(&message.NewIncomingConnection{ServerAddress: resolve(conn.raddr), RequestTimestamp: pk.AcceptedTimestamp, AcceptedTimestamp: timestamp()})
 		close(conn.connected)
+		return err
 	}
-	return conn.send(&message.NewIncomingConnection{ServerAddress: resolve(conn.raddr), RequestTimestamp: pk.RequestTimestamp, AcceptedTimestamp: pk.AcceptedTimestamp, SystemAddresses: pk.SystemAddresses})
 }
 
 // handleNewIncomingConnection handles an incoming connection packet from the

packet.go

@@ -53,11 +53,12 @@ const (
 type packet struct {
 	reliability byte
 
-	content       []byte
 	messageIndex  uint24
 	sequenceIndex uint24
 	orderIndex    uint24
 
+	content []byte
+
 	split      bool
 	splitCount uint32
 	splitIndex uint32