Closed
Bug 1168943
Opened 9 years ago
Closed 8 years ago
Credentials not forwarded when installing hidden app
Categories
(Marketplace Graveyard :: General, defect)
Marketplace Graveyard
General
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: nick, Unassigned)
References
Details
(Whiteboard: [marketplace-transition])
Attachments
(1 file)
459.82 KB,
image/png
|
Details |
Currently, we have a reviewed but unpublished app only available to the team [0]. When we call navigator.mozApps.installPackage, gecko makes an XHR under the hood. This xhr does not have credentials attached (cookies), so the resulting request fails in a 404, which triggers a MANIFEST_URL_ERROR [1]. [0] https://marketplace.firefox.com/app/saint-seiya-cosmo-cards/ [1] https://dxr.mozilla.org/mozilla-central/source/dom/apps/Webapps.jsm#2333
Reporter | ||
Comment 1•9 years ago
|
||
I believe this has to do with separate JS contexts. Cookies added in the content process don't carry over to the chrome process that's performing the installation/xhr.
Reporter | ||
Comment 2•9 years ago
|
||
This is while I'm set to a be a viewer. This leads me to believe that the "Only team members with the following URL can view and install this app:" feature is broken for all packaged apps. Does marketplace have tests that confirm or deny? The mini manifest can't be hidden; the code in gecko was not designed to forward credentials when requesting the mini manifest. It can probably be fixed, but then the code is 18 weeks out from release.
Reporter | ||
Comment 3•9 years ago
|
||
Fabrice mentions that we try to build up a credentialed XHR based on the content documents cookies, it's possible that this code is failing?
Updated•8 years ago
|
Component: Marketplace → General
Product: Developer Documentation → Marketplace
Comment 4•8 years ago
|
||
Seems like the only way to deal with this is to put the manifest at a separate unguessable URL for unpublished apps and don't use auth.
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
Whiteboard: [marketplace-transition]
You need to log in
before you can comment on or make changes to this bug.
Description
•