danielcweeks commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1945150977
##########
aws/src/main/java/org/apache/iceberg/aws/s3/VendedCredentialsProvider.java:
##########
@@ -75,10 +76,13 @@ private RESTClient httpClient() {
if (null == client) {
synchronized (this) {
if (null == client) {
+ DefaultAuthSession authSession =
+ DefaultAuthSession.of(
+
HTTPHeaders.of(OAuth2Util.authHeaders(properties.get(OAuth2Properties.TOKEN))));
Review Comment:
@adutra It's beyond the scope of this PR since this is an existing problem,
but somehow we need the AuthManager/AuthSession to be able to refresh OAuth
tokens here as well. The token here will expire at which point the credential
provider will not be able to get new vended credentials.
--
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: issues-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
