adutra commented on code in PR #12595:
URL: https://github.com/apache/iceberg/pull/12595#discussion_r2035988748


##########
core/src/test/java/org/apache/iceberg/rest/TestRESTCatalog.java:
##########
@@ -121,9 +122,19 @@ public void createCatalog() throws Exception {
         ImmutableMap.of(CatalogProperties.WAREHOUSE_LOCATION, 
warehouse.getAbsolutePath()));
 
     HTTPHeaders catalogHeaders =
-        HTTPHeaders.of(Map.of("Authorization", "Bearer 
client-credentials-token:sub=catalog"));
+        HTTPHeaders.of(

Review Comment:
   @nastra sorry for the confusion, it's my fault. This change is imho the only 
way to exercise the HTTPClient actually sending those headers to the server. 
Don't you think that's valuable?



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

Reply via email to