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

fix: TLS setup (self-signed) error message should include SS_CA_CERT #3168

Merged
merged 1 commit into from Mar 10, 2023

Conversation

jrpear
Copy link
Contributor

@jrpear jrpear commented Mar 9, 2023

Description

Correct self-signed SSL cert error message to report another possibly missing file. This error confused me for a little while.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Copy link
Member

@polarathene polarathene left a comment

Choose a reason for hiding this comment

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

Thanks for catching this and taking the time to contribute for the benefit of others! ❤️

@polarathene polarathene merged commit 0baf82f into docker-mailserver:master Mar 10, 2023
7 checks passed
@polarathene polarathene added this to the v12.0.0 milestone Mar 10, 2023
@polarathene polarathene added area/scripts kind/improvement Improve an existing feature, configuration file or the documentation labels Mar 10, 2023
@polarathene polarathene changed the title Add SS_CA_CERT to self-signed cert error message fix: TLS setup (self-signed) error message should include SS_CA_CERT Mar 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/scripts kind/improvement Improve an existing feature, configuration file or the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants