sarutak opened a new pull request, #53116:
URL: https://github.com/apache/spark/pull/53116

   ### What changes were proposed in this pull request?
   This PR proposes to upgrade Jetty to 12.1.4 and some dependencies including 
Jersey.
   Because Jetty 12 is [significantly changed from 
11.x,](https://jetty.org/docs/jetty/12.1/programming-guide/migration/11-to-12.html)
 this PR mainly considers following things to upgrade.
   
   * Redirect handler for http to https is rewritten using the standard Servlet 
API because most of the code can be reusable with `Filter` while it's difficult 
to do it with Jetty specific `Core API`.
   * `ProxyRedirectHandler` is rewritten using new `Core API` because 
`HandlerWrapper` is no longer available. It's difficult to implement 
`ProxyRedirectHandler` with the standard Servlet API so `Core API` is used.
   * In several places, the expected status code is changed from 302 to 301 to 
adapt Jetty's change.
     * 
https://github.com/jetty/jetty.project/blob/4137e00793d583cfc4ce76bf607e482178a85597/jetty-core/jetty-server/src/main/java/org/eclipse/jetty/server/handler/ContextHandler.java#L1205
   * In several places, consider the case where the value of location header is 
relative form to adapt Jetty's change.
     * 
https://github.com/jetty/jetty.project/blob/4137e00793d583cfc4ce76bf607e482178a85597/jetty-core/jetty-server/src/main/java/org/eclipse/jetty/server/Response.java#L415
     * 
https://github.com/jetty/jetty.project/blob/4137e00793d583cfc4ce76bf607e482178a85597/jetty-core/jetty-server/src/main/java/org/eclipse/jetty/server/handler/ContextHandler.java#L1260
   
   ### Why are the changes needed?
   Jetty 11 is already EOF and starting Jan 1, 2026, no more release will be 
published.
   https://github.com/jetty/jetty.project/issues/13918
   
   ### Does this PR introduce _any_ user-facing change?
   No.
   
   ### How was this patch tested?
   Existing tests with some tweaks.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   No.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to