Merge pull request ej2#66 from subodh-malgonde/master

ej2 · web-flow · commit 665621d3f259 · 2017-01-23T09:06:55.000-06:00
Fixed attachable upload error
diff --git a/quickbooks/client.py b/quickbooks/client.py
@@ -9,6 +9,7 @@
 import json
 import os
 from .exceptions import QuickbooksException, SevereException, AuthorizationException
+import base64
 
 try:
     from rauth import OAuth1Session, OAuth1Service
@@ -254,7 +255,9 @@ def make_request(self, request_type, url, request_body=None, content_type='appli
                 'Connection': 'close'
             })
 
-            binary_data = attachment.read()
+            binary_data = str(base64.b64encode(attachment.read()), 'utf-8')
+
+            content_type = json.loads(request_body)['ContentType']
 
             request_body = textwrap.dedent(
                 """
@@ -266,13 +269,14 @@ def make_request(self, request_type, url, request_body=None, content_type='appli
 
                 --%s
                 Content-Disposition: form-data; name="file_content_01"
-                Content-Type: application/pdf
+                Content-Type: %s
+                Content-Transfer-Encoding: base64
 
                 %s
 
                 --%s--
                 """
-            ) % (boundary, request_body, boundary, binary_data, boundary)
+            ) % (boundary, request_body, boundary, content_type, binary_data, boundary)
 
         req = self.session.request(
             request_type, url, True, self.company_id,
