joseluisll opened a new pull request, #8945:
URL: https://github.com/apache/hadoop/pull/8945

   ### Description of PR
   
   Several Hadoop clients take the reason a server refused a request from the 
HTTP reason phrase; Jetty 12 sends only the canonical phrase. The same text is 
in the response body on Jetty 9.4 and Jetty 12.
   
   ResponseDetail in hadoop-auth reads a capped amount of the error body: the 
MESSAGE row of a Jetty error page, or any other page's text, falling back to 
the phrase when there is no body. It leaves the JSON error envelope alone.
   
   AuthenticatedURL, HttpExceptionUtils#validateResponse and WebHdfsFileSystem 
report failures through it. KMSClientProvider uses it to decide whether to 
reset its token and re-authenticate. WebHdfsFileSystem#jsonParse checks the 
content type before opening the stream, so a body it will not read stays 
readable.
   
   No change against Jetty 9.4 servers.
   
   ### How was this patch tested?
   
   The changed modules' tests, run by the Yetus precommit and the fork's CI on 
this branch.
   
   ### For code changes:
   
   - [x] Does the title of this PR start with the corresponding JIRA issue id?
   - [ ] Object storage: not applicable, no object store integration touched.
   - [x] If adding new dependencies: Guice 6.0.0 and javax.inject already under 
ASF-compatible licenses; LICENSE-binary updated.
   - [x] LICENSE-binary updated where the dependency tree changed.
   
   ### AI Tooling
   
   If an AI tool was used:
   
   - [x] The PR includes the phrase "Contains content generated by <tool>" 
where <tool> is the name of the AI tool used.
   - [x] My use of AI contributions follows the ASF legal policy 
https://www.apache.org/legal/generative-tooling.html
   
   Contains content generated by Claude Code.
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)
   


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

  • [PR] HADOOP-20003. Clients sha... joseluisll (via GitHub) via common-issues

Reply via email to