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

core[patch]: deprecate hwchase17/langchain-hub, address path traversal (CVE-2024-28088) #18600

Merged

Conversation

efriis
Copy link
Member

@efriis efriis commented Mar 5, 2024

Deprecates the old langchain-hub repository. Does not deprecate the new https://smith.langchain.com/hub

@PinkDraconian has correctly raised that in the event someone is loading unsanitized user input into the try_load_from_hub function, they have the ability to load files from other locations in github than the hwchase17/langchain-hub repository.

This PR adds some more path checking to that function and deprecates the functionality in favor of the hub built into LangSmith.

Copy link

vercel bot commented Mar 5, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Visit Preview Mar 5, 2024 8:10pm

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. 🤖:bug Related to a bug, vulnerability, unexpected error with an existing feature labels Mar 5, 2024
@efriis efriis changed the title core[patch]: deprecate langchainhub, address path traversal core[patch]: deprecate hwchase17/langchain-hub, address path traversal Mar 5, 2024
@dosubot dosubot bot added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label Mar 5, 2024
@PinkDraconian
Copy link

Hi! Thank for deprecating this functionality.

Would you also please be so kind to change my report from "informational" to "accepted" on Huntr? That way I get the proper remuneration.
🔗 https://huntr.com/bounties/96398cab-29c0-461b-8ef7-010bc1114c84/

@efriis efriis merged commit e1924b3 into master Mar 5, 2024
95 checks passed
@efriis efriis deleted the erick/core-patch-deprecate-langchainhub-address-path-traversal branch March 5, 2024 20:49
thebhulawat pushed a commit to thebhulawat/langchain that referenced this pull request Mar 6, 2024
langchain-ai#18600)

Deprecates the old langchain-hub repository. Does *not* deprecate the
new https://smith.langchain.com/hub

@PinkDraconian has correctly raised that in the event someone is loading
unsanitized user input into the `try_load_from_hub` function, they have
the ability to load files from other locations in github than the
hwchase17/langchain-hub repository.

This PR adds some more path checking to that function and deprecates the
functionality in favor of the hub built into LangSmith.
@eyurtsev
Copy link
Collaborator

eyurtsev commented Mar 6, 2024

Address CVE: 2024-28088

https://nvd.nist.gov/vuln/detail/CVE-2024-28088

@eyurtsev eyurtsev changed the title core[patch]: deprecate hwchase17/langchain-hub, address path traversal core[patch]: deprecate hwchase17/langchain-hub, address path traversal (CVE-2024-28088) Mar 6, 2024
@eyurtsev
Copy link
Collaborator

eyurtsev commented Mar 6, 2024

@PinkDraconian we've contacted Huntr to change the status of your report, and will assess it today to assign a proper CVSS score.

Going forward we ask that if you disagree with a decision and have a good argument for that please reach out to security@langchain.dev prior to filing the CVE, so we can discuss. Filing the CVE effectively blocks users from being able to deploy their applications, without an option to upgrade to get a security patch, and in this case it's dubious that the code actually poses any risk to users.

@PinkDraconian
Copy link

My apologies for any inconvenience caused there. I will definitely take that advice for the future.

gkorland pushed a commit to FalkorDB/langchain that referenced this pull request Mar 30, 2024
langchain-ai#18600)

Deprecates the old langchain-hub repository. Does *not* deprecate the
new https://smith.langchain.com/hub

@PinkDraconian has correctly raised that in the event someone is loading
unsanitized user input into the `try_load_from_hub` function, they have
the ability to load files from other locations in github than the
hwchase17/langchain-hub repository.

This PR adds some more path checking to that function and deprecates the
functionality in favor of the hub built into LangSmith.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖:bug Related to a bug, vulnerability, unexpected error with an existing feature lgtm PR looks good. Use to confirm that a PR is ready for merging. size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants