Skip to content

fix[nginx]: Use a larger buffer size for /v1a/status#1594

Merged
luislhl merged 1 commit intomasterfrom
fix/nginx/status-buffer-size
Feb 10, 2026
Merged

fix[nginx]: Use a larger buffer size for /v1a/status#1594
luislhl merged 1 commit intomasterfrom
fix/nginx/status-buffer-size

Conversation

@luislhl
Copy link
Contributor

@luislhl luislhl commented Feb 10, 2026

Motivation

We are receiving logs like this in Nginx:

2026/02/10 16:07:20 [warn] 20#20: *10712262 an upstream response is buffered to a temporary file /var/cache/nginx/proxy_temp/2/02/0001378022 while reading upstream, client: <redacted>, server: localhost, request: "GET /v1a/status HTTP/1.1", upstream: "http://127.0.0.1:8080/v1a/status", host: "<redacted>"

Acceptance Criteria

  • We should be able to set the buffers config in OpenAPI definition
  • The automated generator of nginx.conf should read it and add in the config in the right endpoint
  • We should increase the buffers for /v1a/status, which has a response size around 122kb

Checklist

  • If you are requesting a merge into master, confirm this code is production-ready and can be included in future releases as soon as it gets merged

@luislhl luislhl self-assigned this Feb 10, 2026
@luislhl luislhl moved this from Todo to In Progress (Done) in Hathor Network Feb 10, 2026
@luislhl luislhl requested a review from glevco February 10, 2026 16:20
@github-actions
Copy link

🐰 Bencher Report

Branchfix/nginx/status-buffer-size
Testbedubuntu-22.04
Click to view all benchmark results
BenchmarkLatencyBenchmark Result
minutes (m)
(Result Δ%)
Lower Boundary
minutes (m)
(Limit %)
Upper Boundary
minutes (m)
(Limit %)
sync-v2 (up to 20000 blocks)📈 view plot
🚷 view threshold
1.70 m
(-0.61%)Baseline: 1.71 m
1.54 m
(90.55%)
2.06 m
(82.82%)
🐰 View full continuous benchmarking report in Bencher

@github-project-automation github-project-automation bot moved this from In Progress (Done) to In Review (WIP) in Hathor Network Feb 10, 2026
@glevco glevco moved this from In Review (WIP) to In Review (Done) in Hathor Network Feb 10, 2026
@codecov
Copy link

codecov bot commented Feb 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.71%. Comparing base (246a56a) to head (22b9de9).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1594      +/-   ##
==========================================
- Coverage   85.76%   85.71%   -0.05%     
==========================================
  Files         441      441              
  Lines       33829    33829              
  Branches     5285     5285              
==========================================
- Hits        29014    28998      -16     
- Misses       3800     3814      +14     
- Partials     1015     1017       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@luislhl luislhl merged commit 683d475 into master Feb 10, 2026
27 of 30 checks passed
@luislhl luislhl deleted the fix/nginx/status-buffer-size branch February 10, 2026 23:02
@github-project-automation github-project-automation bot moved this from In Review (Done) to Waiting to be deployed in Hathor Network Feb 10, 2026
r4mmer added a commit that referenced this pull request Feb 24, 2026
…print-move-1

* origin/master:
  feat: pydantic settings (#1600)
  fix[thin_wallet]: handle address history invalid tx version (#1590)
  refactor(nano): Make NCBlockExecutor a pure executor with no side effects
  fix[nginx]: Make sure we trust the GCP IPs to get the real client IP (#1595)
  refactor: Upgrade to Pydantic v2
  chore(github): Split GitHub main action into lint, test-cli, test-lib, test-other
  fix[nginx]: Use a larger buffer size for /v1a/status (#1594)
  chore: adjust testnet config for v0.69.0 release
  chore[storage]: Limit the total size of RocksDB WAL files (#1518)
  chore: adjust testnet config for v0.69.0 release
  chore: configure feature activations for v0.69.0 release
  refactor: wallet on_new_tx (#1561)
  refactor(nano): Remove dead reorg cleanup code from block executor
@jansegre jansegre mentioned this pull request Mar 5, 2026
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Waiting to be deployed

Development

Successfully merging this pull request may close these issues.

3 participants