Closed
Bug 1290065
Opened 9 years ago
Closed 9 years ago
Intermittent dom/tests/mochitest/beacon/test_beaconRedirect.html | SendBeacon should follow cross origin redirects! - got "reset", expected "green"
Categories
(Core :: DOM: Security, defect)
Core
DOM: Security
Tracking
()
RESOLVED
FIXED
mozilla51
People
(Reporter: intermittent-bug-filer, Assigned: ckerschb)
References
Details
(Keywords: intermittent-failure, Whiteboard: [domsecurity-intermittent])
Attachments
(1 file)
|
1.15 KB,
patch
|
cbook
:
review+
|
Details | Diff | Splinter Review |
| Assignee | ||
Comment 2•9 years ago
|
||
Hey Carsten, the problem is that sendBeacon does not return a result from the server, hence we have to wait for a certain amount of time and query the result from the server using an XHR request. Before updating sendBeacon to not using 'cors' (within Bug 1280692) the redirect within in the test had to be blocked, hence 2 seconds where enough before querying the result.
Now that we have to wait that the redirect succeeds, we should extend the time frame before we query the result from 2 to 4 seconds.
I suppose the test is not intermittently failing on all platforms, but rather on slow ones only, right? e.g. Windows VM and such.
Flags: needinfo?(ckerschb)
Attachment #8775557 -
Flags: review?(cbook)
| Assignee | ||
Updated•9 years ago
|
Assignee: nobody → ckerschb
Status: NEW → ASSIGNED
Component: DOM → DOM: Security
Whiteboard: [domsecurity-intermittent]
| Comment hidden (Intermittent Failures Robot) |
Updated•9 years ago
|
Attachment #8775557 -
Flags: review?(cbook) → review+
| Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7eb40c545894
Extend timeout for sendBeacon redirect test so redirect can happen before querying the result. r=tomcat
Keywords: checkin-needed
Comment 5•9 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox51:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
Updated•9 years ago
|
status-firefox50:
--- → affected
Comment 8•9 years ago
|
||
| bugherder uplift | ||
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
You need to log in
before you can comment on or make changes to this bug.
Description
•