Closed Bug 886813 Opened 11 years ago Closed 11 years ago

Enable JSONP as a fallback for IE9 on stopwatching.us

Categories

(mozilla.org :: Security Assurance: Review Request, task)

Product:
mozilla.org
Component:
Security Assurance: Review Request
Platform:
x86
macOS
Type:
task
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: bobby, Unassigned)

References

(
URL
)

Details

(Whiteboard: [Web]) 

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.
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
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [Web]
You need to log in before you can comment on or make changes to this bug.