libcloud-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <>
Subject [GitHub] hldh214 opened a new pull request #1277: fix scheme issue in https case
Date Mon, 28 Jan 2019 13:29:28 GMT
hldh214 opened a new pull request #1277: fix scheme issue in https case
   ## fix scheme issue in https case
   ### Description
   Hello everyone, first apologize my poor english
   im beginner and i have a question about http proxy(
   from here:
   if parsed.scheme != 'http':
       raise ValueError('Only http proxies are supported')
   i guess this code means we cant use proxy on https requests, sounds not cool for nowadays
   from here:
   self.session.proxies = {
       # requests assume `self.proxy_scheme` as http/https protocol
       # so if we want proxy our https request
       # we need 2 add `'https': ''` instead of `http` k/v pair
       self.proxy_scheme: proxy_url
   i dont know if this `self.proxy_scheme` has other meaning or not
   please let me know if i have any misunderstand or mistake, Thanks <3
   ### Status
   - done, ready for review
   ### Checklist (tick everything that applies)
   - [x] [Code linting](
(required, can be done after the PR checks)
   - [x] Documentation
   - [x] [Tests](
   - [x] [ICLA](
(required for bigger changes)

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:

With regards,
Apache Git Services

View raw message