adutra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1945236123
##
aws/src/main/java/org/apache/iceberg/aws/s3/VendedCredentialsProvider.java:
##
@@ -75,10 +76,13 @@ private RESTClient httpClient() {
if (null == client) {
danielcweeks merged PR #11992:
URL: https://github.com/apache/iceberg/pull/11992
--
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...@iceb
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)
adutra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1942704258
##
aws/src/main/java/org/apache/iceberg/aws/s3/signer/S3V4RestSignerClient.java:
##
@@ -192,6 +192,7 @@ private RESTClient httpClient() {
HTTPClient.bui
adutra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1942704258
##
aws/src/main/java/org/apache/iceberg/aws/s3/signer/S3V4RestSignerClient.java:
##
@@ -192,6 +192,7 @@ private RESTClient httpClient() {
HTTPClient.bui
adutra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1942701609
##
aws/src/main/java/org/apache/iceberg/aws/s3/signer/S3V4RestSignerClient.java:
##
@@ -192,6 +192,7 @@ private RESTClient httpClient() {
HTTPClient.bui
danielcweeks commented on PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#issuecomment-2635399088
I feel like there may have been a small understanding with how we handle the
initial authSession. We don't want to default it to empty otherwise if someone
adds a request like `cl
danielcweeks commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1942068160
##
core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java:
##
@@ -242,7 +242,10 @@ public void initialize(String name, Map
unresolved) {
}
danielcweeks commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1942062420
##
aws/src/main/java/org/apache/iceberg/aws/s3/VendedCredentialsProvider.java:
##
@@ -74,7 +75,11 @@ private RESTClient httpClient() {
if (null == client) {
danielcweeks commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1942060653
##
aws/src/main/java/org/apache/iceberg/aws/s3/signer/S3V4RestSignerClient.java:
##
@@ -192,6 +192,7 @@ private RESTClient httpClient() {
HTTPClie
jbonofre commented on PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#issuecomment-2634301895
I see @nastra approved. I did a new pass and it looks good to me.
@danielcweeks wdyt ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please
adutra commented on PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#issuecomment-2624547764
@danielcweeks @nastra is there anything else I need to change?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use th
nastra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1933758553
##
core/src/main/java/org/apache/iceberg/rest/HTTPClient.java:
##
@@ -214,92 +227,64 @@ private static void throwFailure(
throw new RESTException("Unhandled error
adutra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1933705991
##
aws/src/main/java/org/apache/iceberg/aws/s3/signer/S3V4RestSignerClient.java:
##
@@ -192,7 +192,8 @@ private RESTClient httpClient() {
HTTPClient.bui
danielcweeks commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1932536821
##
aws/src/main/java/org/apache/iceberg/aws/s3/VendedCredentialsProvider.java:
##
@@ -84,6 +85,7 @@ private RESTClient httpClient() {
private LoadCredential
gaborkaszab commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1932459105
##
aws/src/main/java/org/apache/iceberg/aws/s3/VendedCredentialsProvider.java:
##
@@ -84,6 +85,7 @@ private RESTClient httpClient() {
private LoadCredentials
nastra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1932371163
##
aws/src/main/java/org/apache/iceberg/aws/s3/signer/S3V4RestSignerClient.java:
##
@@ -192,7 +192,8 @@ private RESTClient httpClient() {
HTTPClient.bui
adutra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1932146669
##
aws/src/main/java/org/apache/iceberg/aws/s3/signer/S3V4RestSignerClient.java:
##
@@ -206,21 +206,24 @@ private AuthSession authSession() {
return authSession
nastra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1932128629
##
aws/src/main/java/org/apache/iceberg/aws/s3/signer/S3V4RestSignerClient.java:
##
@@ -206,21 +206,24 @@ private AuthSession authSession() {
return authSession
adutra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1931940876
##
core/src/main/java/org/apache/iceberg/rest/HTTPClient.java:
##
@@ -88,33 +84,30 @@ public class HTTPClient implements RESTClient {
@VisibleForTesting static fi
danielcweeks commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1931252722
##
core/src/main/java/org/apache/iceberg/rest/HTTPClient.java:
##
@@ -88,33 +84,30 @@ public class HTTPClient implements RESTClient {
@VisibleForTesting sta
adutra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1930539690
##
core/src/main/java/org/apache/iceberg/rest/HTTPClient.java:
##
@@ -88,33 +84,30 @@ public class HTTPClient implements RESTClient {
@VisibleForTesting static fi
nastra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1930517106
##
core/src/main/java/org/apache/iceberg/rest/HTTPClient.java:
##
@@ -88,33 +84,30 @@ public class HTTPClient implements RESTClient {
@VisibleForTesting static fi
adutra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1930471448
##
core/src/main/java/org/apache/iceberg/rest/BaseHTTPClient.java:
##
@@ -0,0 +1,220 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more
nastra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1930359628
##
core/src/main/java/org/apache/iceberg/rest/BaseHTTPClient.java:
##
@@ -0,0 +1,220 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more
danielcweeks commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1926123223
##
core/src/main/java/org/apache/iceberg/rest/HTTPClient.java:
##
@@ -214,92 +225,63 @@ private static void throwFailure(
throw new RESTException("Unhandled
danielcweeks commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1926121826
##
core/src/main/java/org/apache/iceberg/rest/HTTPClient.java:
##
@@ -214,92 +225,63 @@ private static void throwFailure(
throw new RESTException("Unhandled
danielcweeks commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1926113976
##
core/src/main/java/org/apache/iceberg/rest/HTTPClient.java:
##
@@ -88,33 +84,30 @@ public class HTTPClient implements RESTClient {
@VisibleForTesting sta
danielcweeks commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1926105323
##
core/src/main/java/org/apache/iceberg/rest/BaseHTTPClient.java:
##
@@ -0,0 +1,220 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or
danielcweeks commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1926099736
##
core/src/main/java/org/apache/iceberg/rest/HTTPClient.java:
##
@@ -214,92 +225,63 @@ private static void throwFailure(
throw new RESTException("Unhandled
nastra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1922259637
##
core/src/test/java/org/apache/iceberg/rest/TestRESTCatalog.java:
##
@@ -1486,12 +1349,14 @@ public void testCatalogTokenRefresh(String
oauth2ServerUri) {
nastra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1922265637
##
core/src/test/java/org/apache/iceberg/rest/TestRESTCatalog.java:
##
@@ -2122,23 +1948,18 @@ public void
testCatalogTokenRefreshDisabledWithCredential(String oauth2
nastra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1922263736
##
core/src/test/java/org/apache/iceberg/rest/TestRESTCatalog.java:
##
@@ -1767,29 +1635,23 @@ public void testCatalogValidBearerTokenIsNotRefreshed()
{
Mockit
nastra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1922262816
##
core/src/test/java/org/apache/iceberg/rest/TestRESTCatalog.java:
##
@@ -1653,7 +1517,6 @@ public void
testCatalogExpiredBearerTokenIsRefreshedWithCredential(String
nastra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1922260527
##
core/src/test/java/org/apache/iceberg/rest/TestRESTCatalog.java:
##
@@ -1508,20 +1373,21 @@ public void testCatalogTokenRefresh(String
oauth2ServerUri) {
nastra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1922258470
##
core/src/test/java/org/apache/iceberg/rest/TestRESTCatalog.java:
##
@@ -1324,89 +1222,67 @@ public void testTableAuth(
Mockito.verify(adapter)
.exec
nastra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1922256880
##
core/src/test/java/org/apache/iceberg/rest/TestRESTCatalog.java:
##
@@ -489,39 +463,29 @@ public void
testCatalogBearerTokenWithClientCredential(String oauth2Serve
nastra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1922256104
##
core/src/test/java/org/apache/iceberg/rest/TestRESTCatalog.java:
##
@@ -337,27 +337,20 @@ public void testCatalogBasicBearerToken() {
// the bearer token shoul
nastra commented on code in PR #11992:
URL: https://github.com/apache/iceberg/pull/11992#discussion_r1922256503
##
core/src/test/java/org/apache/iceberg/rest/TestRESTCatalog.java:
##
@@ -373,39 +366,29 @@ public void testCatalogCredentialNoOauth2ServerUri() {
// no token or
adutra opened a new pull request, #11992:
URL: https://github.com/apache/iceberg/pull/11992
4th PR for the Auth Manager API. Previous ones:
* #11844
* #11809
* #11769
This PR introduces the required changes to `RESTClient` and `HTTPClient`. It
also introduces a `BaseHTTP
40 matches
Mail list logo