roryqi commented on PR #10668: URL: https://github.com/apache/gravitino/pull/10668#issuecomment-4182335581
> 1. **Iceberg spec compliance**: The `type` field in error responses now uses spec-matching exception type names (e.g., `NotAuthorizedException` for 401, `InternalServerError` for 500) instead of generic HTTP status text, matching the [Iceberg REST API specification examples](https://github.com/apache/iceberg/blob/main/open-api/rest-catalog-open-api.yaml). > 2. **Gravitino REST API isolation**: Our approach (custom Jetty `ErrorHandler` registered only in `RESTService`) doesn't affect the `AuthenticationFilter` or the Gravitino REST API. The Gravitino server could separately add its own error handler following Gravitino's error response spec (which uses different numeric codes like `1000`, `1001`, etc.) — but that's out of scope for this Iceberg-specific issue. Maybe we should throw exception in the Authenticator directly and use different ExceptionHandler to handle them. -- 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]
