You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/11/12 15:18:02 UTC

[GitHub] [airflow] Zedmor commented on a change in pull request #12305: JSON as response for 404 not found

Zedmor commented on a change in pull request #12305:
URL: https://github.com/apache/airflow/pull/12305#discussion_r522184788



##########
File path: airflow/www/extensions/init_views.py
##########
@@ -134,12 +134,29 @@ def init_error_handlers(app: Flask):
 

Review comment:
       We override handler only for API, therefore in case we will build it w/o API default handler (with circles) should still be applied. 500 is handled properly by blueprint handler (with JSONify) btw, only 404 and 405 are not.




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org