Check returns properly for receiving offer

Now that we're setting the active attribute of pair to true before
accepting a client offer, we should be careful to reset it to false if
receiving the offer fails
This commit is contained in:
Cecylia Bocovich 2019-07-03 13:15:03 -04:00
parent f6f37c7e68
commit 6210b5ea7f

View file

@ -72,7 +72,11 @@ class Snowflake
@ui.setStatus msg @ui.setStatus msg
recv = @broker.getClientOffer pair.id recv = @broker.getClientOffer pair.id
recv.then (desc) => recv.then (desc) =>
if pair.running then @receiveOffer pair, desc if pair.running
if !@receiveOffer pair, desc
pair.active = false
else
pair.active = false
, (err) -> , (err) ->
pair.active = false pair.active = false
@retries++ @retries++
@ -91,9 +95,14 @@ class Snowflake
offer = JSON.parse desc offer = JSON.parse desc
dbg 'Received:\n\n' + offer.sdp + '\n' dbg 'Received:\n\n' + offer.sdp + '\n'
sdp = new SessionDescription offer sdp = new SessionDescription offer
@sendAnswer pair if pair.receiveWebRTCOffer sdp if pair.receiveWebRTCOffer sdp
@sendAnswer pair
return true
else
return false
catch e catch e
log 'ERROR: Unable to receive Offer: ' + e log 'ERROR: Unable to receive Offer: ' + e
return false
sendAnswer: (pair) -> sendAnswer: (pair) ->
next = (sdp) -> next = (sdp) ->