Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Snyk:Medium] requests Information Exposure (due by 07/23/2023) #5459

Closed
1 task
cnlucas opened this issue May 24, 2023 · 0 comments · Fixed by #5488
Closed
1 task

[Snyk:Medium] requests Information Exposure (due by 07/23/2023) #5459

cnlucas opened this issue May 24, 2023 · 0 comments · Fixed by #5488
Assignees
Labels
Security: general General security concern or issue Security: moderate Remediate within 60 days
Milestone

Comments

@cnlucas
Copy link
Member

cnlucas commented May 24, 2023

Introduced through
requests@2.25.1, prance@0.22.11.4.0 and others
Fixed in
requests@2.31.0

Exploit maturity
No known exploit

Detailed paths and remediation

Introduced through: project@0.0.0 › requests@2.25.1
Fix: Upgrade requests to version 2.31.0

Introduced through: project@0.0.0 › prance@0.22.11.4.0 › requests@2.25.1
Fix: Pin requests to version 2.31.0
Introduced through: project@0.0.0 › requests-aws4auth@1.0 › requests@2.25.1
Fix: Pin requests to version 2.31.0

…and 1 more
Security information
Factors contributing to the scoring:

Snyk: [CVSS 6.1](https://security.snyk.io/vuln/SNYK-PYTHON-REQUESTS-5595532) - Medium Severity
NVD: Not available. NVD has not yet published its analysis.

Why are the scores different? Learn how Snyk evaluates vulnerability scores
Overview

Affected versions of this package are vulnerable to Information Exposure by leaking Proxy-Authorization headers to destination servers during redirects to an HTTPS origin. This is a result of how rebuild_proxies is used to recompute and reattach the Proxy-Authorization header to requests when redirected.

NOTE: This behavior has only been observed to affect proxied requests when credentials are supplied in the URL user information component (e.g. https://username:password@proxy:8080), and only when redirecting to HTTPS:

HTTP → HTTPS: leak

HTTPS → HTTP: no leak

HTTPS → HTTPS: leak

HTTP → HTTP: no leak

For HTTP connections sent through the proxy, the proxy will identify the header in the request and remove it prior to forwarding to the destination server. However when sent over HTTPS, the Proxy-Authorization header must be sent in the CONNECT request as the proxy has no visibility into further tunneled requests. This results in Requests forwarding the header to the destination server unintentionally, allowing a malicious actor to potentially exfiltrate those credentials.

Completion criteria

(What does the end state look like - as long as this task(s) is done, this work is complete)

  • This vulnerability is remediated or ignored (because we have confirmed it doesn't affect us)
@cnlucas cnlucas added Security: moderate Remediate within 60 days Security: general General security concern or issue labels May 24, 2023
@cnlucas cnlucas mentioned this issue May 24, 2023
3 tasks
@tmpayton tmpayton mentioned this issue May 31, 2023
2 tasks
@tmpayton tmpayton mentioned this issue Jun 7, 2023
3 tasks
@pkfec pkfec mentioned this issue Jul 6, 2023
2 tasks
@cnlucas cnlucas added this to the Sprint 22.1 milestone Jul 6, 2023
@cnlucas cnlucas self-assigned this Jul 6, 2023
@pkfec pkfec mentioned this issue Jul 12, 2023
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Security: general General security concern or issue Security: moderate Remediate within 60 days
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant