From 9a0691bf28532831df814ccfda2c153f62a863ed Mon Sep 17 00:00:00 2001 From: Julius Flimmel Date: Mon, 14 Mar 2022 11:03:36 +0100 Subject: [PATCH 1/2] feat: make content size limit configurable from app config #19311 --- src/zsl/application/service_application.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/src/zsl/application/service_application.py b/src/zsl/application/service_application.py index ad41a70..310cc93 100644 --- a/src/zsl/application/service_application.py +++ b/src/zsl/application/service_application.py @@ -29,6 +29,7 @@ from zsl._state import set_current_app from zsl.application.initialization_context import InitializationContext +from zsl.task import app from zsl.utils.warnings import deprecated from zsl.version import version @@ -137,6 +138,8 @@ def _configure(self, config_object=None): if zsl_settings is not None: self.config.from_envvar(SETTINGS_ENV_VAR_NAME) + self._configure_flask_app() + def _initialize(self): """Run the initializers.""" ctx = self.injector.get(InitializationContext) @@ -179,6 +182,10 @@ def _configure_injector(self, modules): self.logger.debug("Injector configuration with modules {0}.".format(modules)) self._dependencies_initialized = True + def _configure_flask_app(self): + if 'MAX_CONTENT_LENGTH' in self.config: + app.config['MAX_CONTENT_LENGTH'] = self.config['MAX_CONTENT_LENGTH'] + @deprecated def get_initialization_context(self): return self.injector.get(InitializationContext) From 72e5db7069b108792fd7a5f1392f21d94f1c24e1 Mon Sep 17 00:00:00 2001 From: Julius Flimmel Date: Mon, 14 Mar 2022 16:51:51 +0100 Subject: [PATCH 2/2] fix: renamed http request max content length config to be more specific --- src/zsl/application/service_application.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/zsl/application/service_application.py b/src/zsl/application/service_application.py index 310cc93..d7e1515 100644 --- a/src/zsl/application/service_application.py +++ b/src/zsl/application/service_application.py @@ -183,8 +183,8 @@ def _configure_injector(self, modules): self._dependencies_initialized = True def _configure_flask_app(self): - if 'MAX_CONTENT_LENGTH' in self.config: - app.config['MAX_CONTENT_LENGTH'] = self.config['MAX_CONTENT_LENGTH'] + if 'HTTP_REQUEST_MAX_CONTENT_LENGTH' in self.config: + app.config['HTTP_REQUEST_MAX_CONTENT_LENGTH'] = self.config['HTTP_REQUEST_MAX_CONTENT_LENGTH'] @deprecated def get_initialization_context(self):