Skip to content

fix: Address Snyk SAST findings - #12820

Open
mfish911 wants to merge 15 commits into
LibreChat-AI:devfrom
mfish911:snyk-sast
Open

mfish911 wants to merge 15 commits into
LibreChat-AI:devfrom
mfish911:snyk-sast

Conversation

@mfish911

Copy link
Copy Markdown
Contributor

Summary

My company uses Snyk SAST as part of security reviews and came back with a list of findings including path traversal, SSRF, XSS and ReDoS potential issues. This pull request addresses all the findings.

Change Type

Please delete any irrelevant options.

  • Bug fix (non-breaking change which fixes an issue)

Testing

Validated that core functionality continues to work with these updates.

Test Configuration:

Checklist

Please delete any irrelevant options.

  • I have performed a self-review of my own code

@mfish911
mfish911 marked this pull request as draft April 25, 2026 03:40
@mfish911
mfish911 marked this pull request as ready for review April 27, 2026 14:41
@mfish911 mfish911 changed the title Address Snyk SAST findings fix: Address Snyk SAST findings Apr 27, 2026

@danny-avila danny-avila left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Check the eslint warnings

@mfish911
mfish911 requested a review from danny-avila April 29, 2026 23:42
@codegraph-librechat codegraph-librechat Bot added 🗺️ Server Core codegraph: the taxonomy area this belongs to (classifier, confidence ≥ 0.9) 🗺️ Platform Security codegraph: the taxonomy area this belongs to (classifier, confidence ≥ 0.9) labels Oct 3, 2026
@github-actions
github-actions Bot changed the base branch from main to dev October 9, 2026 18:23
@github-actions

github-actions Bot commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

👋 Thanks for the contribution! LibreChat merges all changes into dev first — main only moves at release time — so this pull request's base branch was switched from main to dev automatically.

Nothing is needed from you; your commits, reviews and discussion are unchanged. If the diff now shows files you did not touch, rebase onto dev:

git remote add upstream https://github.com/LibreChat-AI/LibreChat.git
git fetch upstream dev
git rebase upstream/dev
git push --force-with-lease

Maintainers: apply the target: main label and restore the base branch if this one genuinely belongs on main.

@codegraph-librechat codegraph-librechat Bot added the 🗺️ Core Infra & Sec codegraph: the taxonomy area this belongs to (classifier, confidence ≥ 0.9) label Oct 15, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🗺️ Core Infra & Sec codegraph: the taxonomy area this belongs to (classifier, confidence ≥ 0.9) 🗺️ Platform Security codegraph: the taxonomy area this belongs to (classifier, confidence ≥ 0.9) 🛡️ security review 🗺️ Server Core codegraph: the taxonomy area this belongs to (classifier, confidence ≥ 0.9)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants