This repository has been archived by the owner. It is now read-only.

Update api url random-user.coffee #1314

Merged
merged 1 commit into from Feb 10, 2014

Conversation

Projects
None yet
2 participants
@hashashin
Contributor

hashashin commented Feb 8, 2014

fix: Error occured parsing response body: SyntaxError: Unexpected token <

Update api url
fix: Error occured parsing response body: SyntaxError: Unexpected token <
@technicalpickles

This comment has been minimized.

Show comment
Hide comment
@technicalpickles

technicalpickles Feb 10, 2014

Member

It might be worth checking res.statusCode too. The old URL was probably returning a non-200 with HTML output, but JSON.parse was being called regardless.

Member

technicalpickles commented Feb 10, 2014

It might be worth checking res.statusCode too. The old URL was probably returning a non-200 with HTML output, but JSON.parse was being called regardless.

technicalpickles added a commit that referenced this pull request Feb 10, 2014

Merge pull request #1314 from hashashin/fix-random-user
Update api url random-user.coffee

@technicalpickles technicalpickles merged commit 9514439 into github:master Feb 10, 2014

@hashashin hashashin deleted the hashashin:fix-random-user branch Feb 10, 2014

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.