[PR #2902] [MERGED] Audio file validations #4168

Closed
opened 2026-02-22 18:35:17 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/Mintplex-Labs/anything-llm/pull/2902
Author: @shatfield4
Created: 12/26/2024
Status: Merged
Merged: 12/30/2024
Merged by: @timothycarambat

Base: masterHead: patch-low-bitrate-dos


📝 Commits (3)

  • 26ae044 add audio file validations
  • a6ec0da patch sharp to support wavfile parsing
  • 4e0076c Merge branch 'master' into patch-low-bitrate-dos

📊 Changes

3 files changed (+194 additions, -1 deletions)

View changed files

📝 collector/package.json (+1 -0)
📝 collector/utils/WhisperProviders/localWhisper.js (+38 -0)
📝 collector/yarn.lock (+155 -1)

📄 Description

Pull Request Type

  • feat
  • 🐛 fix
  • ♻️ refactor
  • 💄 style
  • 🔨 chore
  • 📝 docs

Relevant Issues

resolves #xxx

What is in this change?

  • Add validations to audio file uploads
  • Prevent DOS attack caused by uploading malicious low sample rate audio files

Additional Information

Developer Validations

  • I ran yarn lint from the root of the repo & committed changes
  • Relevant documentation has been updated
  • I have tested my code functionality
  • Docker build succeeds locally

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/Mintplex-Labs/anything-llm/pull/2902 **Author:** [@shatfield4](https://github.com/shatfield4) **Created:** 12/26/2024 **Status:** ✅ Merged **Merged:** 12/30/2024 **Merged by:** [@timothycarambat](https://github.com/timothycarambat) **Base:** `master` ← **Head:** `patch-low-bitrate-dos` --- ### 📝 Commits (3) - [`26ae044`](https://github.com/Mintplex-Labs/anything-llm/commit/26ae0445469d11ed89e0979fcc3ba170d39c8e12) add audio file validations - [`a6ec0da`](https://github.com/Mintplex-Labs/anything-llm/commit/a6ec0daa8bfe60cbc57939999815d8ee0d0242b0) patch sharp to support wavfile parsing - [`4e0076c`](https://github.com/Mintplex-Labs/anything-llm/commit/4e0076c98787621c137d3c6aa4616d00fe1fc1f1) Merge branch 'master' into patch-low-bitrate-dos ### 📊 Changes **3 files changed** (+194 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `collector/package.json` (+1 -0) 📝 `collector/utils/WhisperProviders/localWhisper.js` (+38 -0) 📝 `collector/yarn.lock` (+155 -1) </details> ### 📄 Description ### Pull Request Type <!-- For change type, change [ ] to [x]. --> - [x] ✨ feat - [ ] 🐛 fix - [ ] ♻️ refactor - [ ] 💄 style - [ ] 🔨 chore - [ ] 📝 docs ### Relevant Issues <!-- Use "resolves #xxx" to auto resolve on merge. Otherwise, please use "connect #xxx" --> resolves #xxx ### What is in this change? <!-- Describe the changes in this PR that are impactful to the repo. --> - Add validations to audio file uploads - Prevent DOS attack caused by uploading malicious low sample rate audio files ### Additional Information <!-- Add any other context about the Pull Request here that was not captured above. --> ### Developer Validations <!-- All of the applicable items should be checked. --> - [x] I ran `yarn lint` from the root of the repo & committed changes - [x] Relevant documentation has been updated - [x] I have tested my code functionality - [x] Docker build succeeds locally --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-22 18:35:17 -05:00
yindo closed this issue 2026-02-22 18:35:17 -05:00
yindo changed title from [PR #2902] Audio file validations to [PR #2902] [MERGED] Audio file validations 2026-06-05 15:17:11 -04:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Mintplex-Labs/anything-llm#4168