Depends on D111247

When removing the support for FTP proxies we should return a proper InvalidArgument error for the New Session command, and note that this kind of proxy is no longer available.

As of now I assume we don't have to do anything for geckodriver given that we want to keep backward compatibility. But maybe we could add a comment in which Firefox release the support has been removed, so that we can eventually already return that error on the driver side without having to launch Firefox. Or we could already check for the version?

James, is there anything else that would need to be done that I missed above?

That sounds right. On the GeckoDriver side we currently allow any proxy settings and rely on marionette to interpret them: https://searchfox.org/mozilla-central/source/testing/geckodriver/src/capabilities.rs#177 We could make that check for an ftp* field and perform a maximum version check, but I suspect this isn't used enough to make it worthwhile.

Valentin, does the landing of this patch block all the other patches in the stack? Maybe we should uncouple it for now? I might not have time to do a further check before next week.

The test will fail if I land it without this.
I can land the test changes if that's OK, followed by removing the proxy settings support.

Depends on D111252

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

The test will fail if I land it without this.
I can land the test changes if that's OK, followed by removing the proxy settings support.

I think that' fine. Can you please move the test removal patch over to the other bug then? That way we can keep this bug for the ftp proxy removal, and we will take care of the remaining bits hopefully soon. Thanks.

Comment on attachment 9217329 [details]
Bug 1703805 - Don't check marionette FTP proxy support in test_session.js r=whimboo

Revision D112906 was moved to bug 1574475. Setting attachment 9217329 [details] to obsolete.

(In reply to Henrik Skupin (:whimboo) [⌚️UTC+1] from comment #7)

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

The test will fail if I land it without this.
I can land the test changes if that's OK, followed by removing the proxy settings support.

I think that' fine. Can you please move the test removal patch over to the other bug then? That way we can keep this bug for the ftp proxy removal, and we will take care of the remaining bits hopefully soon. Thanks.

Thanks!

As discussed I will take it.

Valentin, did we actually get rid of any FTP related code? We also still have the following ftp schema specific code for handling cookies:
https://searchfox.org/mozilla-central/rev/716d7cd80b3dcd81c005ad13b51d3e6a85bd7e73/testing/marionette/driver.js#1964

I assume this also needs an update?

Yes, the FTP code was removed in bug 1574475.
There are a few follow-ups needed. That one should not cause any problems, but should be fixed either way.

Thanks. As it looks like this check is wrong anyway because the HTML spec says:

A Document object that falls into one of the following conditions is a cookie-averse Document object:

    A Document object whose browsing context is null.
    A Document whose URL's scheme is not an HTTP(S) scheme.

So FTP should have never been accepted anyway, and as such can easily be removed.

Depends on D111256

https://hg.mozilla.org/mozilla-central/rev/cf0e44b8205e
https://hg.mozilla.org/mozilla-central/rev/644af69790cb