roryqi commented on code in PR #10704:
URL: https://github.com/apache/gravitino/pull/10704#discussion_r3084785582
##########
iceberg/iceberg-rest-server/src/main/java/org/apache/gravitino/server/web/filter/BaseMetadataAuthorizationMethodInterceptor.java:
##########
@@ -205,4 +213,23 @@ public Object invoke(MethodInvocation methodInvocation)
throws Throwable {
return IcebergExceptionMapper.toRESTResponse(e);
}
}
+
+ // Assuming all REST catalog instances are Gravitino servers, for this
scenario, the Gravitino
+ // server would not perform authorization and would only act as a proxy.
+ private boolean skipAuthorizationForRestCatalog(
Review Comment:
Maybe we can't reuse the method authorizationCompleted and move logic to
IcebergMetadataAuthorizationMethodInterceptor.
--
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]
