Summary: | update NGINX config examples to increase proxy_buffer_size | ||
---|---|---|---|
Product: | Koha | Reporter: | Galen Charlton <gmcharlt> |
Component: | Architecture, internals, and plumbing | Assignee: | Galen Charlton <gmc> |
Status: | CLOSED FIXED | QA Contact: | Marcel de Rooy <m.de.rooy> |
Severity: | minor | ||
Priority: | P5 - low | CC: | arthur.suzuki, brendan, lucas, m.de.rooy |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | Unsponsored | |
Patch complexity: | Trivial patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: |
Set proxy_buffer_size in the example NGINX configuration to reduce chances that REST API responses that use pagination get dropped by NGINX
|
|
Version(s) released in: |
23.05.00,22.11.05,22.05.13,21.11.21
|
Circulation function: | |
Attachments: |
Bug 32716: larger proxy_buffer_size in NGINX example config
Bug 32716: larger proxy_buffer_size in NGINX example config Bug 32716: larger proxy_buffer_size in NGINX example config Bug 32716: larger proxy_buffer_size in NGINX example config Bug 32716: larger proxy_buffer_size in NGINX example config |
Description
Galen Charlton
2023-01-24 19:04:50 UTC
Created attachment 145635 [details] [review] Bug 32716: larger proxy_buffer_size in NGINX example config 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> Created attachment 148911 [details] [review] Bug 32716: larger proxy_buffer_size in NGINX example config 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> Created attachment 149641 [details] [review] Bug 32716: larger proxy_buffer_size in NGINX example config 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> Pushed to master for 23.05. Nice work everyone, thanks! Nice work everyone! Pushed to stable for 22.11.x Backported to 22.05.x for upcoming 22.05.13. applied to 21.11 for 21.11.21 |