jose-correia / brain

Infrastructure for a technological conference, serving both the administration app and attendees app. Has a role-based authentication and can also be used to manage the website of the event.
https://brain.jeec.ist/admin/
GNU General Public License v3.0
1 stars 2 forks source link

Add image upload size limit #18

Closed jose-correia closed 4 years ago

jose-correia commented 4 years ago

Should be below 100kb

jose-correia commented 4 years ago

@pfernandes21 someone with the minimal python experience should be able to handle this

pfernandes21 commented 4 years ago

@rafaelandrealves is on it

jose-correia commented 4 years ago

@rafaelandrealves @pfernandes21 adding a condition that if the file size is bigger then a config threshold should be enough.

jose-correia commented 4 years ago

keep in mind that every issue should have it's branch and posterior merge request

rafaelandrealves commented 4 years ago

@rafaelandrealves @pfernandes21 adding a condition that if the file size is bigger then a config threshold should be enough.

  • Add condition to upload_image_service.py
  • Add max_image_size to config.py Config() object

I will do my best!