Merge pull request #25 from pusher/support-proxy-env-vars

Add support for loading proxy config from the environment

Author: jonathanlloyd

CHANGELOG.md

@@ -5,6 +5,9 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
+### Added
+ - Support for `HTTP_PROXY` environment variables. This should enable use of the
+   SDK on the PythonAnywhere platform (see issue #24)
 
 ## [1.1.0] - 2019-02-06
 ### Added

pusher_push_notifications/__init__.py

@@ -3,6 +3,7 @@
 import copy
 import datetime
 import json
+import os
 import re
 import time
 import warnings
@@ -76,6 +77,13 @@ def _make_url(scheme, host, path):
     ])
 
 
+def _get_proxies_from_env():
+    return {
+        'http': os.environ.get('HTTP_PROXY') or os.environ.get('http_proxy'),
+        'https': os.environ.get('HTTPS_PROXY') or os.environ.get('https_proxy'),
+    }
+
+
 class PushNotifications(object):
     """Pusher Push Notifications API client
     This client class can be used to publish notifications to the Pusher
@@ -117,6 +125,14 @@ def _make_request(self, method, path, path_params, body=None):
         url = _make_url(scheme='https', host=self.endpoint, path=path)
 
         session = requests.Session()
+        # We've had multiple support requests about this library not working
+        # on PythonAnywhere (a popular python deployment platform)
+        # They require that proxy servers be loaded from the environment when
+        # making requests (on their free plan).
+        # This reintroduces the proxy support that is the default in requests
+        # anyway.
+        session.proxies = _get_proxies_from_env()
+
         request = requests.Request(
             method,
             url,