jscheffl commented on PR #60108: URL: https://github.com/apache/airflow/pull/60108#issuecomment-4263165104
> LGTM, but as Jens pointed out we should remove the extra new validity we used and use the existing queued timeout. > > Oh, though I guess that could be not set, in which case those tokens would be valid for ever which probably isn't the correct behaviour. Hmmmm. The queued timeout is defaulted to float(600.0) - so should not be end-less. (I believe the default of only 10min is way too short in my view - and today if you make your first setup and have the first real load really takes a long time to understand where the problem lies...) <img width="740" height="479" alt="image" src="https://github.com/user-attachments/assets/7ee51809-db28-48d3-8ea4-3b962a2b7432"; /> So we should point this out in the docs with this PR that JWT token lifetime is also influenced by this parameter. -- 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]
