Add headers param to API calls, auto-detect JSON for encoding to raw data, prevent exceptions caused by logging non UTF-8 data

[vbuck]

When using endpoints like POST /api/v2/uploads.json where binary data is transferred in the request body, we need to extend the abstract class to allow for such non-JSON data. We also need a means for setting the content-type associated with the transfer, which this update provides.

For backwards compatibility, these additions will fall back to their previous implementation (JSON-specific) when not used.

As a side-effect, this caused logging errors because of non UTF-8 characters in the request body. So that also needed to be accounted for.

With this update, it's now possible to use the attachments resource in the following way:

$extendingClass->_call(
  'uploads.json',
  array('filename' => $name, 'token' => $token),
  'POST',
  file_get_contents($someImage),
  array('Content-Type' => 'image/png')
);

Ref: https://developer.zendesk.com/rest_api/docs/core/attachments#uploading-files

[jwswj]

This looks fine. What are you using the uploads API for?

[vbuck]

I'm building an import tool to migrate data from an existing support ticket platform (Magento extension) to Zendesk. So part of the import process involves transferring attachments to obtain upload tokens which are added to the request bodies bound for imports/tickets.json.

[jwswj]

Cool - would love to hear more about the tool. Email me jsmale@zendesk.com 👍