Closed Bug 1030526 Opened 10 years ago Closed 10 years ago

add cors headers

Categories

(Input :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: willkg, Assigned: willkg)

Details

(Whiteboard: u=dev c=api p=2 s=input.2014q2)

The front page dashboard can produce JSON output. This can be useful for dashboards, but we need to enable CORS to allow for XHR for that data.

I think it's safe/appropriate to do:

Access-Control-Allow-Origin: *

for the frontpage dashboard for GET requests.

This bug covers making sure that's correct and implementing it for the frontpage dashboard.
Mike added a cors_enabled decorator to Kitsune. We should probably just use that for now.
Grabbing this one.
Assignee: nobody → willkg
Whiteboard: u=dev c=api p=1 s=input.2014q2
Landed in master: https://github.com/mozilla/fjord/commit/070f992

This makes some changes to the feedback posting api, so I'm going to push this out now and watch it over the next few hours to see how it affects things.
This fixes an issue that isn't a big issue now, but could be in the future: https://github.com/mozilla/fjord/commit/f6f5bc9

Pushed it all to prod. Works smooth.

This ended up being 2 points because I had to write a whole new API endpoint. Whee.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: u=dev c=api p=1 s=input.2014q2 → u=dev c=api p=2 s=input.2014q2
You need to log in before you can comment on or make changes to this bug.