Skip to content

Conversation

@claudiamurialdo
Copy link
Collaborator

Fix error: Multipart body length limit 134217728 exceeded making that limit configurable and with a default of 500MB instead of 128MB
Issue:94849

… 134217728 exceeded making that limit configurable and with a default of 500MB instead of 128MB
@claudiamurialdo claudiamurialdo temporarily deployed to external-storage-tests June 23, 2022 21:25 Inactive
@genexusbot
Copy link
Collaborator

Cherry pick to beta success

ggallotti
ggallotti previously approved these changes Jun 24, 2022
@claudiamurialdo claudiamurialdo temporarily deployed to external-storage-tests June 24, 2022 14:54 Inactive
@genexusbot
Copy link
Collaborator

Cherry pick to beta success

…fig which considers the environment variables overriding values defined at appsettings.json as reviewed by @ggallotti
@claudiamurialdo claudiamurialdo temporarily deployed to external-storage-tests June 28, 2022 19:28 Inactive
@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@ggallotti
Copy link
Member

ggallotti commented Jun 28, 2022

Added these new Env Vars to document:
https://iwiki.genexus.com/wiki.aspx?31354,Configuraci%c3%b3n+Apps+GeneXus+en+Ambientes+Contenerizados,

@claudiamurialdo claudiamurialdo merged commit 4535465 into master Jun 28, 2022
@claudiamurialdo claudiamurialdo deleted the mulitpart-body-length-limit branch June 28, 2022 22:43
claudiamurialdo added a commit that referenced this pull request Jun 29, 2022
…#617)

* Fix error System.IO.InvalidDataException: Multipart body length limit 134217728 exceeded making that limit configurable and with a default of 500MB instead of 128MB

* Adjust DEFAULT_MAX_FILE_UPLOAD_SIZE_BYTES to 528MB

* Settings read at Startup were not using the standard mechanism of Config which considers the environment variables overriding values defined at appsettings.json as reviewed by @ggallotti

(cherry picked from commit 4535465)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants