Attachment 149641 Details for Bug 32716 – Bug 32716: larger proxy_buffer_size in NGINX example config

[patch] Bug 32716: larger proxy_buffer_size in NGINX example config

Bug-32716-larger-proxybuffersize-in-NGINX-example-.patch (text/plain), 2.00 KB, created by Marcel de Rooy on 2023-04-14 06:37:05 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2023-04-14 06:37:05 UTC

Size: 2.00 KB

patch

obsolete

>From b2abb4faeb2b41ba74237397964c232bf3aff1e8 Mon Sep 17 00:00:00 2001
>From: Galen Charlton <gmc@equinoxOLI.org>
>Date: Tue, 24 Jan 2023 14:10:32 -0500
>Subject: [PATCH] Bug 32716: larger proxy_buffer_size in NGINX example config
>Content-Type: text/plain; charset=utf-8
>
>This patch updates the example NGINX config to increase the
>proxy_buffer_size to 16k. The default value of 4k (on some platforms)
>has empirically been shown to be a bit too small for the Link
>headers emitted by the REST API when pagination is requested.
>
>To test
>-------
>[1] Set up a Koha system with NGINX as a reverse proxy in
>    front of it (either in front of Apache or in front of
>    of Starman).
>[2] Perform a patron search that returns at least two pages
>    of results and navigate to the second page.
>[3] Note that the navigation can fail with a 502 HTTP error
>    and an "upstream sent too big header while reading response
>    header from upstream" error in the NGINX log.
>
>    The problem is most likely when the pagesize of the server
>    running NGINX is 4096 bytes.
>[4] Update the NGINX configuration per this patch and restart
>    NGINX.
>[5] This time, repeating step 2 should work.
>
>Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>
>Signed-off-by: Brendan Gallagher <brendan@bywatersolutions.com>
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> etc/nginx.conf | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
>diff --git a/etc/nginx.conf b/etc/nginx.conf
>index 4701a9a645..d6e8aba9be 100644
>--- a/etc/nginx.conf
>+++ b/etc/nginx.conf
>@@ -17,6 +17,10 @@ server {
>     location / {
>         include proxy_params;
>         proxy_pass http://intranet;
>+
>+    # provide room for the Link headers emitted
>+    # by REST API methods when doing pagination
>+    proxy_buffer_size 16k;
>     }
> }
> 
>@@ -29,5 +33,9 @@ server {
>     location / {
>         include proxy_params;
>         proxy_pass http://opac;
>+
>+    # provide room for the Link headers emitted
>+    # by REST API methods when doing pagination
>+    proxy_buffer_size 16k;
>     }
> }
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 32716: 145633 | 145634 | 145635 | 148911 | 149641