Skip to content
This repository has been archived by the owner on May 31, 2022. It is now read-only.

[FEATURE REQUEST] support for allowlisting uploads for improved security #123

Closed
rjd22 opened this issue Nov 16, 2021 · 2 comments · Fixed by #134
Closed

[FEATURE REQUEST] support for allowlisting uploads for improved security #123

rjd22 opened this issue Nov 16, 2021 · 2 comments · Fixed by #134
Assignees
Labels
P-HTTP Plugin: HTTP Y-high Priority: High
Milestone

Comments

@rjd22
Copy link

rjd22 commented Nov 16, 2021

Is your feature request related to a problem? Please describe.
At this moment you can deny what files are uploaded by the uploads.forbid configuration option. Most of the time applications have a really limited scope of files they want to process. Adding an uploads.allow_only option would improve server security by a lot because the user only needs to know what he wants to support.

Describe the solution you'd like
Have a config option like the following, to only support uploading a limited set:

  # File uploading settings.
  uploads:
    # Allow only files with the following extensions to upload. This will overrule the `forbid` config option when not empty
    #
    # Default: []
    allow_only: [".jpg", ".png", ".doc", ".docx"]
@rustatian rustatian transferred this issue from roadrunner-server/roadrunner Nov 16, 2021
@rustatian rustatian added the P-HTTP Plugin: HTTP label Nov 16, 2021
@rustatian rustatian moved this to Todo in RoadRunner Nov 16, 2021
@rustatian
Copy link
Member

Hey @rjd22 . Thanks for the FR. Agree with you. Will plan this feature for the next after 2.6 release.

@rustatian rustatian added this to the 2.6.0 milestone Nov 24, 2021
@rustatian rustatian added the Y-high Priority: High label Nov 24, 2021
@rustatian
Copy link
Member

@rjd22 Will be in the v2.6.0

Repository owner moved this from Todo to Done in RoadRunner Nov 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
P-HTTP Plugin: HTTP Y-high Priority: High
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants