Move establishDataChannel to after exchangeSDP.

This commit is contained in:
David Fifield 2020-04-24 11:44:40 -06:00
parent 85277274fd
commit e8c41650ae

View file

@ -120,11 +120,6 @@ func (c *WebRTCPeer) connect(config *webrtc.Configuration, broker *BrokerChannel
if err != nil { if err != nil {
return err return err
} }
err = c.establishDataChannel()
if err != nil {
// nolint: golint
return errors.New("WebRTC: Could not establish DataChannel")
}
answer := exchangeSDP(broker, c.pc.LocalDescription()) answer := exchangeSDP(broker, c.pc.LocalDescription())
log.Printf("Received Answer.\n") log.Printf("Received Answer.\n")
err = c.pc.SetRemoteDescription(*answer) err = c.pc.SetRemoteDescription(*answer)
@ -132,6 +127,11 @@ func (c *WebRTCPeer) connect(config *webrtc.Configuration, broker *BrokerChannel
log.Println("WebRTC: Unable to SetRemoteDescription:", err) log.Println("WebRTC: Unable to SetRemoteDescription:", err)
return err return err
} }
err = c.establishDataChannel()
if err != nil {
// nolint: golint
return errors.New("WebRTC: Could not establish DataChannel")
}
go c.checkForStaleness() go c.checkForStaleness()
return nil return nil
} }