jedcunningham commented on a change in pull request #21640:
URL: https://github.com/apache/airflow/pull/21640#discussion_r812239205
##########
File path: chart/values.yaml
##########
@@ -1382,9 +1382,6 @@ config:
# For Airflow 1.10, backward compatibility; moved to [logging] in 2.0
colored_console_log: 'False'
remote_logging: '{{- ternary "True" "False" .Values.elasticsearch.enabled
}}'
- # Authentication backend used for the experimental API
- api:
- auth_backends: airflow.api.auth.backend.deny_all
Review comment:
This looks good, though we should probably add something in
`chart/UPDATING.md` about it no longer being set.
##########
File path: docs/helm-chart/airflow-configuration.rst
##########
@@ -46,7 +46,7 @@ application. See the bottom line of the example:
remote_logging: '{{- ternary "True" "False"
.Values.elasticsearch.enabled }}'
# Authentication backend used for the experimental API
api:
Review comment:
```suggestion
```
##########
File path: docs/helm-chart/airflow-configuration.rst
##########
@@ -46,7 +46,7 @@ application. See the bottom line of the example:
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:
```suggestion
```
--
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]
