Skip to content

feat: add supported extensions in appsettings#354

Open
ElenaStamenkovskaCC wants to merge 5 commits intomainfrom
feature/whitelist-extension-types
Open

feat: add supported extensions in appsettings#354
ElenaStamenkovskaCC wants to merge 5 commits intomainfrom
feature/whitelist-extension-types

Conversation

@ElenaStamenkovskaCC
Copy link
Copy Markdown
Collaborator

@ElenaStamenkovskaCC ElenaStamenkovskaCC commented Mar 26, 2026

What Changed?

  • This PR adds a fluent validator for FileUploadRequest. It check the validity of the files based on their extensions. It also checks for valid signature of the extension.

Why?

  • To improve security by preventing users from uploading files with fake extensions.
  • Ensures only supported file types are accepted, reducing potential runtime errors or security risks.

Closes #261

Type of Change

  • 🐛 Bug fix (fixes an issue without breaking existing functionality)

How Did You Test This?

  • Tested this by uploading files to some test collections, with Swagger.

What I tested:

  • [ ]
  • [ ]

How to test it:

Review Checklist

  • My code follows the project's style and conventions
  • I've reviewed my own code for obvious issues
  • I've added comments where the code might be confusing
  • I've updated relevant documentation (if needed)
  • I've tested my changes and they work as expected

Screenshots / Additional Context

@fvanderflier
Copy link
Copy Markdown
Collaborator

@ElenaStamenkovskaCC ElenaStamenkovskaCC marked this pull request as ready for review April 2, 2026 10:14
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.

Whitelist extension types

2 participants