{"action":"created","issue":{"url":"https://api.github.com/repos/slightlyoff/ServiceWorker/issues/766","labels_url":"https://api.github.com/repos/slightlyoff/ServiceWorker/issues/766/labels{/name}","comments_url":"https://api.github.com/repos/slightlyoff/ServiceWorker/issues/766/comments","events_url":"https://api.github.com/repos/slightlyoff/ServiceWorker/issues/766/events","html_url":"https://github.com/slightlyoff/ServiceWorker/issues/766","id":113036641,"number":766,"title":"should respondWith(undefined) be treated the same as not calling respondWith()","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":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-10-23T14:57:00Z","updated_at":"2015-10-23T14:59:32Z","closed_at":null,"body":"@ehsan debugged the guardian's new service worker on firefox this morning:\r\n\r\n  https://bugzilla.mozilla.org/show_bug.cgi?id=1217501\r\n\r\nIt seems its calling respondWith(undefined) in some circumstances.  Firefox treats this as a NetworkError because undefined is not a Response.  It seems chrome treats respondWth(undefined) the same as not calling respondWith() at all.\r\n\r\nWhat is the desired behavior?\r\n\r\nThe spec actually says that respondWith(undefined) should return an error.  In respondWith() step 6.3:\r\n\r\n> 1) f response is a Response object, then:\r\n>   ...\r\n> 2) else:\r\n>  1) Set the respond-with error flag.\r\n\r\nI can see why maybe the chrome behavior would be desired, though.\r\n\r\ncc / @jakearchibald"},"comment":{"url":"https://api.github.com/repos/slightlyoff/ServiceWorker/issues/comments/150598875","html_url":"https://github.com/slightlyoff/ServiceWorker/issues/766#issuecomment-150598875","issue_url":"https://api.github.com/repos/slightlyoff/ServiceWorker/issues/766","id":150598875,"user":{"login":"jakearchibald","id":93594,"avatar_url":"https://avatars.githubusercontent.com/u/93594?v=3","gravatar_id":"","url":"https://api.github.com/users/jakearchibald","html_url":"https://github.com/jakearchibald","followers_url":"https://api.github.com/users/jakearchibald/followers","following_url":"https://api.github.com/users/jakearchibald/following{/other_user}","gists_url":"https://api.github.com/users/jakearchibald/gists{/gist_id}","starred_url":"https://api.github.com/users/jakearchibald/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jakearchibald/subscriptions","organizations_url":"https://api.github.com/users/jakearchibald/orgs","repos_url":"https://api.github.com/users/jakearchibald/repos","events_url":"https://api.github.com/users/jakearchibald/events{/privacy}","received_events_url":"https://api.github.com/users/jakearchibald/received_events","type":"User","site_admin":false},"created_at":"2015-10-23T14:59:32Z","updated_at":"2015-10-23T14:59:32Z","body":"Ohhh, no I think Firefox is doing the right thing here. Let me help The Guardian fix this, and file a bug with Chrome.\r\n\r\nThanks for spotting this!"}}