closeio / closeio-api

Python API Client for Close
http://developer.close.com/
MIT License
65 stars 47 forks source link

make sure TZ-Offset is an str #89

Closed eengoron closed 6 years ago

eengoron commented 6 years ago

There appears to be an issue with some versions of the requests library where we need to wrap tz_offset in a string.

More info on this here: https://github.com/requests/requests/issues/4333

The error some customers are getting is:

 File "/Library/Python/2.7/site-packages/closeio_api/__init__.py", line 56, in get
    return self.dispatch('get', endpoint)
  File "/Library/Python/2.7/site-packages/closeio_api/__init__.py", line 34, in dispatch
    headers={'Content-Type': 'application/json', 'X-TZ-Offset': self.tz_offset}
  File "/Library/Python/2.7/site-packages/requests/sessions.py", line 521, in get
    return self.request('GET', url, **kwargs)
  File "/Library/Python/2.7/site-packages/requests/sessions.py", line 494, in request
    prep = self.prepare_request(req)
  File "/Library/Python/2.7/site-packages/requests/sessions.py", line 437, in prepare_request
    hooks=merge_hooks(request.hooks, self.hooks),
  File "/Library/Python/2.7/site-packages/requests/models.py", line 306, in prepare
    self.prepare_headers(headers)
  File "/Library/Python/2.7/site-packages/requests/models.py", line 440, in prepare_headers
    check_header_validity(header)
  File "/Library/Python/2.7/site-packages/requests/utils.py", line 872, in check_header_validity
    "bytes, not %s" % (name, value, type(value)))
requests.exceptions.InvalidHeader: Value for header {X-TZ-Offset: -8} must be of type str or bytes, not <type 'int'>