Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

ADR about the usage of the TLS registry in client extensions establishing TLS connection #41126

Merged
merged 1 commit into from
Jun 12, 2024
Merged

ADR about the usage of the TLS registry in client extensions establishing TLS connection #41126

merged 1 commit into from
Jun 12, 2024

Conversation

cescoffier
Copy link
Member

@cescoffier cescoffier commented Jun 11, 2024 •
edited
Loading

Fix #41042.

@cescoffier cescoffier requested review from gsmet and geoand June 11, 2024 14:42
@quarkus-bot quarkus-bot bot added the area/adr label Jun 11, 2024
@cescoffier cescoffier mentioned this pull request Jun 11, 2024
Closed
@cescoffier cescoffier force-pushed the tls-registry-client-adr branch from e6cccbb to 63e0b12 Compare June 11, 2024 15:23
@cescoffier
Copy link
Member Author

@geoand As you did the exercise, is there anything I forgot to mention?

@geoand
Copy link
Contributor

geoand commented Jun 11, 2024

Will let you know in a few minutes once I've read the whole thing :)

@geoand
Copy link
Contributor

geoand commented Jun 11, 2024

Will let you know in a few minutes once I've read the whole thing :)

Read it all, nothing missing that I can think of!

@cescoffier cescoffier force-pushed the tls-registry-client-adr branch from 63e0b12 to bc04005 Compare June 12, 2024 06:18
@geoand geoand merged commit 3c6e8f9 into quarkusio:main Jun 12, 2024
3 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.13 - main milestone Jun 12, 2024
@cescoffier cescoffier deleted the tls-registry-client-adr branch June 12, 2024 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@geoand geoand geoand approved these changes

@gsmet gsmet Awaiting requested review from gsmet

Assignees
No one assigned
Labels
area/adr
Projects
None yet
Milestone
3.13.0.CR1
Development

Successfully merging this pull request may close these issues.

Write an ADR explaining how clients should use the TLS registry
2 participants
@cescoffier @geoand