bugraoz93 commented on code in PR #46981:
URL: https://github.com/apache/airflow/pull/46981#discussion_r1966643277
##########
airflow/config_templates/config.yml:
##########
@@ -1416,12 +1419,77 @@ api:
description: |
Number in seconds until the JWT token used for authentication expires
for CLI commands.
When the token expires, all CLI calls using this token will fail on
authentication.
+
Make sure that time on ALL the machines that you run airflow
components on is synchronized
(for example using ntpd) otherwise you might get "forbidden" errors.
version_added: 3.0.0
type: integer
example: ~
default: "3600"
+api_auth:
Review Comment:
I don't have a strong opinion on moving it under `api_auth`. It looks good.
The separate heading would be more memorizable. Thanks!
Shouldn't we move `auth_jwt_expiration_time` and
`auth_jwt_cli_expiration_time` since they relate to `api_auth`? Maybe not in
this PR, though, since they are not directly related to this piece of work. I
am asking out of curiosity. I can follow up with that piece of work 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]
