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.

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

Enable TLS from single dedicated config file #140

Merged
merged 2 commits into from
Aug 14, 2024
Merged

Enable TLS from single dedicated config file #140

merged 2 commits into from
Aug 14, 2024

Conversation

minottic
Copy link
Contributor

follows from #58 and #139

@minottic minottic requested a review from a team April 22, 2024 13:41
@minottic minottic marked this pull request as draft April 23, 2024 08:37
@minottic minottic marked this pull request as ready for review April 23, 2024 12:32
@minottic minottic force-pushed the tls_config branch 4 times, most recently from 5369e02 to d271c66 Compare April 23, 2024 14:28
@minottic minottic force-pushed the tls_config branch 2 times, most recently from 885e3bc to 6cb64ae Compare May 27, 2024 07:33
@minottic minottic force-pushed the tls_config branch 2 times, most recently from 2b853b8 to 539e08e Compare August 13, 2024 14:04
@minottic
Copy link
Contributor Author

@fpotier @consolethinks I finally tried this, and the certificate gets issued correctly. I think, after your review, we can merge it

@minottic
Copy link
Contributor Author

closes #58
closes #139

@minottic minottic merged commit 56c8f52 into main Aug 14, 2024
8 checks passed
@minottic minottic deleted the tls_config branch August 14, 2024 08:21
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