You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "pierrejeambrun (via GitHub)" <gi...@apache.org> on 2024/11/25 10:42:26 UTC

[PR] Fix 401 and 403 handling new UI [airflow]

pierrejeambrun opened a new pull request, #44342:
URL: https://github.com/apache/airflow/pull/44342

   Small improvements:
   - Redirection URL is changed to use the legacy `login` endpoint for now. At the moment the new API does not expose any `/login` route, and the current redirection will cause a plain `404`. (We can swap that back later when the new login endpoint is implemented)
   
   - Do now handle 403. Permissions errors mean that the user is authenticated and recognized, he just does not have permission to do that. In that case redirecting him to `/login` does not serve any purpose as the user is already authenticated. We should instead handle the `403` and display the an appropriate message to the User.


-- 
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: commits-unsubscribe@airflow.apache.org

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


Re: [PR] AIP-38 Fix 401 and 403 handling [airflow]

Posted by "pierrejeambrun (via GitHub)" <gi...@apache.org>.
pierrejeambrun merged PR #44342:
URL: https://github.com/apache/airflow/pull/44342


-- 
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: commits-unsubscribe@airflow.apache.org

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