adutra commented on code in PR #14241:
URL: https://github.com/apache/iceberg/pull/14241#discussion_r2445654626
##########
core/src/main/java/org/apache/iceberg/rest/HTTPClient.java:
##########
@@ -360,6 +391,37 @@ protected <T extends RESTResponse> T execute(
}
}
+ @Nullable
+ private HTTPRequest processChallenge(
+ HTTPRequest request,
+ CloseableHttpResponse response,
+ HttpClientContext context,
+ int retryAttempt) {
+
+ Map<String, AuthChallenge> challengeMap =
+ authenticationHandler.extractChallengeMap(ChallengeType.TARGET,
response, context);
Review Comment:
That's for when the _proxy_ server returns a challenge (response type 407).
This PR does not add support for proxy authentication challenges. But it
could be added in the future (although, I would be in favor of configuring it
at the HTTP layer along with other proxy settings – not here).
--
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]
