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

Handle attempt to close channel during TLS handshake in SSLDriver #32144

Closed
Tim-Brooks opened this issue Jul 17, 2018 · 2 comments
Closed

Handle attempt to close channel during TLS handshake in SSLDriver #32144

Tim-Brooks opened this issue Jul 17, 2018 · 2 comments
Assignees
@Tim-Brooks
Labels
>bug :Security/TLS SSL/TLS, Certificates v7.0.0-beta1

Comments

@Tim-Brooks
Copy link
Contributor

Tim-Brooks commented Jul 17, 2018 •
edited
Loading

This issue arose from these tests failures:

#32122

JDK11 changed the behavior of the SSLEngine when it is closed in the middle of a TLS handshake. The SSLEngine has always had finicky behavior around this edge case.

I have reached out to the security-dev openjdk mailing list to clarify the correct behavior.

In the meantime, I will mute this test.
@Tim-Brooks Tim-Brooks added >bug v7.0.0 :Security/TLS SSL/TLS, Certificates labels Jul 17, 2018
@Tim-Brooks Tim-Brooks self-assigned this Jul 17, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-security

This was referenced Jul 17, 2018
Merged
Closed
Tim-Brooks added a commit that referenced this issue Jul 18, 2018
@Tim-Brooks
Adjust SSLDriver behavior for JDK11 changes (#32145)
90fcb38 
This is related to #32122. A number of things changed related to adding
TLS 1.3 support in JDK11. Some exception messages and other SSLEngine
behavior changed. This commit fixes assertions on exception messages.
Additionally it identifies two bugs related to how the SSLDriver behaves
in regards to JDK11 changes. Finally, it mutes a tests until correct
behavior can be identified. There is another open issue for that muted
test (#32144).
@Tim-Brooks
Copy link
Contributor Author

Should be closed by #34398

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Tim-Brooks Tim-Brooks

Labels
>bug :Security/TLS SSL/TLS, Certificates v7.0.0-beta1
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@colings86 @Tim-Brooks @elasticmachine