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

Netty4.1: remove our handler when original handler is removed #3026

Merged
merged 9 commits into from
May 19, 2021
Merged

Netty4.1: remove our handler when original handler is removed #3026

merged 9 commits into from
May 19, 2021

Commits on May 18, 2021

  1. Netty4.1: remove our handler when orignal handler is removed

    @laurit
    laurit committed May 18, 2021
    Configuration menu
    Copy the full SHA
    691a1b4 View commit details
    Browse the repository at this point in the history

  2. imporve comment

    @laurit
    laurit committed May 18, 2021
    Configuration menu
    Copy the full SHA
    0df577c View commit details
    Browse the repository at this point in the history

  3. review fix

    @laurit
    laurit committed May 18, 2021
    Configuration menu
    Copy the full SHA
    5d55729 View commit details
    Browse the repository at this point in the history

Commits on May 19, 2021

  1. add test

    @laurit
    laurit committed May 19, 2021
    Configuration menu
    Copy the full SHA
    aff6429 View commit details
    Browse the repository at this point in the history

  2. Update instrumentation/netty/netty-4.1/javaagent/src/test/groovy/Chan… 

    …nelPipelineTest.groovy

Co-authored-by: Mateusz Rzeszutek <mrzeszutek@splunk.com>
    @laurit
    laurit and Mateusz Rzeszutek authored May 19, 2021
    Configuration menu
    Copy the full SHA
    7278c0f View commit details
    Browse the repository at this point in the history

  3. add missing import

    @laurit
    laurit committed May 19, 2021
    Configuration menu
    Copy the full SHA
    fe47fc0 View commit details
    Browse the repository at this point in the history

  4. disable epoll to see whether it makes any difference

    @laurit
    laurit committed May 19, 2021
    Configuration menu
    Copy the full SHA
    2317d03 View commit details
    Browse the repository at this point in the history

  5. fix netty with epoll/kqueue native library

    @laurit
    laurit committed May 19, 2021
    Configuration menu
    Copy the full SHA
    95c0ea2 View commit details
    Browse the repository at this point in the history

  6. should return class instread of instance

    @laurit
    laurit committed May 19, 2021
    Configuration menu
    Copy the full SHA
    6c3ec39 View commit details
    Browse the repository at this point in the history