From: Kenneth Reitz <me@kennethreitz.com>
Date: Sat, 12 Nov 2011 20:47:04 +0000 (-0800)
Subject: keep-alive as config option only
X-Git-Tag: v0.8.0~27
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=02408973c6da4bd89401d2a1994b4ea26dc260c9;p=services%2Fpython-requests.git

keep-alive as config option only

better defaults
---

diff --git a/requests/defaults.py b/requests/defaults.py
index eee4ae5..7a5a3fb 100644
--- a/requests/defaults.py
+++ b/requests/defaults.py
@@ -35,8 +35,8 @@ defaults['base_headers'] = {
 defaults['verbose'] = None
 defaults['max_redirects'] = 30
 defaults['decode_unicode'] = True
-defaults['keep_alive'] = True
 defaults['pool_connections'] = 10
-defaults['pool_maxsize'] = 1
+defaults['pool_maxsize'] = 10
 defaults['max_retries'] = 0
 defaults['safe_mode'] = False
+defaults['keep_alive'] = True
diff --git a/requests/sessions.py b/requests/sessions.py
index 6f7270b..a1fb6e5 100644
--- a/requests/sessions.py
+++ b/requests/sessions.py
@@ -63,8 +63,7 @@ class Session(object):
         proxies=None,
         hooks=None,
         params=None,
-        config=None,
-        keep_alive=True):
+        config=None):
 
         self.headers = headers or {}
         self.cookies = cookies or {}
@@ -74,7 +73,6 @@ class Session(object):
         self.hooks = hooks or {}
         self.params = params or {}
         self.config = config or {}
-        self.keep_alive = keep_alive
 
         for (k, v) in defaults.items():
             self.config.setdefault(k, v)
@@ -136,8 +134,12 @@ class Session(object):
 
         method = str(method).upper()
 
-        if cookies is None:
-            cookies = {}
+        # Default empty dicts for dict params.
+        cookies = {} if cookies is None else cookies
+        data = {} if data is None else data
+        files = {} if files is None else files
+        headers = {} if headers is None else headers
+        params = {} if params is None else params
 
         # Expand header values
         if headers:
@@ -221,7 +223,7 @@ class Session(object):
         return self.request('HEAD', url, **kwargs)
 
 
-    def post(self, url, data='', **kwargs):
+    def post(self, url, data=None, **kwargs):
         """Sends a POST request. Returns :class:`Response` object.
 
         :param url: URL for the new :class:`Request` object.
@@ -232,7 +234,7 @@ class Session(object):
         return self.request('post', url, data=data, **kwargs)
 
 
-    def put(self, url, data='', **kwargs):
+    def put(self, url, data=None, **kwargs):
         """Sends a PUT request. Returns :class:`Response` object.
 
         :param url: URL for the new :class:`Request` object.
@@ -243,7 +245,7 @@ class Session(object):
         return self.request('put', url, data=data, **kwargs)
 
 
-    def patch(self, url, data='', **kwargs):
+    def patch(self, url, data=None, **kwargs):
         """Sends a PATCH request. Returns :class:`Response` object.
 
         :param url: URL for the new :class:`Request` object.
@@ -251,7 +253,7 @@ class Session(object):
         :param **kwargs: Optional arguments that ``request`` takes.
         """
 
-        return self.request('patch', url,  data='', **kwargs)
+        return self.request('patch', url,  data=data, **kwargs)
 
 
     def delete(self, url, **kwargs):