[CMIS-993] String concatenation error when specifying custom headers with BrowserBinding

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: cmislib 0.6.0
• Component/s: python-cmislib

Environment

Linux, python 2.7x, Alfresco 5.0.x, cmislib dev

• Flags: Patch
• Assignee: Jeffrey Taylor Potts
• Reporter: Laurent Mignon
• Created: 05/Aug/16 06:03
• Updated: 29/Dec/16 23:24
• Resolved: 30/Aug/16 23:07

Description

diff --git a/src/cmislib/net.py b/src/cmislib/net.py
index 59b3038..9a3db18 100644
--- a/src/cmislib/net.py
+++ b/src/cmislib/net.py
@@ -51,7 +51,7 @@ class RESTService(object):
             if 'headers' in kwargs:
                 headers = kwargs['headers']
                 del kwargs['headers']
-                self.logger.debug('Headers passed in:' + headers)
+                self.logger.debug('Headers passed in: %s', headers)
             if url.find('?') >= 0:
                 url = url + '&' + urlencode(kwargs)
             else:
@@ -74,7 +74,7 @@ class RESTService(object):
             if 'headers' in kwargs:
                 headers = kwargs['headers']
                 del kwargs['headers']
-                self.logger.debug('Headers passed in:' + headers)
+                self.logger.debug('Headers passed in %s', headers)
             if url.find('?') >= 0:
                 url = url + '&' + urlencode(kwargs)
             else:
@@ -107,7 +107,7 @@ class RESTService(object):
             if 'headers' in kwargs:
                 headers = kwargs['headers']
                 del kwargs['headers']
-                self.logger.debug('Headers passed in:' + headers)
+                self.logger.debug('Headers passed in: %s',  headers)
             if url.find('?') >= 0:
                 url = url + '&' + urlencode(kwargs)
             else:
@@ -141,7 +141,7 @@ class RESTService(object):
             if 'headers' in kwargs:
                 headers = kwargs['headers']
                 del kwargs['headers']
-                self.logger.debug('Headers passed in:' + headers)
+                self.logger.debug('Headers passed in: %s' % headers)
             if url.find('?') >= 0:
                 url = url + '&' + urlencode(kwargs)
             else:

https://github.com/lmignon/python-cmislib/commit/159f0202d7dd012230b31a42ef13ef0d03b46f96?diff=unified