Skip to content

KCORE-105: Revert "KAFKA-9460: Enable only TLSv1.2 by default and disable other TLS protocol versions (KIP-553) (#7998)"#275

Merged
Rajini Sivaram (rajinisivaram) merged 1 commit into
2.5from
KCORE-105-revert-tls-change
Mar 3, 2020
Merged

KCORE-105: Revert "KAFKA-9460: Enable only TLSv1.2 by default and disable other TLS protocol versions (KIP-553) (#7998)"#275
Rajini Sivaram (rajinisivaram) merged 1 commit into
2.5from
KCORE-105-revert-tls-change

Conversation

@rajinisivaram

Copy link
Copy Markdown

This reverts commit 172409c

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

…TLS protocol versions (KIP-553) (apache#7998)"

This reverts commit 172409c
@rajinisivaram

Copy link
Copy Markdown
Author

Ismael Juma (@ijuma) This is a straightforward revert of the AK commit that disabled old TLS versions. I wasn't sure exactly where this should be reverted. Is confluentinc/kafka 2.5 branch the right place for this?

@rajinisivaram

Copy link
Copy Markdown
Author

Manikumar Reddy (@omkreddy) I have confirmed with Ismael Juma (@ijuma) that we need this change in confluentinc/kafka as well as ce-kafka. Can you review please? It is a straightforward revert of the AK PR that disabled TLSv1.0 and TLSv1.1. Thanks.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Rajini Sivaram (@rajinisivaram) Thanks for the PR. LGTM.

@rajinisivaram

Copy link
Copy Markdown
Author

Manikumar Reddy (@omkreddy) Thanks for the review, merging to 2.5 branch.

Andrew Egelhofer (andrewegel) referenced this pull request in andrewegel/kafka Jul 27, 2021
Revert apache#275 Removed These tests - As per discussion in apache#602 these can
be left out.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants