jedcunningham commented on a change in pull request #21640:
URL: https://github.com/apache/airflow/pull/21640#discussion_r809586118
##########
File path: chart/values.yaml
##########
@@ -1384,7 +1384,7 @@ config:
remote_logging: '{{- ternary "True" "False" .Values.elasticsearch.enabled
}}'
# Authentication backend used for the experimental API
api:
- auth_backends: airflow.api.auth.backend.deny_all
+ auth_backends: airflow.api.auth.backend.session
Review comment:
This will need to be conditional based on the airflow version.
We might also consider just removing this completely and let the default
handle it - the default changed back in 1.10.11, I think it'd be okay as long
as we add it to the charts UPDATING.md too.
--
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]
