Skip to content

Commit

Permalink
Fix linting issues
Browse files Browse the repository at this point in the history
  • Loading branch information
nateprewitt committed May 22, 2023
1 parent 147c851 commit 6e5b15d
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 9 deletions.
2 changes: 1 addition & 1 deletion requests/sessions.py
Original file line number Diff line number Diff line change
Expand Up @@ -326,7 +326,7 @@ def rebuild_proxies(self, prepared_request, proxies):

# urllib3 handles proxy authorization for us in the standard adapter.
# Avoid appending this to TLS tunneled requests where it may be leaked.
if not scheme.startswith('https') and username and password:
if not scheme.startswith("https") and username and password:
headers["Proxy-Authorization"] = _basic_auth_str(username, password)

return new_proxies
Expand Down
17 changes: 9 additions & 8 deletions tests/test_requests.py
Original file line number Diff line number Diff line change
Expand Up @@ -647,25 +647,26 @@ def test_proxy_authorization_preserved_on_request(self, httpbin):

assert sent_headers.get("Proxy-Authorization") == proxy_auth_value


@pytest.mark.parametrize(
"url,has_proxy_auth",
(
('http://example.com', True),
('https://example.com', False),
("http://example.com", True),
("https://example.com", False),
),
)
def test_proxy_authorization_not_appended_to_https_request(self, url, has_proxy_auth):
def test_proxy_authorization_not_appended_to_https_request(
self, url, has_proxy_auth
):
session = requests.Session()
proxies = {
'http': 'http://test:pass@localhost:8080',
'https': 'http://test:pass@localhost:8090',
"http": "http://test:pass@localhost:8080",
"https": "http://test:pass@localhost:8090",
}
req = requests.Request('GET', url)
req = requests.Request("GET", url)
prep = req.prepare()
session.rebuild_proxies(prep, proxies)

assert ('Proxy-Authorization' in prep.headers) is has_proxy_auth
assert ("Proxy-Authorization" in prep.headers) is has_proxy_auth

def test_basicauth_with_netrc(self, httpbin):
auth = ("user", "pass")
Expand Down

0 comments on commit 6e5b15d

Please sign in to comment.