Skip to content

Fixed TlsHandshakeType enum label#1614

Merged
Tratcher merged 1 commit intodotnet:mainfrom
ritchiecarroll:main
May 23, 2022
Merged

Fixed TlsHandshakeType enum label#1614
Tratcher merged 1 commit intodotnet:mainfrom
ritchiecarroll:main

Conversation

@ritchiecarroll
Copy link
Copy Markdown
Contributor

Fixes #1613

See similar issue and PR in dotnet/runtime:
dotnet/runtime#66781
dotnet/runtime#66786

@Tratcher
Copy link
Copy Markdown
Member

Unfortunately the typo here is in a public API and fixing it causes a compile time breaking change. We shouldn't merge this fix until 2.0.

The same API in runtime is internal.

@karelz
Copy link
Copy Markdown
Member

karelz commented Mar 24, 2022

We need to hold the PR for post-1.1 (which should happen in ~April) when we are open for 2.0.
Let's close it for now, once we are branched off for 1.1, we can reopen and merge it.
Thanks for your fix @ritchiecarroll!

@karelz karelz closed this Mar 24, 2022
@Tratcher Tratcher reopened this May 23, 2022
@Tratcher Tratcher added this to the YARP 2.0.0 milestone May 23, 2022
@Tratcher Tratcher enabled auto-merge (squash) May 23, 2022 19:05
@Tratcher Tratcher merged commit 7900ddf into dotnet:main May 23, 2022
@Tratcher
Copy link
Copy Markdown
Member

Thanks

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.

TlsHandshakeType value 24 should be labeled KeyUpdate

3 participants