Closed Bug 1792187 Opened 2 years ago Closed 2 years ago

Webrtc needs some API for determining whether its origin was loaded via a web proxy

Categories

(Core :: Networking, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
109 Branch
Tracking Status
firefox109 --- fixed

People

(Reporter: bwc, Assigned: mleclair)

References

Details

(Whiteboard: [necko-triaged][necko-priority-queue])

Attachments

(1 file)

Right now, we've been trying to use nsIProxiedChannel::GetProxyInfo(), but this just does not work on the content process because that API is not implemented.

In short, webrtc needs this in order to implement the media.peerconnection.ice.proxy_only_if_behind_proxy pref. The intent here is to disable UDP networking, as well as direct p2p TCP connections, when this pref is set and a web-proxy is enabled.

We do not necessarily need a full ProxyInfo, we just need to answer the question "Was my Document loaded via a web proxy?". We are already grabbing the LoadInfo so we can pass it along to the TCP networking code in the socket process, so maybe something simple could be added to the LoadInfo.

Blocks: 1790270

I should also point out that since our ICE stack runs in the content process when the socket process is disabled, and delegates the actual network operations to the parent process, we cannot simply get this information after we've made the hop to the socket process.

Setting priority to P2 because the feature bug for the WebRTC team is P2.

Severity: -- → N/A
Priority: -- → P2
Whiteboard: [necko-triaged][necko-priority-review]
Whiteboard: [necko-triaged][necko-priority-review] → [necko-triaged][necko-priority-queue]
Assignee: nobody → mleclair
Attachment #9301485 - Attachment description: WIP: Bug 1792187 - WIP: added proxy check → Bug 1792187 - added proxy attribute to verify if a proxy is used r=kershaw
Pushed by mleclair@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2bb0a039dfbf
added proxy attribute to verify if a proxy is used r=necko-reviewers,kershaw

Woohoo! Thanks!

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 109 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: