Re: [ServiceWorker] Should ServiceWorker.scriptURL have fragment removed? (#742)

Currently, it doesn't strip the fragment for scriptURL, and the scriptURL comparison is in two places: one in the Register algorithm (step 4.2.2) and the other in Update algorithm (step 4.25). This means, tweaking the fragment part of the url makes it get through to Install phase (possibly checking the network) for the same javascript file as putting a different query string does. If this case is desirable and handy to devs, we may want to keep it as-is. Otherwise, excluding the fragment for scriptURL makes sense to me.

In either case, I think we need to change the Update algorithm step 4.25 to not use the fragment in the url comparison. Otherwise, the user agent continues to install the byte-for-byte matching script resource, which is meaningless.

---
Reply to this email directly or view it on GitHub:
https://github.com/slightlyoff/ServiceWorker/issues/742#issuecomment-137310312

Received on Thursday, 3 September 2015 02:58:41 UTC