{"action":"opened","issue":{"url":"https://api.github.com/repos/web-platform-tests/wpt/issues/13662","repository_url":"https://api.github.com/repos/web-platform-tests/wpt","labels_url":"https://api.github.com/repos/web-platform-tests/wpt/issues/13662/labels{/name}","comments_url":"https://api.github.com/repos/web-platform-tests/wpt/issues/13662/comments","events_url":"https://api.github.com/repos/web-platform-tests/wpt/issues/13662/events","html_url":"https://github.com/web-platform-tests/wpt/issues/13662","id":372645162,"node_id":"MDU6SXNzdWUzNzI2NDUxNjI=","number":13662,"title":"beacon-error.window.html makes (unjustified?) assumptions about the max beacon size","user":{"login":"dholbert","id":426803,"node_id":"MDQ6VXNlcjQyNjgwMw==","avatar_url":"https://avatars3.githubusercontent.com/u/426803?v=4","gravatar_id":"","url":"https://api.github.com/users/dholbert","html_url":"https://github.com/dholbert","followers_url":"https://api.github.com/users/dholbert/followers","following_url":"https://api.github.com/users/dholbert/following{/other_user}","gists_url":"https://api.github.com/users/dholbert/gists{/gist_id}","starred_url":"https://api.github.com/users/dholbert/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dholbert/subscriptions","organizations_url":"https://api.github.com/users/dholbert/orgs","repos_url":"https://api.github.com/users/dholbert/repos","events_url":"https://api.github.com/users/dholbert/events{/privacy}","received_events_url":"https://api.github.com/users/dholbert/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-10-22T18:38:57Z","updated_at":"2018-10-22T18:38:57Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"The test https://wpt.fyi/results/beacon/beacon-error.window.html has the following hardcoded value in a JS helper file:\r\n````js\r\nvar maxPayloadSize = 65536; // The maximum payload size allowed for a beacon request.\r\n````\r\nhttps://github.com/web-platform-tests/wpt/blame/master/beacon/beacon-common.sub.js#L7\r\n\r\n...and then it has two subtests (in beacon-error.window.js) that depend on that value:\r\n- \"Verify calling 'navigator.sendBeacon()' with a large payload returns 'false'\"\r\n- \"Verify the behavior after the quota is exhausted.\" [exhaused by sending maxPayloadSize]).\r\n\r\nThese two tests fail in Firefox right now (i.e. Firefox doesn't seem to think this payload is too large), and I'm trying to figure out whether the failure is legitimate or not. At first glance, it's not clear to me that this maxPayloadSize is rooted in the spec...  The spec is [here](https://w3c.github.io/beacon/) and it says there is *some* UA-imposed maximum but it doesn't say what that maximum is. (\"The user agent restricts the maximum accepted payload size to ensure that beacon requests are able to complete quickly and in a timely manner.\")\r\n\r\nCC @Brandr0id who wrote this line of the test and @yutakahirano who's made some edits since then -- do either of you have thoughts here or know if this particular maximum value is mandated somewhere?"}}
{"actor":{"display_login":"dholbert"}}