downloads should utilize streaming

Author: maxtepkeev

CHANGELOG.rst

@@ -58,6 +58,7 @@ Changelog
   resource that already exists (`Issue #182 <https://github.com/maxtepkeev/python-redmine/issues/182>`__)
 - Enumeration, Version, Group and Notes ``custom_fields`` attribute was returned as a list of dicts instead
   of being converted to ``ResourceSet`` object
+- Downloads were downloaded fully into memory instead of being streamed as needed
 - ``ResourceRequirementsError`` exception was broken since v2.0.0
 - RedmineUP CRM Contact and Deal resources sometimes weren't converted to Resource objects using Search API
 

redminelib/__init__.py

@@ -124,8 +124,8 @@ def download(self, url, savepath=None, filename=None, params=None):
         :param string filename: (optional). Name that will be used for the file.
         :param dict params: (optional). Params to send in the query string.
         """
-        with self.session(return_raw_response=True):
-            response = self.engine.request('get', url, params=dict(params or {}, **{'stream': True}))
+        with self.session(requests={'stream': True}, return_raw_response=True):
+            response = self.engine.request('get', url, params=params or {})
 
         # If a savepath wasn't provided we return a response directly
         # so a user can have maximum control over response data