Closed
Bug 1030526
Opened 11 years ago
Closed 11 years ago
add cors headers
Categories
(Input :: General, defect)
Input
General
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.
|
Assignee | |
Comment 1•11 years ago
|
||
Mike added a cors_enabled decorator to Kitsune. We should probably just use that for now.
|
|
Assignee | |
Comment 2•11 years ago
|
||
Grabbing this one.
Assignee: nobody → willkg
Whiteboard: u=dev c=api p=1 s=input.2014q2
|
|
Assignee | |
Comment 3•11 years ago
|
||
|
|
Assignee | |
Comment 4•11 years ago
|
||
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.
|
|
Assignee | |
Comment 5•11 years ago
|
||
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: 11 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.
Description
•