jojochuang commented on pull request #2453:
URL: https://github.com/apache/hadoop/pull/2453#issuecomment-725275520


   Ok... so this is getting more involved. It turns out that Jetty 9.4.21 has a 
breaking change (https://github.com/eclipse/jetty.project/pull/3912/files)
   Previously, our o.h.a.s.authentication.server.AuthenticationFilter sends a 
custom message associated with 401 error. After 9.4.21, Response.sendError() no 
longer allows a custom error message. You have to call 
Response.setStatusWithReason() to set that. But that requires an additional 
dependency on jetty-server.
   
   I don't feel that strong to include jetty-server just so that we can set a 
custom error message. We could update the test code instead too.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

Reply via email to