Skip to content
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.

Sign up for GitHub

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

Jump to bottom

docs: idle timeout configuration #46521

Merged
merged 18 commits into from
Sep 23, 2024
Merged

docs: idle timeout configuration #46521

merged 18 commits into from
Sep 23, 2024

Conversation

mmcallister
Copy link
Contributor

Task to better document idle timeout configuration - #32855

@mmcallister mmcallister self-assigned this Sep 12, 2024
@github-actions GitHub Actions
Copy link

🤖 Vercel preview here: https://docs-5317qq2sz-goteleport.vercel.app/docs/ver/preview

@mmcallister mmcallister linked an issue Sep 12, 2024 that may be closed by this pull request
Add clarity to client_idle_timeout documentation #32855
Closed
@github-actions GitHub Actions
Copy link

🤖 Vercel preview here: https://docs-q90coi1im-goteleport.vercel.app/docs/ver/preview

@mmcallister mmcallister requested review from ptgott, atburke, stevenGravy and pschisa and removed request for atburke September 12, 2024 22:19
@github-actions GitHub Actions
Copy link

🤖 Vercel preview here: https://docs-dzjezogpr-goteleport.vercel.app/docs/ver/preview

@github-actions GitHub Actions
Copy link

🤖 Vercel preview here: https://docs-a9pmkq7yh-goteleport.vercel.app/docs/ver/preview

@github-actions GitHub Actions
Copy link

🤖 Vercel preview here: https://docs-b1mmlhn1g-goteleport.vercel.app/docs/ver/preview

@mmcallister mmcallister marked this pull request as ready for review September 12, 2024 23:09
@github-actions GitHub Actions
Copy link

🤖 Vercel preview here: https://docs-cj9hvfta3-goteleport.vercel.app/docs/ver/preview

@mmcallister
Copy link
Contributor Author

@zmb3 Thanks for the feedback! I've added this as a dedicated page under management/security instead, as it seems like it falls under a security best-practice. Let me know what you think

@github-actions GitHub Actions
Copy link

🤖 Vercel preview here: https://docs-awkeqymat-goteleport.vercel.app/docs/ver/preview

@github-actions GitHub Actions
Copy link

🤖 Vercel preview here: https://docs-9odyz3fe1-goteleport.vercel.app/docs/ver/preview

@mmcallister mmcallister marked this pull request as ready for review September 19, 2024 17:58
@github-actions GitHub Actions
Copy link

🤖 Vercel preview here: https://docs-kqe8q9elr-goteleport.vercel.app/docs/ver/preview

@mmcallister
Copy link
Contributor Author

Updated preview: https://docs-kqe8q9elr-goteleport.vercel.app/docs/admin-guides/management/security/client-timeout/

@mmcallister mmcallister added this pull request to the merge queue Sep 23, 2024
Merged via the queue into master with commit 25b6e54 Sep 23, 2024
40 checks passed
@mmcallister mmcallister deleted the 0924_idle_timeout branch September 23, 2024 18:58
@public-teleport-github-review-bot
Copy link

@mmcallister See the table below for backport results.

Branch Result
branch/v14 Create PR
branch/v15 Create PR
branch/v16 Create PR

@ptgott
Copy link
Contributor

ptgott commented Oct 7, 2024

@mmcallister Are we backporting this one?

@ptgott ptgott mentioned this pull request Oct 15, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ptgott ptgott ptgott approved these changes

@stevenGravy stevenGravy stevenGravy approved these changes

@pschisa pschisa Awaiting requested review from pschisa

@r0mant r0mant Awaiting requested review from r0mant

@xinding33 xinding33 Awaiting requested review from xinding33

@zmb3 zmb3 Awaiting requested review from zmb3

Assignees

@mmcallister mmcallister

Labels
backport/branch/v14 backport/branch/v15 backport/branch/v16 documentation no-changelog Indicates that a PR does not require a changelog entry size/sm
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add clarity to client_idle_timeout documentation
4 participants
@mmcallister @ptgott @zmb3 @stevenGravy