openstack/python-swiftclient
Code Issues Proposed changes

Merge "don't use mutable defaults in kwargs"

Browse Source
This commit is contained in:
Jenkins 2014-04-01 15:21:22 +00:00 committed by Gerrit Code Review
commit 98e36e9438
1 changed files with 4 additions and 2 deletions

6
swiftclient/client.py

@ -189,8 +189,10 @@ class HTTPConnection:
""" Final wrapper before requests call, to be patched in tests """
return requests.request(*arg, **kwarg)
def request(self, method, full_path, data=None, headers={}, files=None):
def request(self, method, full_path, data=None, headers=None, files=None):
""" Encode url and header, then call requests.request """
if headers is None:
headers = {}
headers = dict((encode_utf8(x.lower()), encode_utf8(y)) for x, y in
headers.items())
# set a default User-Agent header if it wasn't passed in
@ -204,7 +206,7 @@ class HTTPConnection:
files=files, **self.requests_args)
return self.resp
def putrequest(self, full_path, data=None, headers={}, files=None):
def putrequest(self, full_path, data=None, headers=None, files=None):
"""
Use python-requests files upload