Skip to content

Fix: Unencrypted Web Request Could Expose Sensitive Data in docext/app/utils.py #44

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kira-offgrid
Copy link
Contributor

Context and Purpose:

This PR automatically remediates a security vulnerability:

  • Description: Detected a request using 'http://'. This request will be unencrypted, and attackers could listen into traffic on the network and be able to obtain sensitive information. Use 'https://' instead.
  • Rule ID: python.lang.security.audit.insecure-transport.requests.request-with-http.request-with-http
  • Severity: LOW
  • File: docext/app/utils.py
  • Lines Affected: 15 - 15

This change is necessary to protect the application from potential security risks associated with this vulnerability.

Solution Implemented:

The automated remediation process has applied the necessary changes to the affected code in docext/app/utils.py to resolve the identified issue.

Please review the changes to ensure they are correct and integrate as expected.

…ith-http.request-with-http-docext-app-utils.py
@mandalsouvik3333
Copy link
Collaborator

@kira-offgrid If the vLLM is hosted locally, then HTTPS won't work unless you have set it up. Local calls won't need HTTPS. You can probably first try with HTTPS, then HTTP.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants