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.

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

Improved handling of 100 Continue #12113

Merged

Commits on Jul 30, 2024

  1. Improvements to HttpSender.

    * Changed ContentSender demand from iterate()+IDLE to succeeded()+SCHEDULED.
      This ensures that there is no re-iteration in case a 100 Continue response arrives.
      This, in turn, avoids that the demand is performed multiple times, causing ISE to be thrown.
    * Changed the 100 Continue action of the proxy Servlet/Handler, that provides the request content, to be executed by the HttpSender, rather than by the HttpReceiver.
    
    Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
    sbordet committed Jul 30, 2024
    Configuration menu
    Copy the full SHA
    944b9e8 View commit details
    Browse the repository at this point in the history

Commits on Jul 31, 2024

  1. Improved handling of 100 Continue.

    * Now `HttpClient` removed the `Expect` header if there is no request content.
    * Changed AbstractProxyServlet and ProxyHandler check for request content: now the Content-Type header is not taken into consideration.
    * Now the server avoids sending the 100 Continue response if there is no request content.
    
    Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
    sbordet committed Jul 31, 2024
    Configuration menu
    Copy the full SHA
    7964f40 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    c7674f6 View commit details
    Browse the repository at this point in the history

Commits on Aug 1, 2024

  1. Fixes after test failure.

    Now the request body is not defaulted if missing, but just kept null.
    
    Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
    sbordet committed Aug 1, 2024
    Configuration menu
    Copy the full SHA
    4e9755d View commit details
    Browse the repository at this point in the history

Commits on Aug 2, 2024

  1. Merged branch 'jetty-12.0.x' into 'fix/jetty-12.0.x/httpsender-proxy-…

    …continue-fixes'.
    
    Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
    sbordet committed Aug 2, 2024
    Configuration menu
    Copy the full SHA
    6b307e1 View commit details
    Browse the repository at this point in the history

Commits on Aug 13, 2024

  1. Updates from review.

    Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
    sbordet committed Aug 13, 2024
    Configuration menu
    Copy the full SHA
    e11ab97 View commit details
    Browse the repository at this point in the history
  2. Updates after review.

    Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
    sbordet committed Aug 13, 2024
    Configuration menu
    Copy the full SHA
    cd50711 View commit details
    Browse the repository at this point in the history