New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[v13] Allow setting max_session_ttl from clusterauth preferences #28130
Conversation
There was a bug in the original PR, right? Should we avoid backporting this on its own and include the other fix in here too? |
Yeah, this now includes that backport |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Preemptively blocking the merge in case #26824 (comment) turns out to be a bug.
Document `default_session_ttl` in the reference.
use default session ttl for `tctl auth sign` leave tctl alone
Make use of updateClientFromPingReponse
364809f
to
0c53992
Compare
(#28185) * Allow setting max_session_ttl from clusterauth preferences Document `default_session_ttl` in the reference. * Set the default session ttl in generateCert use default session ttl for `tctl auth sign` leave tctl alone * fix merge error * Fix wrong CertTTL the KeyTTL if its zero in teleterm Make use of updateClientFromPingReponse
backport #26824 and #28138