Skip to content

get client random if setting TlsSecrets on server#3481

Merged
nibanks merged 1 commit intomicrosoft:mainfrom
wfurt:clientRandom
Mar 8, 2023
Merged

get client random if setting TlsSecrets on server#3481
nibanks merged 1 commit intomicrosoft:mainfrom
wfurt:clientRandom

Conversation

@wfurt
Copy link
Copy Markdown
Member

@wfurt wfurt commented Mar 7, 2023

OpenSSL only, contributes to #2344

@wfurt wfurt requested a review from a team as a code owner March 7, 2023 16:24
Copy link
Copy Markdown
Collaborator

@nibanks nibanks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cc @anrossi, I'm ok with this as a temporary measure, but I'd really prefer a TLS-agnostic solution in the QUIC code, long term.

@anrossi
Copy link
Copy Markdown
Collaborator

anrossi commented Mar 7, 2023

@nibanks Agreed. This is fine for now.

@wfurt
Copy link
Copy Markdown
Member Author

wfurt commented Mar 7, 2023

related to dotnet/runtime#83001. I work out hook in .NET (for debug build only)
I started with Linux as it was super easy to fix.
cc: @ManickaP

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.

3 participants