{"action":"created","issue":{"url":"https://api.github.com/repos/slightlyoff/ServiceWorker/issues/793","repository_url":"https://api.github.com/repos/slightlyoff/ServiceWorker","labels_url":"https://api.github.com/repos/slightlyoff/ServiceWorker/issues/793/labels{/name}","comments_url":"https://api.github.com/repos/slightlyoff/ServiceWorker/issues/793/comments","events_url":"https://api.github.com/repos/slightlyoff/ServiceWorker/issues/793/events","html_url":"https://github.com/slightlyoff/ServiceWorker/issues/793","id":119727879,"number":793,"title":"navigations that are not intercepted should still allow interception on further redirects","user":{"login":"wanderview","id":1256526,"avatar_url":"https://avatars.githubusercontent.com/u/1256526?v=3","gravatar_id":"","url":"https://api.github.com/users/wanderview","html_url":"https://github.com/wanderview","followers_url":"https://api.github.com/users/wanderview/followers","following_url":"https://api.github.com/users/wanderview/following{/other_user}","gists_url":"https://api.github.com/users/wanderview/gists{/gist_id}","starred_url":"https://api.github.com/users/wanderview/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wanderview/subscriptions","organizations_url":"https://api.github.com/users/wanderview/orgs","repos_url":"https://api.github.com/users/wanderview/repos","events_url":"https://api.github.com/users/wanderview/events{/privacy}","received_events_url":"https://api.github.com/users/wanderview/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/slightlyoff/ServiceWorker/labels/decided","name":"decided","color":"eb6420"},{"url":"https://api.github.com/repos/slightlyoff/ServiceWorker/labels/fetch","name":"fetch","color":"bfd4f2"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/slightlyoff/ServiceWorker/milestones/2","html_url":"https://github.com/slightlyoff/ServiceWorker/milestones/Version%201","labels_url":"https://api.github.com/repos/slightlyoff/ServiceWorker/milestones/2/labels","id":746092,"number":2,"title":"Version 1","description":"","creator":{"login":"tobie","id":3392,"avatar_url":"https://avatars.githubusercontent.com/u/3392?v=3","gravatar_id":"","url":"https://api.github.com/users/tobie","html_url":"https://github.com/tobie","followers_url":"https://api.github.com/users/tobie/followers","following_url":"https://api.github.com/users/tobie/following{/other_user}","gists_url":"https://api.github.com/users/tobie/gists{/gist_id}","starred_url":"https://api.github.com/users/tobie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tobie/subscriptions","organizations_url":"https://api.github.com/users/tobie/orgs","repos_url":"https://api.github.com/users/tobie/repos","events_url":"https://api.github.com/users/tobie/events{/privacy}","received_events_url":"https://api.github.com/users/tobie/received_events","type":"User","site_admin":false},"open_issues":31,"closed_issues":160,"state":"open","created_at":"2014-08-07T10:50:36Z","updated_at":"2016-08-09T07:28:54Z","due_on":"2016-03-30T15:00:00Z","closed_at":null},"comments":16,"created_at":"2015-12-01T14:29:05Z","updated_at":"2016-08-09T13:35:29Z","closed_at":null,"body":"After talking with @annevk, it seems the intention of the spec is that every redirect of a navigation should check for service worker interception again.  Currently the spec is quite vague about this.\r\n\r\nCurrently step 4.2 of HTTP Fetch unconditionally set skip-service-worker on the Request if a service worker does not intercept in step 3.  In the case of a navigation that redirects, this results in a Request with skip-service-worker being set returning an opaque redirect back to the navigation spec logic.  The http spec navigation logic is somewhat vague about if a new Request is created or not.  The important question here is \"does the skip-service-worker\" flag propagate across redirects for the navigation?\r\n\r\nTo sidestep the http spec vagueness, @annevk has suggested we simply change HTTP Fetch step 4.2 to only set skip-service-worker if the RequestRedirect value is \"follow\"."},"comment":{"url":"https://api.github.com/repos/slightlyoff/ServiceWorker/issues/comments/238554471","html_url":"https://github.com/slightlyoff/ServiceWorker/issues/793#issuecomment-238554471","issue_url":"https://api.github.com/repos/slightlyoff/ServiceWorker/issues/793","id":238554471,"user":{"login":"jungkees","id":1331169,"avatar_url":"https://avatars.githubusercontent.com/u/1331169?v=3","gravatar_id":"","url":"https://api.github.com/users/jungkees","html_url":"https://github.com/jungkees","followers_url":"https://api.github.com/users/jungkees/followers","following_url":"https://api.github.com/users/jungkees/following{/other_user}","gists_url":"https://api.github.com/users/jungkees/gists{/gist_id}","starred_url":"https://api.github.com/users/jungkees/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jungkees/subscriptions","organizations_url":"https://api.github.com/users/jungkees/orgs","repos_url":"https://api.github.com/users/jungkees/repos","events_url":"https://api.github.com/users/jungkees/events{/privacy}","received_events_url":"https://api.github.com/users/jungkees/received_events","type":"User","site_admin":false},"created_at":"2016-08-09T13:35:29Z","updated_at":"2016-08-09T13:35:29Z","body":"> Why the check if the flag is unset? Seems redundant.\r\n\r\nSorry. Coming back from other context, I can't remember exactly. Seems redundant though I thought there was a reason.\r\n\r\n> Although something close to this is needed if the 401/407 is followed by a redirect.\r\n\r\nYes, this is the scenario I thought unsetting the flag is needed.\r\n\r\n> Which is why I think a variable that we pass along rather than a flag stored on request is probably the way to go here.\r\n\r\nI'm fine with the way you think of it. But I think I'm not quite clear with the entire flow in detail. I'll give a feedback when you provide an outline or changes to the spec."}}
{"actor":{"display_login":"jungkees"}}