1968911 - Make it possible to add a ProxyFilter for "system", "autodetect" and "pac"

Status: NEW

(Core :: Networking: Proxy, task, P3)

Description

[Alexandra Borovova [:Sasha]]

In the scope of bug 1967653, we introduced the ability to override proxy settings which are applied for a session with preferences on user context level by using proxy filters for a channel. That works fine for "manual" and "direct" type. But we also have to support "system", "autodetect" and "pac" types. In this case we don't know the proxy host and port so we can no set up the filter. So I think we either need a platform API to get this data or an API to just set directly (or set some special filter type) for "system", "autodetect" or "pac" proxy to the channel.

Comment 1

[Valentin Gosu [:valentin] (he/him)]

Do I understand the use case correctly?
If Firefox is set up to use "direct" (no-proxy), we want to set up a ProxyFilter that allows the channel to use "system"/"autodetect"/"custom PAC" proxy settings instead, right?

That would require some changes to the ProtocolProxyService, as we can't currently suport both system and autodetect (for example).

Some extra info that would be nice:
Do other browsers already support this?
What is the timeline we need for this? Are the workarounds in bug 1967653 enough?

Comment 2

[Alexandra Borovova [:Sasha]]

(In reply to Valentin Gosu [:valentin] (he/him) from comment #1)

Do I understand the use case correctly?
If Firefox is set up to use "direct" (no-proxy), we want to set up a ProxyFilter that allows the channel to use "system"/"autodetect"/"custom PAC" proxy settings instead, right?

That's right.

Some extra info that would be nice:
Do other browsers already support this?

Looks like Chrome supports setting "system" proxy, but not "autodetect" or "pac".
Safari doesn't support setting a proxy per user context, so it's a bit hard to say if they have platform support for this.

What is the timeline we need for this? Are the workarounds in bug 1967653 enough?

In the scope of bug 1967653 we just implemented "direct" and "manual" proxy, and it's enough for now, since it looks like that are the only things which are required for Playwright and Puppeteer. But that's what the WebDriver BiDi spec supports, so we will need it for completeness.

Comment 3

[Valentin Gosu [:valentin] (he/him)]

Thanks for the info. If the priority changes please let us know.