trim incoming data before trying to parse it as json to make sure there ...#208
trim incoming data before trying to parse it as json to make sure there ...#208prusik wants to merge 4 commits intodanwrong:masterfrom
Conversation
…re is something to parse.
|
Wish this would get merged, as having the same issue. @prusik you may want to remove your package.json changes from the PR to make it easier on them (otherwise they have to change it after they merge) |
|
Ok so I removed the modifications I made to the package.json. Sorry this was my first pull request. If there is anything else I can do, please let me know. |
|
hi can you just rebase out anything to do with the package.json so to get rid of those last 3 commits do something like:
it should remove them and you can force push the changes back to your repo. Cheers. |
|
Sorry but I have no clue how to rebase and have no more time for this now. I did try but was unsuccessful. |
|
No worries. Let me know when you manage to rebase it and I can merge this in. I need to keep the commits clean so it can't go in as it is at the moment I'm afraid. Thanks. |
...is something to parse.
As context, I need this in an integration with the GetHarvest API, where they use restler but return " " (empty string with space) in their response. Ideally I would prefer that they do the fix on their side, however since it is not open source I feel like this would take forever. I have added the fix and included a test.
issue: #207
thanks!