dimas-b commented on code in PR #2711:
URL: https://github.com/apache/polaris/pull/2711#discussion_r2392089139
##########
runtime/service/src/intTest/java/org/apache/polaris/service/it/RestCatalogMinIOSpecialIT.java:
##########
@@ -301,6 +302,62 @@ public void testInternalEndpoints() throws IOException {
}
}
+ @Test
+ public void testCreateTableFailureWithCredentialVendingWithoutSts() throws
IOException {
Review Comment:
It's not about config defaults in this case, it's about MinIO being the only
backend available in OSS now that can operate without STS so we can only test
vended credential enforcement on MinIO.
The `ALLOW_EMPTY_VENDED_CREDENTIALS` setting is really for users to restore
old Polaris behaviour if the enforcement is problematic for them.
If we change the `ALLOW_EMPTY_VENDED_CREDENTIALS` default, this test will
have to override that config locally.
--
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]
