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

Jetty 9.4.x 4331 async close complete #4378

Closed
wants to merge 12 commits into from
Closed

Jetty 9.4.x 4331 async close complete #4378

wants to merge 12 commits into from

Commits on Nov 28, 2019

  1. Issue #4376 Async Content Complete 

    Added test harness to reproduce unready completing write.
Fixed test by not closing output prior to becoming READY

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Nov 28, 2019
    Copy the full SHA
    b30755f View commit details
    Browse the repository at this point in the history

Commits on Nov 29, 2019

  1. Issue #4331 Async Write Complete 

    Test harness to reproduce unready when closing/completing.

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Nov 29, 2019
    Copy the full SHA
    529ea5e View commit details
    Browse the repository at this point in the history

  2. Issue #4331 Async Write Complete 

    test both PENDING and UNREADY

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Nov 29, 2019
    Copy the full SHA
    41b3d70 View commit details
    Browse the repository at this point in the history

  3. Issue #4331 Async Write Complete 

    test cleanups

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Nov 29, 2019
    Copy the full SHA
    a872522 View commit details
    Browse the repository at this point in the history

  4. Issue #4376 Async Content Complete 

    ERROR state still needs to be closed!

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Nov 29, 2019
    Copy the full SHA
    21209d7 View commit details
    Browse the repository at this point in the history

  5. Merge branch 'jetty-9.4.x-4376-asynccontentcomplete' into jetty-9.4.x… 

    …-4331-asyncCloseComplete
    @gregw
    gregw committed Nov 29, 2019
    Copy the full SHA
    f43a484 View commit details
    Browse the repository at this point in the history

  6. Issue #4376 Async Content Complete 

    close after last blocking write

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Nov 29, 2019
    Copy the full SHA
    d254158 View commit details
    Browse the repository at this point in the history

  7. Issue #4376 Async Content Complete 

    If completion has to do a flush, then we need a call to closed to
avoid leaking buffers.

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Nov 29, 2019
    Copy the full SHA
    abb7e3b View commit details
    Browse the repository at this point in the history

  8. Merge branch 'jetty-9.4.x-4376-asynccontentcomplete' into jetty-9.4.x… 

    …-4331-asyncCloseComplete
    @gregw
    gregw committed Nov 29, 2019
    Copy the full SHA
    2fe92fc View commit details
    Browse the repository at this point in the history

Commits on Nov 30, 2019

  1. Issue #4331 Async Close Complete 

    WIP - moved the outstate from HttpOutput to HttpChannelState
Error handling needs a big review

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Nov 30, 2019
    Copy the full SHA
    2b5ffeb View commit details
    Browse the repository at this point in the history

Commits on Dec 2, 2019

  1. Issue #4376 Async Content Complete 

    Reformat

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Dec 2, 2019
    Copy the full SHA
    36cc246 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'jetty-9.4.x-4376-asynccontentcomplete' into jetty-9.4.x… 

    …-4331-asyncCloseComplete

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Dec 2, 2019
    Copy the full SHA
    50c9740 View commit details
    Browse the repository at this point in the history