1285318 - Turn off WebRTC TCP packet filter for e10s

Status: RESOLVED FIXED

(Core :: WebRTC: Networking, defect, P1)

Description

[Nils Ohlmeier [:drno]]

Bugs 1279514 and 1275216 revealed that with the current implementation of the TCP packet filter for e10s users who use a HTTP proxy and try to use a WebRTC service which provides a TURN TCP server will experience an (intended) content process crash.
The problem is that the TCP filter from bug 1244926 only allows STUN packets as the initial packets to be exchanged over a TCP connection created by WebRTC content. But in case of an HTTP proxy is used first an HTTP CONNECT is send to the proxy and only after the proxy responded with a 200 OK our implementation starts to send STUN packets to the TURN server.

We need to find a different solution then the code in bug 1244926 to handle all possible scenarios which can occur over a WebRTC TCP connection.

To prevent a regression from the users perspective when e10s gets turned on for them (using TURN TCP through an HTTP proxy works without e10s) we need to turn the packet filtering off until we have a better solution.

Comment 1

[Nils Ohlmeier [:drno]]

Attachment: Bug 1285318: turn of WebRTC TCP packet filtering.

Review commit: https://reviewboard.mozilla.org/r/62906/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/62906/

Comment 2

[Nils Ohlmeier [:drno]]

Comment on attachment 8768873 [details]
Bug 1285318: turn of WebRTC TCP packet filtering.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/62906/diff/1-2/

Comment 3

[Michael Froman [:mjf]]

Comment on attachment 8768873 [details]
Bug 1285318: turn of WebRTC TCP packet filtering.

https://reviewboard.mozilla.org/r/62906/#review59814

Looks good to me.

Comment 4

[Nils Ohlmeier [:drno]]

Comment on attachment 8768873 [details]
Bug 1285318: turn of WebRTC TCP packet filtering.

Approval Request Comment
[Feature/regressing bug #]: 1244926 introduce a new packet filter

[User impact if declined]: For users who have a HTTP proxy configured in FF and try to a WebRTC service with a TURN TCP relay (good WebRTC service do provide one to increase chance for a successful call) the content process under e10s crashes.

[Describe test coverage new/current, TreeHerder]: manual testing only.

[Risks and why]: no risk as it only turns off the feature which got added in bug 1244926.

[String/UUID change made/needed]: N/A

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b6984012fb0b
turn of WebRTC TCP packet filtering. r=mjf

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/b6984012fb0b

Comment 7

[Liz Henry (:lizzard) (relman/hg->git project)]

Comment on attachment 8768873 [details]
Bug 1285318: turn of WebRTC TCP packet filtering.

Revert to previous behavior to avoid a crash, let's try it. This should be in beta 7 build next Monday.

Comment 8

[Randell Jesup [:jesup] (needinfo me)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/11c6cfe72f40
https://hg.mozilla.org/releases/mozilla-beta/rev/d9400e38d2d1