1570158 - Add proxy information to candidates table in about:webrtc

Status: RESOLVED FIXED

(Core :: WebRTC, task, P2)

Description

[Ryan Alderete]

It would be convenient to have a field for each ICE candidate on about:webrtc that indicates whether that candidate is behind a proxy or not. That way users do not have to inspect packets via Wireshark or some other tool to determine which communications are through a proxy.

Comment 1

[Ryan Alderete]

Attachment: Bug 1570158 - Add proxy information to candidates table in about:webrtc

This adds a field to about:webrtc which indicates whether an associated
candidate is behind a proxy or not.

Comment 2

[C.M.Chang[:chunmin]]

It seems to me a on-going task. Set the priority to P2 for now. Feel free to change it.

Comment 3

[Byron Campen [:bwc]]

Added a couple more review comments, should be fine after those are done.

Comment 4

[Bogdan Tara[:bogdan_tara | bogdant]]

Tried to land, but got: "applying /tmp/tmpsdUV74 2 out of 2 hunks ignored -- saving rejects to file media/mtransport/nricectx.cpp.rej abort: patch command failed: exited with status 256"

Comment 5

[Ryan Alderete]

(In reply to Bogdan Tara[:bogdan_tara] from comment #4)

Tried to land, but got: "applying /tmp/tmpsdUV74 2 out of 2 hunks ignored -- saving rejects to file media/mtransport/nricectx.cpp.rej abort: patch command failed: exited with status 256"

I'm not totally sure what caused this, but it may have been an issue with whitespace in that file. I've modified that out of the diff, so hopefully that makes it ready to land.

Comment 6

[Cosmin Sabou [:CosminS]]

Ryan, you need to update the commit message to contain one of the below reviewers:
Please, request review from either: remote: - Andrea Marchesini (:baku) remote: - Andreas Farre (:farre) remote: - Andrew McCreight (:mccr8) remote: - Bobby Holley (:bholley) remote: - Boris Zbarsky (:bz) remote: - Ehsan Akhgari (:ehsan) remote: - Henri Sivonen (:hsivonen) remote: - Nika Layzell (:mystor) remote: - Olli Pettay (:smaug) remote: - Peter Van der Beken (:peterv)

hg error in cmd: hg push -r tip upstream: pushing to ssh://hg.mozilla.org/integration/autoland searching for changes remote: adding changesets remote: adding manifests remote: adding file changes remote: added 1 changesets with 14 changes to 14 files remote: remote: ******************************* ERROR ******************************* remote: Changeset c4c94dd05ecf alters WebIDL file(s) without DOM peer review: remote: dom/webidl/RTCStatsReport.webidl remote: remote: Please, request review from either: remote: - Andrea Marchesini (:baku) remote: - Andreas Farre (:farre) remote: - Andrew McCreight (:mccr8) remote: - Bobby Holley (:bholley) remote: - Boris Zbarsky (:bz) remote: - Ehsan Akhgari (:ehsan) remote: - Henri Sivonen (:hsivonen) remote: - Nika Layzell (:mystor) remote: - Olli Pettay (:smaug) remote: - Peter Van der Beken (:peterv) remote: ********************************************************************* remote: remote: transaction abort! remote: rollback completed remote: pretxnchangegroup.mozhooks hook failed abort: push failed on remote

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8f803211fda5
Add proxy information to candidates table in about:webrtc r=bwc,baku

Comment 8

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/8f803211fda5