Bump buffer size for HTTP response headers (#378)

To accommodate 4 memory pages, e.g. 16k on most platforms.
weserv · Jun 30, 2023 · ca63088 · ca63088
1 parent adc6b9a
commit ca63088
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/nginx/module.cpp b/src/nginx/module.cpp
@@ -443,8 +443,11 @@ void *ngx_weserv_create_loc_conf(ngx_conf_t *cf) {
 
     // The hardcoded values
     lc->upstream_conf.buffering = 1;
-    lc->upstream_conf.buffer_size = ngx_pagesize;
-    lc->upstream_conf.busy_buffers_size = 2 * ngx_pagesize;
+    lc->upstream_conf.buffer_size =
+        4 * ngx_pagesize;  // 4 memory pages should be enough for most response
+                           // headers
+    lc->upstream_conf.busy_buffers_size =
+        6 * ngx_pagesize;  // Essentially, buffer_size + 2 memory pages
     lc->upstream_conf.bufs.num = 256;
     lc->upstream_conf.bufs.size = ngx_pagesize;
     lc->upstream_conf.max_temp_file_size = 0;