Skip to content

Navigation Menu

Appearance settings

View all features
- BY COMPANY SIZE
  Enterprises
  Small and medium teams
  Startups
  Nonprofits
- BY USE CASE
  App Modernization
  DevSecOps
  DevOps
  CI/CD
  View all use cases
- BY INDUSTRY
  Healthcare
  Financial services
  Manufacturing
  Government
  View all industries
View all solutions
- EXPLORE BY TOPIC
  AI
  Software Development
  DevOps
  Security
  View all topics
- EXPLORE BY TYPE
  Customer stories
  Events & webinars
  Ebooks & reports
  Business insights
  GitHub Skills
- SUPPORT & SERVICES
  Documentation
  Customer support
  Community forum
  Trust center
  Partners
View all resources
- COMMUNITY
  GitHub SponsorsFund open source developers
- PROGRAMS
  Security Lab
  Maintainer Community
  Accelerator
  GitHub Stars
  Archive Program
- REPOSITORIES
  Topics
  Trending
  Collections
- ENTERPRISE SOLUTIONS
  Enterprise platformAI-powered developer platform
- AVAILABLE ADD-ONS
  GitHub Advanced SecurityEnterprise-grade security features
  Copilot for BusinessEnterprise-grade AI features
  Premium SupportEnterprise-grade 24/7 support
Pricing

Search code, repositories, users, issues, pull requests...

Search

Clear

Search syntax tips

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Name

Query

To see all available qualifiers, see our documentation.

Appearance settings

You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session.

Dismiss alert

ledgetech / lua-resty-http Public

Notifications You must be signed in to change notification settings
Fork 626
Star 2.1k

Code
Issues 33
Pull requests 18
Discussions
Actions
Projects
Wiki
Security and quality
Insights

Additional navigation options

Code
Issues
Pull requests
Discussions
Actions
Projects
Wiki
Security and quality
Insights

[Feature] Pool only when HTTP response fully read#255

Open

GuyLewin wants to merge 16 commits into

ledgetech:masterledgetech/lua-resty-http:masterfrom

GuyLewin:feature/pool-only-when-http-body-readGuyLewin/lua-resty-http:feature/pool-only-when-http-body-readCopy head branch name to clipboard

Conversation Commits16 (16)Checks Files changed

Open

[Feature] Pool only when HTTP response fully read#255
GuyLewin wants to merge 16 commits into
ledgetech:masterfrom
GuyLewin:feature/pool-only-when-http-body-read

Commits

Commits on Apr 20, 2022

feat: implement pool_only_after_response option to restrict keep alive
GuyLewin
committed
doc: document pool_only_after_response
GuyLewin
committed
test: add tests to check pool_only_after_response
GuyLewin
committed
fix: failing test
GuyLewin
committed
fix: use keepalive_supported instead of keepalive
GuyLewin
committed
fix: reset keepalive_ready once sending a new request
GuyLewin
committed
fix: start with keepalive_ready being true since socket is empty before requests are made
GuyLewin
committed
test: add tests to check that client pooling works before any requests, and after 2 requests
GuyLewin
committed
fix: typo in test
GuyLewin
committed
fix: test failing because connect missing
GuyLewin
committed
fix: keepalive_ready mark only after connection
GuyLewin
committed
fix: mark keepalive_ready if no body and no trailer
GuyLewin
committed
feat: convert coroutines to closures, remove unnecessary duplicate call to closures if there are trailers
GuyLewin
committed
feat: remove optional flag pool_only_after_response, its always on
GuyLewin
committed
feat: rewrite keepalive_ready feature based on reader_state table
GuyLewin
committed
fix: luacheck errors
GuyLewin
committed

Footer

© 2026 GitHub, Inc.

Footer navigation

Terms
Privacy
Security
Status
Community
Docs
Contact

You can’t perform that action at this time.