olayiwola-gbenga commented on PR #70681: URL: https://github.com/apache/airflow/pull/70681#issuecomment-5907672407
> 1. This breaks the airflow entrypoint for everyone who is running airflow and accidentally adds a bad config. The worst part is they cannot even use `airflow config get-value` to debug. > 2. I also noticed this while trying to view my variables: > > <img alt="image" width="2000" height="718" src="https://private-user-images.githubusercontent.com/35884252/661061498-10478749-c663-4bb8-9659-81899bb157e3.png?jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3OTA3NTA2OTUsIm5iZiI6MTc5MDc1MDM5NSwicGF0aCI6Ii8zNTg4NDI1Mi82NjEwNjE0OTgtMTA0Nzg3NDktYzY2My00YmI4LTk2NTktODE4OTliYjE1N2UzLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNjA5MzAlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjYwOTMwVDA2Mzk1NVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTNmMzRkOGViNjZmYTIwNDg3MTQ1MzMyNzdmNTMxOTI4OTE4MDRiYzE4YjcxYjI0MjQ1YjIwMWVlZDgxYTljNmYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JnJlc3BvbnNlLWNvbnRlbnQtdHlwZT1pbWFnZSUyRnBuZyJ9.MgD7IGdmBsgPq49mJL_BXjQEIDLnXVIQRrSWTDZs8qU"> Thanks for pointing this out. The `403 Forbidden` error happens in the modal because `_check_expose_config()` was added to `GET /ui/backends_order` during an earlier review, and `[api] expose_config` is `False` by default. We can fix this by reading `backends_order` from the existing `/ui/config` endpoint instead so it works out of the box without needing `expose_config=True`. -- 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]
