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 12 - Review client notifiers #9335

Merged
merged 7 commits into from
Feb 13, 2023
Merged

Jetty 12 - Review client notifiers #9335

merged 7 commits into from
Feb 13, 2023

Commits on Feb 9, 2023

  1. Introduced ResponseListeners to reduce per-request allocation. 

    Removed ResponseNotifier in favor of ResponseListeners.

Signed-off-by: Simone Bordet <[email protected]>
    @sbordet
    sbordet committed Feb 9, 2023
    Configuration menu
    Copy the full SHA
    ad2c847 View commit details
    Browse the repository at this point in the history

  2. Introduced RequestListeners to reduce the per-request allocation. 

    Removed RequestNotifier in favor of RequestListeners.

Signed-off-by: Simone Bordet <[email protected]>
    @sbordet
    sbordet committed Feb 9, 2023
    Configuration menu
    Copy the full SHA
    6d145b5 View commit details
    Browse the repository at this point in the history

  3. Removed unnecessary sharing of HttpRequest.requestListeners. 

    Fixed RequestListeners notifications.
Fixed ResponseListeners handling of CompleteListeners.

Signed-off-by: Simone Bordet <[email protected]>
    @sbordet
    sbordet committed Feb 9, 2023
    Configuration menu
    Copy the full SHA
    85e8c2c View commit details
    Browse the repository at this point in the history

  4. Do not notify empty request buffers. 

    Signed-off-by: Simone Bordet <[email protected]>
    @sbordet
    sbordet committed Feb 9, 2023
    Configuration menu
    Copy the full SHA
    17123fc View commit details
    Browse the repository at this point in the history

  5. Fixed demultiplexing logic, now relies on the listeners list rather t… 

    …han the contentSource list.

Signed-off-by: Simone Bordet <[email protected]>
    @sbordet
    sbordet committed Feb 9, 2023
    Configuration menu
    Copy the full SHA
    edc00f2 View commit details
    Browse the repository at this point in the history

  6. Fixed and clarified HTTP/2 upgrade. 

    Signed-off-by: Simone Bordet <[email protected]>
    @sbordet
    sbordet committed Feb 9, 2023
    Configuration menu
    Copy the full SHA
    e777c8c View commit details
    Browse the repository at this point in the history

Commits on Feb 13, 2023

  1. Updates after review. 

    Signed-off-by: Simone Bordet <[email protected]>
    @sbordet
    sbordet committed Feb 13, 2023
    Configuration menu
    Copy the full SHA
    2e43449 View commit details
    Browse the repository at this point in the history