Skip to content

Read multiple TLS packets in one read call #41820

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

Merged
merged 1 commit into from
May 6, 2019

Conversation

Tim-Brooks
Copy link
Contributor

This is related to #27260. Currently we have a single read buffer that
is no larger than a single TLS packet. This prevents us from reading
multiple TLS packets in a single socket read call. This commit modifies
our TLS work to support reading similar to the plaintext case. The data
will be copied to a (potentially) recycled TLS packet-sized buffer for
interaction with the SSLEngine.

This is related to elastic#27260. Currently we have a single read buffer that
is no larger than a single TLS packet. This prevents us from reading
multiple TLS packets in a single socket read call. This commit modifies
our TLS work to support reading similar to the plaintext case. The data
will be copied to a (potentially) recycled TLS packet-sized buffer for
interaction with the SSLEngine.
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-security

@Tim-Brooks Tim-Brooks changed the title Read multiple TLS packets in one read call (#41725) Read multiple TLS packets in one read call May 5, 2019
@Tim-Brooks Tim-Brooks merged commit 9270134 into elastic:7.x May 6, 2019
@Tim-Brooks Tim-Brooks deleted the backport_ssl_read_changes branch April 30, 2020 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants