URL for packaged web app cannot have query string




3 years ago
2 years ago


(Reporter: hchang, Unassigned)


Firefox Tracking Flags

(Not tracked)


(Whiteboard: [necko-would-take])



3 years ago
For example, when you navigate to 


[1] will not consider this format of URL a packaged web content. Instead, it will start downloading "http://foo.com/foo.pak".

Comment 2

3 years ago
Hi Valentin, Jonas,

Could we have your input regarding this issue? IMO, 

http://foo.com/foo.pak?user=moz!//index.html should work as

http://foo.com/foo.pak!//index.html except the server would respond http://foo.com/foo.pak based on query string "?user=moz".

However, for http://foo.com/foo.pak!//index.html?user=moz

I have no idea what kind of request we should send to the server (http://foo.com/foo.pak or http://foo.com/foo.pak?user=moz)
Flags: needinfo?(valentin.gosu)
Flags: needinfo?(jonas)
I dealt with the first half of this question in bug 1172697 (based on an IRC conversation with Jonas, from what I recall)
Meaning if the "token" is in the URL query, we do not treat this as packaged app.
For the second case we just fetch the package and look for a resource named index.html?user=moz
The thinking was that a user could use other request headers to specify which package the server should send.

Of course, this is fairly arbitrary, and we could change it at any time if we feel we should.
Flags: needinfo?(valentin.gosu)
Blocks: 1170823
I don't have a strong opinion about what we should do in either case here. Definitely something that I don't think is a high priority to solve. I'd rather do nothing until we have more author feedback.

My gut guess is that it might be good to let "http://foo.com/foo.pak!//index.html?user=moz" simply load "index.html" from the page, and let page JS act on any query parameters. However, that's a pure guess. So better wait for author feedback I think.
Flags: needinfo?(jonas)
Whiteboard: [necko-would-take]
The web packaged apps implementation was removed from the tree.
Last Resolved: 2 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.