Opened 2 years ago

Closed 2 years ago

Last modified 7 months ago

#33852 closed New feature (wontfix)

Ability to exclude a specific view or form from DATA_UPLOAD_MAX_NUMBER_FIELDS

Reported by: Victor Kovalev Owned by: nobody
Component: Forms Version: 4.0
Severity: Normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Use case & rationale:

We have a large, complicated CMS application. One of the advanced power views this application has uses a form that requires an unusually large number of fields. This particular view is only accessible by Admin users, behind 2 separate authentication gates. So, we'd like that view to support having this form with a large number of fields.

Having said that, given that one of the stated purposes of DATA_UPLOAD_MAX_NUMBER_FIELDS is protection from DoS, increasing the DATA_UPLOAD_MAX_NUMBER_FIELDS value for our entire app across the board (or setting it to None to disable the check) seems counter-productive in terms of security / DoS-protection.

I would really love to have a way to specify "hey, this particular view or form is OK to use a myriad of fields" without affecting the rest of the app. Hence the feature request!

Thank you very much for all the wonderful work y'all do with the Django framework.

Change History (2)

comment:1 by Mariusz Felisiak, 2 years ago

Resolution: wontfix
Status: newclosed

Thanks for this suggestion, however it seems to be really niche and not worth maintaining by Django itself. I'm also not sure how forms with 1000+ fields can be filled by a human.

Please follow the triaging guidelines with regards to wontfix tickets and take this to DevelopersMailingList, where you'll reach a wider audience and see what other think,

comment:2 by Jonas Dittrich, 7 months ago

I'm also not sure how forms with 1000+ fields can be filled by a human.

With (prefilled) Django's ModelMultipleChoiceField this can happen quite easily. I'd consider reopening this ticket.

Note: See TracTickets for help on using tickets.
Back to Top