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.

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

Updated TLS handshake timeout. #6692

Merged
merged 1 commit into from
May 4, 2021
Merged

Updated TLS handshake timeout. #6692

merged 1 commit into from
May 4, 2021

Conversation

russjones
Copy link
Contributor

Updated TLS handshake timeout. During some operations, Teleport can flood the network with traffic which causes the TLS handshake to occur slower than 1 second.

One example is during SSO login. The initial connection is an unauthenticated connection, and upon successful SSO login a "types.User" is created and replicated to all nodes. For large clusters this can mean 10k+ "types.User" objects getting replicated at the same time the user attempts to re-establishing another connection to Auth this time with valid identity credentials. This connection sometimes can take longer than the original 1 second timeout.

@russjones russjones requested a review from awly May 4, 2021 20:09
lib/multiplexer/tls.go Outdated Show resolved Hide resolved
Updated TLS handshake timeout. During some operations, Teleport can
flood the network with traffic which causes the TLS handshake to occur
slower than 1 second.

One example is during SSO login. The initial connection is an
unauthenticated connection, and upon successful SSO login a "types.User"
is created and replicated to all nodes. For large clusters this can mean
10k+ "types.User" objects getting replicated at the same time the user
attempts to re-establishing another connection to Auth this time with
valid identity credentials. This connection sometimes can take longer
than the original 1 second timeout.
This was referenced May 4, 2021
@russjones russjones merged commit 64696ca into master May 4, 2021
@russjones russjones deleted the rjones/update-timeouts branch May 4, 2021 23:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants