View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0000247 | NNW API Service | Uncategorized | public | August 20, 2014, 9:17 am | January 13, 2017, 11:36 am |
| Reporter | Scott Bradford | Assigned To | Scott Bradford | ||
| Priority | high | Severity | minor | ||
| Status | closed | Resolution | fixed | ||
| Product Version | 0.7 | ||||
| Summary | 0000247: Verify JSON Validity and Auth State Before Sending to Client | ||||
| Description | Verify JSON Validity and Auth State Before Sending to Client | ||||
|
|
In at least some cases we're generating invalid JSON, which is throwing an error on the client side when they receive it. In my case, it was 'Not authorized.' -- which shouldn't happen when you're visiting from the app! How did we lose the session? So we need to do a few things: 1) Not-authorized should be returned as JSON so the client can process and act on it. 2) If the client receives the 'Not-authorized' error, we should reload the page to reestablish the PHP session. 3) We should log all authorization errors (with API key and/or session ID). 4) We should test all JSON to see if it's valid before we spit it out to the client. If not, we should log it and give a nicer error message. |
|
|
Done. |
|
|
Closing (all resolved and published) |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| December 31, 1969, 7:33 pm | Scott Bradford | Status | @0@ => @0@ |
| December 31, 1969, 7:33 pm | Scott Bradford | Resolution | @0@ => @0@ |
| August 20, 2014, 9:17 am | Scott Bradford | New Issue | |
| August 18, 2015, 10:43 am | Scott Bradford | Project | @44@ => No-Nonsense Weather |
| August 18, 2015, 11:19 am | Scott Bradford | Category | General => Uncategorized |
| August 18, 2015, 1:23 pm | Scott Bradford | Note Added: 0001056 | |
| August 18, 2015, 1:23 pm | Scott Bradford | Status | resolved => closed |
| September 24, 2016, 5:14 pm | Scott Bradford | Project | No-Nonsense Weather => NNW API Service |
| January 13, 2017, 11:36 am | Changeset attached | => NNW Web trunk r694 | |
| January 13, 2017, 11:36 am | Changeset attached | => NNW Web trunk r704 |