Fixes ability to download PDFs by setting the accept header.

freakboy3742 · Jun 10, 2015 · ae2dc3a · ae2dc3a
1 parent 5eaf4ed
commit ae2dc3a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/xero/__init__.py b/xero/__init__.py
@@ -1,4 +1,4 @@
 from .api import Xero
 
-__version__ = "0.7.0-alpha1"
+__version__ = "0.7.0-alpha2"
 
diff --git a/xero/manager.py b/xero/manager.py
@@ -163,8 +163,15 @@ def wrapper(*args, **kwargs):
 
  if headers is None:
  headers = {}
- headers['Accept'] = 'application/json'
+
+ # Use the JSON API by default, but remember we might request a PDF (application/pdf)
+ # so don't force the Accept header.
+ if 'Accept' not in headers:
+ headers['Accept'] = 'application/json'
+
+ # Set a user-agent so Xero knows the traffic is coming from pyxero
  headers['User-Agent'] = 'pyxero/%s ' % VERSION + requests.utils.default_user_agent()
+
  response = getattr(requests, method)(
  uri, data=body, headers=headers, auth=self.credentials.oauth,
  params=params, cert=cert, timeout=timeout)