Enable JSONP as a fallback for IE9 on stopwatching.us

RESOLVED FIXED

Status

mozilla.org
Security Assurance: Review Request
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: secretrobotron, Unassigned)

Tracking

Details

(Whiteboard: [Web], URL)

(Reporter)

Description

5 years ago
I noticed that our counter has some trouble in IE9 and earlier. We've had jsonp support on our server for a while, but we've never enabled it on the client. This patch enables it: https://github.com/secretrobotron/stopwatchingus/commit/3691256111d60bd5efc43bac05265fa1db460187#L0R1275

Requesting a quick security review before making it live.

Comment 1

5 years ago
This looks fine to me. The code adds an error path that performs a second request to the js endpoint instead of json on fialure. This data is then passed to a jsonp callback.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: [Web]
You need to log in before you can comment on or make changes to this bug.