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

Update setup-mta.sh to include tls_certcheck #2187

Merged
merged 1 commit into from
May 2, 2024

Conversation

Nicolaj-H
Copy link
Contributor

What

Added a simple option to ignore certificate errors.

Why

could be if your running self-signed, or if the root certificate isn't accepted, maybe you just need a fast way to enable sending reports while waiting for your mail provider to update their expired certificate.

@Nicolaj-H Nicolaj-H requested review from a team as code owners May 1, 2024 07:43
Copy link

github-actions bot commented May 1, 2024

Conventional Commits Report

😢 No conventional commits found.

👉 Learn more about the conventional commits usage at Greenbone.

@bjoernricks
Copy link
Contributor

Hi and thanks for your contribution! Could you also update the docs at https://github.com/greenbone/docs/blob/main/src/22.4/container/workflows.md?plain=1#L336 for this change?

@Nicolaj-H
Copy link
Contributor Author

Hi and thanks for your contribution! Could you also update the docs at https://github.com/greenbone/docs/blob/main/src/22.4/container/workflows.md?plain=1#L336 for this change?

i have made a PR at the docs, documenting the option.
greenbone/docs#467

Added option to ignore Certificate errors
@bjoernricks bjoernricks enabled auto-merge (rebase) May 2, 2024 07:25
@bjoernricks bjoernricks disabled auto-merge May 2, 2024 07:26
@bjoernricks bjoernricks merged commit fcb6cf4 into greenbone:main May 2, 2024
4 of 8 checks passed
@Nicolaj-H Nicolaj-H deleted the patch-1 branch May 2, 2024 07:26
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