Skip to content

Conversation

@marsishandsome
Copy link
Collaborator

Signed-off-by: marsishandsome marsishandsome@gmail.com

What problem does this PR solve?

What is changed and how it works?

Check List

Tests

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code

Code changes

  • Has exported function/method change
  • Has exported variable/fields change
  • Has interface methods change
  • Has persistent data change

Side effects

  • Possible performance regression
  • Increased code complexity
  • Breaking backward compatibility

Related changes

  • Need to cherry-pick to the release branch
  • Need to update the documentation
  • Need to be included in the release note

Signed-off-by: marsishandsome <marsishandsome@gmail.com>
@marsishandsome marsishandsome changed the title fix TLS fix TLS: ALPN unsupported. Nov 23, 2021
@marsishandsome marsishandsome changed the title fix TLS: ALPN unsupported. fix TLS: ALPN unsupported Nov 23, 2021
Signed-off-by: marsishandsome <marsishandsome@gmail.com>
Signed-off-by: marsishandsome <marsishandsome@gmail.com>
Signed-off-by: marsishandsome <marsishandsome@gmail.com>
Signed-off-by: marsishandsome <marsishandsome@gmail.com>
Signed-off-by: marsishandsome <marsishandsome@gmail.com>
Signed-off-by: marsishandsome <marsishandsome@gmail.com>
Signed-off-by: marsishandsome <marsishandsome@gmail.com>
@marsishandsome marsishandsome changed the title fix TLS: ALPN unsupported [DNM] fix TLS: ALPN unsupported Nov 24, 2021
@ankita25
Copy link
Contributor

ankita25 commented Dec 1, 2021

when will this be merged ?

@marsishandsome
Copy link
Collaborator Author

when will this be merged ?

It will be merged until a better fix is found.

@ankita25
Copy link
Contributor

ankita25 commented Dec 9, 2021

Can we get a timeline on when will it be merged ?

@zz-jason
Copy link
Member

zz-jason commented Dec 10, 2021

@ankita25 I created a feature request (#386) for it. could you describe more details about your use cases and the problems you met?

@zz-jason zz-jason added the status/DNM Do NOT merge this PR label Dec 10, 2021
@zz-jason
Copy link
Member

I'm going to close this PR since it hasn't been updated for a long time. Feel free to reopen it if you wish to advance it in the future.

@zz-jason zz-jason closed this Dec 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/DNM Do NOT merge this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants