Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

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

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

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

wants to merge 16 commits into from

Commits on Apr 20, 2022

  1. feat: implement pool_only_after_response option to restrict keep alive

    @GuyLewin
    GuyLewin committed Apr 20, 2022
    Configuration menu
    Copy the full SHA
    2a8439a View commit details
    Browse the repository at this point in the history

  2. doc: document pool_only_after_response

    @GuyLewin
    GuyLewin committed Apr 20, 2022
    Configuration menu
    Copy the full SHA
    679b0ec View commit details
    Browse the repository at this point in the history

  3. test: add tests to check pool_only_after_response

    @GuyLewin
    GuyLewin committed Apr 20, 2022
    Configuration menu
    Copy the full SHA
    a428ea8 View commit details
    Browse the repository at this point in the history

  4. fix: failing test

    @GuyLewin
    GuyLewin committed Apr 20, 2022
    Configuration menu
    Copy the full SHA
    f5075a2 View commit details
    Browse the repository at this point in the history

  5. fix: use keepalive_supported instead of keepalive

    @GuyLewin
    GuyLewin committed Apr 20, 2022
    Configuration menu
    Copy the full SHA
    58b02e2 View commit details
    Browse the repository at this point in the history

  6. fix: reset keepalive_ready once sending a new request

    @GuyLewin
    GuyLewin committed Apr 20, 2022
    Configuration menu
    Copy the full SHA
    b6e776c View commit details
    Browse the repository at this point in the history

  7. fix: start with keepalive_ready being true since socket is empty befo… 

    …re requests are made
    @GuyLewin
    GuyLewin committed Apr 20, 2022
    Configuration menu
    Copy the full SHA
    ab80363 View commit details
    Browse the repository at this point in the history

  8. test: add tests to check that client pooling works before any request… 

    …s, and after 2 requests
    @GuyLewin
    GuyLewin committed Apr 20, 2022
    Configuration menu
    Copy the full SHA
    af8a922 View commit details
    Browse the repository at this point in the history

  9. fix: typo in test

    @GuyLewin
    GuyLewin committed Apr 20, 2022
    Configuration menu
    Copy the full SHA
    01dfb01 View commit details
    Browse the repository at this point in the history

  10. fix: test failing because connect missing

    @GuyLewin
    GuyLewin committed Apr 20, 2022
    Configuration menu
    Copy the full SHA
    5872991 View commit details
    Browse the repository at this point in the history

  11. fix: keepalive_ready mark only after connection

    @GuyLewin
    GuyLewin committed Apr 20, 2022
    Configuration menu
    Copy the full SHA
    6a4075c View commit details
    Browse the repository at this point in the history

  12. fix: mark keepalive_ready if no body and no trailer

    @GuyLewin
    GuyLewin committed Apr 20, 2022
    Configuration menu
    Copy the full SHA
    2d4feca View commit details
    Browse the repository at this point in the history

  13. feat: convert coroutines to closures, remove unnecessary duplicate ca… 

    …ll to closures if there are trailers
    @GuyLewin
    GuyLewin committed Apr 20, 2022
    Configuration menu
    Copy the full SHA
    7feec2e View commit details
    Browse the repository at this point in the history

  14. feat: remove optional flag pool_only_after_response, its always on

    @GuyLewin
    GuyLewin committed Apr 20, 2022
    Configuration menu
    Copy the full SHA
    91935c4 View commit details
    Browse the repository at this point in the history

  15. feat: rewrite keepalive_ready feature based on reader_state table

    @GuyLewin
    GuyLewin committed Apr 20, 2022
    Configuration menu
    Copy the full SHA
    5d3a82c View commit details
    Browse the repository at this point in the history

  16. fix: luacheck errors

    @GuyLewin
    GuyLewin committed Apr 20, 2022
    Configuration menu
    Copy the full SHA
    948d652 View commit details
    Browse the repository at this point in the history