Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Add TLSv1.2 support for Kafka and Admin API #395

Merged
merged 2 commits into from
Jun 25, 2021
Merged

Add TLSv1.2 support for Kafka and Admin API #395

merged 2 commits into from
Jun 25, 2021

Conversation

MikeEdgar
Copy link
Contributor

In addition to TLSv1.3

Signed-off-by: Michael Edgar medgar@redhat.com

@MikeEdgar
Add TLSv1.2 support for Kafka and Admin API
96a7a33 
In addition to TLSv1.3

Signed-off-by: Michael Edgar <medgar@redhat.com>
@MikeEdgar MikeEdgar requested a review from ppatierno June 25, 2021 15:04
@MikeEdgar
Update test expectation
31bd32d 
Signed-off-by: Michael Edgar <medgar@redhat.com>
@MikeEdgar
Copy link
Contributor Author

Confirmed correct behavior in OSD deployment for both Kafka and Admin API server:

  • openssl s_client -connect mk-test-create.apps.medgar-kafka.5mdg.s1.devshift.org:443 -tls1_3 ✔️
  • openssl s_client -connect mk-test-create.apps.medgar-kafka.5mdg.s1.devshift.org:443 -tls1_2 ✔️
  • openssl s_client -connect mk-test-create.apps.medgar-kafka.5mdg.s1.devshift.org:443 -tls1_1 ❌

@MikeEdgar MikeEdgar added this to the 0.7.0 milestone Jun 25, 2021
@MikeEdgar MikeEdgar merged commit bc1335e into bf2fc6cc711aee1a0c2a:main Jun 25, 2021
@MikeEdgar MikeEdgar deleted the add_tls12_support branch June 25, 2021 17:07
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@ppatierno ppatierno ppatierno approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.7.0
Development

Successfully merging this pull request may close these issues.
2 participants
@MikeEdgar @ppatierno