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

Leak detection properties are added for the test execution #2461

Merged
merged 15 commits into from Oct 3, 2022
Merged

Leak detection properties are added for the test execution #2461

merged 15 commits into from Oct 3, 2022

Commits on Oct 3, 2022

  1. [test] Leak detection properties are added for the test execution

    @violetagg
    violetagg committed Oct 3, 2022
    Configuration menu
    Copy the full SHA
    9a1be78 View commit details
    Browse the repository at this point in the history

  2. Fix memory leaks when processing EmptyLastHttpContent 

    Fix memory leak in test
    @violetagg
    violetagg committed Oct 3, 2022
    Configuration menu
    Copy the full SHA
    456bdde View commit details
    Browse the repository at this point in the history

  3. Avoid buffer leak by closing the HttpRequest passed to the WebSocketS… 

    …erverHandshaker.handshake method
    @pderop @violetagg
    pderop authored and violetagg committed Oct 3, 2022
    Configuration menu
    Copy the full SHA
    09483b9 View commit details
    Browse the repository at this point in the history

  4. Directly close the http request instead of using Resource.dispose

    @pderop @violetagg
    pderop authored and violetagg committed Oct 3, 2022
    Configuration menu
    Copy the full SHA
    eeb1dd7 View commit details
    Browse the repository at this point in the history

  5. Close buffers passed to the MessageToMessageEncoder.encodeAndClose me… 

    …thod in NettyOutboundTest
    @pderop @violetagg
    pderop authored and violetagg committed Oct 3, 2022
    Configuration menu
    Copy the full SHA
    946d457 View commit details
    Browse the repository at this point in the history

  6. Override MessageToMessageEncoder.encode method, which will close the … 

    …message for us
    @pderop @violetagg
    pderop authored and violetagg committed Oct 3, 2022
    Configuration menu
    Copy the full SHA
    d7acb9e View commit details
    Browse the repository at this point in the history

  7. Avoid buf leak in TcpServerTests.retryStrategiesWhenServerFails by cl… 

    …osing BOUNDARY in NettyOutbound.sendGroups
    @pderop @violetagg
    pderop authored and violetagg committed Oct 3, 2022
    Configuration menu
    Copy the full SHA
    d0826d7 View commit details
    Browse the repository at this point in the history

  8. Revert "Avoid buf leak in TcpServerTests.retryStrategiesWhenServerFai… 

    …ls by closing BOUNDARY in NettyOutbound.sendGroups"

This reverts commit c47ba03.
    @violetagg
    violetagg committed Oct 3, 2022
    Configuration menu
    Copy the full SHA
    a87336a View commit details
    Browse the repository at this point in the history

  9. Declare ReactorNetty#BOUNDARY as on-heap non-releasable buffer

    @violetagg
    violetagg committed Oct 3, 2022
    Configuration menu
    Copy the full SHA
    11e16f3 View commit details
    Browse the repository at this point in the history

  10. Do not close the empty full message when invoking SimpleCompressionHa… 

    …ndler out of the pipeline
    @violetagg
    violetagg committed Oct 3, 2022
    Configuration menu
    Copy the full SHA
    d539649 View commit details
    Browse the repository at this point in the history

  11. Ensure the messages are closed after the test execution

    @violetagg
    violetagg committed Oct 3, 2022
    Configuration menu
    Copy the full SHA
    f0c6b90 View commit details
    Browse the repository at this point in the history

  12. Fixed leak reproduced by HttpServerTests.testIssue1978H2WithDelay. 

    The Http2StreamBridgeServerHandler.channelRead method was still using old ReferenceCountUtil.release() instead of Resource.dispose() method.
    @pderop @violetagg
    pderop authored and violetagg committed Oct 3, 2022
    Configuration menu
    Copy the full SHA
    bf457f0 View commit details
    Browse the repository at this point in the history

  13. Fix memory leak in test

    @violetagg
    violetagg committed Oct 3, 2022
    Configuration menu
    Copy the full SHA
    600a3b1 View commit details
    Browse the repository at this point in the history

  14. Ensure Http2FrameCodec is created only when there is a need for proto… 

    …col upgrade
    @violetagg
    violetagg committed Oct 3, 2022
    Configuration menu
    Copy the full SHA
    638d10d View commit details
    Browse the repository at this point in the history

  15. Ensure Http2FrameCodec.Encoder is closed when upgrade is rejected 

    Ensure Http2FrameCodec.Encoder is closed when Exception happened
before decoding the server response
    @violetagg
    violetagg committed Oct 3, 2022
    Configuration menu
    Copy the full SHA
    15c05bd View commit details
    Browse the repository at this point in the history