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

ssl: Convey alert information to passive socket operations recv and s… #8823

Open
wants to merge 1 commit into
base: maint
Choose a base branch
from

Conversation

IngelaAndin
Copy link
Contributor

…etopts

If a TLS-1.3 server fails client certification the alert might arrive in the connection state and even after data has been sent. Make sure the alert information will be available in error reason returned from passive socket API functions recv and setopt.

@IngelaAndin IngelaAndin self-assigned this Sep 18, 2024
@IngelaAndin IngelaAndin added the team:PS Assigned to OTP team PS label Sep 18, 2024
Copy link
Contributor

github-actions bot commented Sep 18, 2024

CT Test Results

    2 files     67 suites   48m 7s ⏱️
  793 tests   749 ✅  44 💤 0 ❌
3 764 runs  2 970 ✅ 794 💤 0 ❌

Results for commit 2f89b65.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

…etopts

If a TLS-1.3 server fails client certification the alert might arrive
in the connection state and even after data has been sent. Make sure
the alert information will be available in error reason returned from
passive socket API functions recv and setopt.
@IngelaAndin IngelaAndin force-pushed the ingela/ssl/recv-setopts-alert/OTP-19236 branch from ebac4cc to 2f89b65 Compare September 18, 2024 17:59
@IngelaAndin IngelaAndin added the testing currently being tested, tag is used by OTP internal CI label Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team:PS Assigned to OTP team PS testing currently being tested, tag is used by OTP internal CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant