Add DATA_UPLOAD_MAX_MEMORY_SIZE as an configurable variable #8190

Closed
opened 2025-12-29 20:33:36 +01:00 by adam · 3 comments
Owner

Originally created by @ziggekatten on GitHub (Jun 12, 2023).

Originally assigned to: @jeremystretch on GitHub.

NetBox version

v3.5.3

Feature type

New functionality

Proposed functionality

Add to settings.py: DATA_UPLOAD_MAX_MEMORY_SIZE = getattr(configuration, 'DATA_UPLOAD_MAX_MEMORY_SIZE', 2621440)

Use case

We use custom scripts, called through the API for data validation, and there is an hard limit on 2,6MB on the payload (body). We have cases were we are well above that (posting an JSON with all VM's fron an vCenter for validation), and I therefore suggest that variable is being added to the settings.py just as the file upload variable. Tests by setting this in settings.py our selves solves the challenge, but one must remember this when upgrading....

Database changes

None

External dependencies

None

Originally created by @ziggekatten on GitHub (Jun 12, 2023). Originally assigned to: @jeremystretch on GitHub. ### NetBox version v3.5.3 ### Feature type New functionality ### Proposed functionality Add to settings.py: DATA_UPLOAD_MAX_MEMORY_SIZE = getattr(configuration, 'DATA_UPLOAD_MAX_MEMORY_SIZE', 2621440) ### Use case We use custom scripts, called through the API for data validation, and there is an hard limit on 2,6MB on the payload (body). We have cases were we are well above that (posting an JSON with all VM's fron an vCenter for validation), and I therefore suggest that variable is being added to the settings.py just as the file upload variable. Tests by setting this in settings.py our selves solves the challenge, but one must remember this when upgrading.... ### Database changes None ### External dependencies None
adam added the status: acceptedtype: feature labels 2025-12-29 20:33:36 +01:00
adam closed this issue 2025-12-29 20:33:36 +01:00
Author
Owner

@jeremystretch commented on GitHub (Jun 14, 2023):

Does this need to be configurable? Should we just set it to a much higher (but sane) value, or disable it entirely?

@jeremystretch commented on GitHub (Jun 14, 2023): Does this need to be configurable? Should we just set it to a much higher (but sane) value, or disable it entirely?
Author
Owner

@ziggekatten commented on GitHub (Jun 15, 2023):

I suggest it to be configured so one can limit it to its own need, with the default as is (~2,5 MB).

@ziggekatten commented on GitHub (Jun 15, 2023): I suggest it to be configured so one can limit it to its own need, with the default as is (~2,5 MB).
Author
Owner

@github-actions[bot] commented on GitHub (Sep 14, 2023):

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. NetBox is governed by a small group of core maintainers which means not all opened issues may receive direct feedback. Do not attempt to circumvent this process by "bumping" the issue; doing so will result in its immediate closure and you may be barred from participating in any future discussions. Please see our contributing guide.

@github-actions[bot] commented on GitHub (Sep 14, 2023): This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. NetBox is governed by a small group of core maintainers which means not all opened issues may receive direct feedback. **Do not** attempt to circumvent this process by "bumping" the issue; doing so will result in its immediate closure and you may be barred from participating in any future discussions. Please see our [contributing guide](https://github.com/netbox-community/netbox/blob/develop/CONTRIBUTING.md).
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#8190