Skip to content

Commit 2a851d6

Browse files
authored
Merge pull request #108 from polis-community/remove-cloudflare-bypass-adapter
Remove cloudflare bypass adapter
2 parents 8df46a4 + 33e12a7 commit 2a851d6

2 files changed

Lines changed: 2 additions & 32 deletions

File tree

reddwarf/data_loader.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
import csv
88
from io import StringIO
99
from reddwarf.models import Vote, Statement
10-
from reddwarf.helpers import CachedLimiterSession, CloudflareBypassHTTPAdapter
10+
from reddwarf.helpers import CachedLimiterSession
1111

1212
ua = UserAgent()
1313

@@ -537,8 +537,6 @@ def init_http_client(self):
537537
)
538538
else:
539539
self.session = LimiterSession(per_second=5)
540-
adapter = CloudflareBypassHTTPAdapter()
541-
self.session.mount(self.polis_instance_url, adapter)
542540
self.session.headers = {
543541
"User-Agent": ua.random,
544542
}

reddwarf/helpers.py

Lines changed: 1 addition & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,4 @@
1-
from urllib3.util import ssl_
21
from requests import Session
3-
from requests.adapters import HTTPAdapter
42
from requests_cache import CacheMixin
53
from requests_ratelimiter import LimiterMixin
64

@@ -10,30 +8,4 @@ class CachedLimiterSession(CacheMixin, LimiterMixin, Session):
108
LimiterSession and CachedSession.
119
1210
See: See: https://requests-cache.readthedocs.io/en/stable/user_guide/compatibility.html#requests-ratelimiter
13-
"""
14-
15-
class CloudflareBypassHTTPAdapter(HTTPAdapter):
16-
"""
17-
A TransportAdapter that forces TLSv1.3 in Requests, so that Cloudflare doesn't flag us.
18-
19-
Source: https://lukasa.co.uk/2017/02/Configuring_TLS_With_Requests/
20-
"""
21-
22-
def init_patched_ssl_context(self):
23-
context = ssl_.create_urllib3_context()
24-
context.load_default_certs()
25-
# Only available in Python 3.7
26-
if hasattr(ssl_, "TLSVersion"):
27-
context.minimum_version = ssl_.TLSVersion.TLSv1_3
28-
else:
29-
context.options |= ssl_.OP_NO_TLSv1_2
30-
31-
return context
32-
33-
def init_poolmanager(self, *args, **kwargs):
34-
kwargs['ssl_context'] = self.init_patched_ssl_context()
35-
return super(CloudflareBypassHTTPAdapter, self).init_poolmanager(*args, **kwargs)
36-
37-
def proxy_manager_for(self, *args, **kwargs):
38-
kwargs['ssl_context'] = self.init_patched_ssl_context()
39-
return super(CloudflareBypassHTTPAdapter, self).proxy_manager_for(*args, **kwargs)
11+
"""

0 commit comments

Comments
 (0)