1721460 - Implement cookieStoreId option in browser.downloads.download()

Status: NEW

(GeckoView :: Extensions, enhancement, P3)

Description

[karim]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:89.0) Gecko/20100101 Firefox/89.0

Steps to reproduce:

In container x, set a cookie for host y. Then, call browser.downloads.download({url: y}) in a browser extension.

Actual results:

browser.downloads.download will use the cookies for host y in the default container, and not in container x. (As stated in the docs, "[i]f the specified url uses the HTTP or HTTPS protocol, then the request will include all cookies currently set for its hostname.").

Expected results:

I believe browser.downloads.download() should have an option to specify the container or 'cookieStoreId' which has the desired cookies.

Comment 1

[karim]

This issue has been filed as per https://phabricator.services.mozilla.com/D119967.

Comment 2

[Rob Wu [:robwu]]

GeckoView currently uses a different implementation for cookieStoreId, i.e. geckoViewSessionContextId. Adding a dependency on bug 1643740, because that bug tracks the effort to convert from the one to the other (https://bugzilla.mozilla.org/show_bug.cgi?id=1643740#c1).

That bug is currently unassigned, if you're interested in contributing there I and Agi will gladly support you.

Comment 3

[karim]

(In reply to Rob Wu [:robwu] from comment #2)

GeckoView currently uses a different implementation for cookieStoreId, i.e. geckoViewSessionContextId. Adding a dependency on bug 1643740, because that bug tracks the effort to convert from the one to the other (https://bugzilla.mozilla.org/show_bug.cgi?id=1643740#c1).

That bug is currently unassigned, if you're interested in contributing there I and Agi will gladly support you.

Yes, I've added a comment under bug 1643740 expressing my interest. Thanks for your support, Rob!

Comment 4

[Chris Peterson [:cpeterson]]

Tasks and enhancements should have severity N/A.