The "upload_state_store" configuration directive is not enforced. about nginx-upload-module HOT 4 CLOSED

Indeed, it's a bug. Thanks for checking!

from nginx-upload-module.

Could you please try a patch?

diff --git a/ngx_http_upload_module.c b/ngx_http_upload_module.c
index 3e1fcc5..64c8ca6 100644
--- a/ngx_http_upload_module.c
+++ b/ngx_http_upload_module.c
@@ -1050,7 +1050,7 @@ static ngx_int_t ngx_http_upload_start_handler(ngx_http_upload_ctx_t u) { / {{
ngx_http_upload_loc_conf_t *ulcf = ngx_http_get_module_loc_conf(r, ngx_http_upload_module);

 ngx_file_t  *file = &u->output_file;

• ngx_path_t *path = ulcf->store_path;
• ngx_path_t *path = (u->session_id.len == 0 ? ulcf->store_path : ulcf->state_store_path);
  uint32_t n;
  ngx_uint_t i;
  ngx_int_t rc;

from nginx-upload-module.

Alternatively:
vkholodkov@8cfa513

from nginx-upload-module.

Well, it kind-of works, except both the uploaded file and the .state file are now stored in the state_store_path folder,
not only the .state file. This seems in contradiction with the documentation as far as I understand it (uploaded files
in store_path, and (maybe optionally?) state files in state_store_path (or in store_path if the configuration directive
is not specified?)).

What do you think?

from nginx-upload-module.