Closed
Bug 1147400
Opened 9 years ago
Closed 9 years ago
call failed to connect (link clicker could hear me, but link generator got no video/audio)
Categories
(Core :: WebRTC: Signaling, defect)
Core
WebRTC: Signaling
Tracking
()
RESOLVED
FIXED
mozilla39
People
(Reporter: Gavin, Assigned: bwc)
References
Details
Attachments
(1 file, 1 obsolete file)
I invited Standard8 to join my room (I generated the link and he clicked it). He joined, I got a notification, and he apparently could hear/see me, but all I ever saw was a grey screen, and couldn't hear him. Eventually my UI gave me: "The stream was unable to connect due to a network error. Make sure your connection isn't blocked by a firewall." I'll attach about:webrtc logs.
Comment 1•9 years ago
|
||
Whilst I was on the conversation with Gavin, I was using Nightly build (today's). We failed to connect three times in a row. On the error console was: 14:11:02.756 "Error while setting RemoteDescription InvalidSessionDescriptionError: Answer changes mid for level, was 'sdparta_0', now ''" sdk.js:1409:12 I tested with Gavin on aurora, which has the same version of the sdk as nightly, and that also failed. This looks like it could be a regression from bug 1144432. I can attach more logs if necessary.
Blocks: 1144432
Component: General → WebRTC: Signaling
Flags: needinfo?(drno)
Flags: needinfo?(docfaraday)
Product: Loop → Core
Assignee | ||
Comment 2•9 years ago
|
||
What version was on the other end? I can see this happening when the other end doesn't support mid. Any logs containing the SDP would tell me for sure.
Flags: needinfo?(docfaraday)
Assignee | ||
Comment 3•9 years ago
|
||
/r/6069 - Bug 1147400: Do not check mid in answer if not present. Pull down this commit: hg pull review -r f48afdbd058da77b204ddb03270c856ed753cdb3
Assignee | ||
Comment 4•9 years ago
|
||
Comment on attachment 8583138 [details] MozReview Request: bz://1147400/bwc https://treeherder.mozilla.org/#/jobs?repo=try&revision=a02c2aeb24b0
Attachment #8583138 -
Flags: review?(drno)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → docfaraday
Flags: needinfo?(drno)
Comment 5•9 years ago
|
||
https://reviewboard.mozilla.org/r/6069/#review5085 LGTM
Assignee | ||
Comment 6•9 years ago
|
||
Seeing some infra bustage on try, retriggering some stuff.
Flags: needinfo?(docfaraday)
Comment 7•9 years ago
|
||
Comment on attachment 8583138 [details]
MozReview Request: bz://1147400/bwc
Now review board has apparently decided to publish my review comment, but not the "ship it" flag. Thanks to this awesome integration I have double work now.
Attachment #8583138 -
Flags: review?(drno) → review+
Assignee | ||
Comment 8•9 years ago
|
||
Still waiting on the OS X debug M3 to finish, and inbound to reopen. Leaving checkin to the sheriff.
Flags: needinfo?(docfaraday)
Keywords: checkin-needed
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 9•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=97134d07092a
Comment 10•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/97134d07092a
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox39:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Comment 11•9 years ago
|
||
Byron, Can you write the aurora approval for this bug to go along with the uplift request for bug 1144432?
Flags: needinfo?(docfaraday)
Assignee | ||
Comment 12•9 years ago
|
||
Comment on attachment 8583138 [details] MozReview Request: bz://1147400/bwc Approval Request Comment See approval request in bug 1144432
Flags: needinfo?(docfaraday)
Attachment #8583138 -
Flags: approval-mozilla-aurora?
Updated•9 years ago
|
status-firefox38:
--- → affected
Updated•9 years ago
|
Attachment #8583138 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 13•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/7d33ef433d9e
Assignee | ||
Comment 14•9 years ago
|
||
Attachment #8583138 -
Attachment is obsolete: true
Attachment #8619868 -
Flags: review+
Assignee | ||
Comment 15•9 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•