Closed Bug 1212818 Opened 9 years ago Closed 8 years ago

URL for packaged web app cannot have query string

Categories

(Core :: Networking, defect)

defect
Not set
normal

Tracking

()

RESOLVED INVALID

People

(Reporter: hchang, Unassigned)

References

Details

(Whiteboard: [necko-would-take])

For example, when you navigate to http://foo.com/foo.pak?user=moz!//index.html [1] will not consider this format of URL a packaged web content. Instead, it will start downloading "http://foo.com/foo.pak".
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)
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.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.