[PR #63] [MERGED] perf(proxy): Add HTTP client reuse and buffer pooling #348

Closed
opened 2026-05-12 20:00:25 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/fosrl/gerbil/pull/63
Author: @LaurenceJJones
Created: 3/13/2026
Status: Merged
Merged: 4/3/2026
Merged by: @marcschaeferger

Base: devHead: proxy-perf-improvements


📝 Commits (3)

  • c7d9c72 Add HTTP client reuse and buffer pooling for performance
  • c3ed355 Merge branch 'dev' into proxy-perf-improvements
  • 58415de refactor: remove redundant HTTP client instantiation in getRoute method

📊 Changes

1 file changed (+40 additions, -8 deletions)

View changed files

📝 proxy/proxy.go (+40 -8)

📄 Description

Community Contribution License Agreement

By creating this pull request, I grant the project maintainers an unlimited,
perpetual license to use, modify, and redistribute these contributions under any terms they
choose, including both the AGPLv3 and the Fossorial Commercial license terms. I
represent that I have the right to grant this license for all contributed content.

Description

  • Add reusable HTTP client with connection pooling for requests
  • Add sync.Pool for 32KB buffers used in connection piping
  • Clear buffers before returning to pool to prevent data leakage
  • Reduces GC pressure and improves throughput under load

How to test?

Internal proxy usage


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/fosrl/gerbil/pull/63 **Author:** [@LaurenceJJones](https://github.com/LaurenceJJones) **Created:** 3/13/2026 **Status:** ✅ Merged **Merged:** 4/3/2026 **Merged by:** [@marcschaeferger](https://github.com/marcschaeferger) **Base:** `dev` ← **Head:** `proxy-perf-improvements` --- ### 📝 Commits (3) - [`c7d9c72`](https://github.com/fosrl/gerbil/commit/c7d9c72f2937c3d8594eaf58694d7dd8bc55b6d9) Add HTTP client reuse and buffer pooling for performance - [`c3ed355`](https://github.com/fosrl/gerbil/commit/c3ed3551275946f780118734330157898003634e) Merge branch 'dev' into proxy-perf-improvements - [`58415de`](https://github.com/fosrl/gerbil/commit/58415dee7e6979b3bff4a1cc422f81e52c54b7cb) refactor: remove redundant HTTP client instantiation in getRoute method ### 📊 Changes **1 file changed** (+40 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `proxy/proxy.go` (+40 -8) </details> ### 📄 Description ## Community Contribution License Agreement By creating this pull request, I grant the project maintainers an unlimited, perpetual license to use, modify, and redistribute these contributions under any terms they choose, including both the AGPLv3 and the Fossorial Commercial license terms. I represent that I have the right to grant this license for all contributed content. ## Description - Add reusable HTTP client with connection pooling for requests - Add sync.Pool for 32KB buffers used in connection piping - Clear buffers before returning to pool to prevent data leakage - Reduces GC pressure and improves throughput under load ## How to test? Internal proxy usage --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-05-12 20:00:25 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gerbil#348