roryqi commented on issue #11190: URL: https://github.com/apache/gravitino/issues/11190#issuecomment-4514669259
> Instead of throwing `NoSuchCatalogException` directly from `shouldSkipAuthorization()`, the skip check now catches it and returns `false`. This is intentional because `shouldSkipAuthorization()` runs inside the authorization interceptor; throwing there would still be wrapped by `BaseMetadataAuthorizationMethodInterceptor.invoke()` as an authorization internal error. > > Returning `false` lets the request continue through the normal Iceberg REST path, where the missing catalog can be reported as the proper catalog-not-found error. I also added a unit test for this missing-catalog skip-check path. [e9df8e2](https://github.com/apache/gravitino/commit/e9df8e28b98e596687a7ca9793ef0aae12e0a118) Maybe we can raise a dedicated pull request to fix this issue. We can merge it quickly. -- 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]
