Closed Bug 698333 Opened 8 years ago Closed 8 years ago

oauth code doesn't support non-json responses nor return the status code

Categories

(Web Apps :: General, defect, P4)

defect

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: markh, Unassigned)

Details

The OWA framework (services.js) does the json parsingof the body before passing it back to the app.  This fails if the response is not json, which it may not be even in our use-cases (eg, Twitter 500 errors come back as HTML).  Further, there is no way to get back the status code from the call (eg, there is no way to know that twitter returned a 500)

I think the correct fix is to (a) pass the raw response string back and force the caller to parse the JSON and (b) pass back almost all of the XHR attributes (eg, status, statusText, etc - probably even all the headers)
Component: OpenWebApps → General
Priority: -- → P4
Product: Mozilla Labs → Web Apps
QA Contact: openWebApps → general
Version: Trunk → unspecified
Could you clarify what bug component this applies to?
This applied to a much earlier version of firefox share while it was still in the webapps "activities" component.  Activities is now broken out on its own and IIUC, the current FF Share implementation doesn't use client-side oauth at all.  So its out of date.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.