bito-code-review[bot] commented on PR #43896: URL: https://github.com/apache/superset/pull/43896#issuecomment-5963962189
<!-- Bito Reply --> The flagged issue is correct: the current error handling in `superset/views/error_handling.py` is too narrow, as it only catches specific exceptions and allows other filesystem-related errors like `PermissionError` or `NotADirectoryError` to escape, potentially leading to unhandled 500 errors. To resolve this, you should broaden the exception handling to catch a wider range of `OSError` subclasses or simply `OSError` itself, ensuring the application remains resilient to various filesystem failures. Would you like me to fetch all other comments on this PR to validate them and implement a comprehensive fix for all identified issues? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
