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

Add support for SAML certificate to zabbix.conf.php #408

Merged
merged 2 commits into from
Jun 24, 2021

Conversation

wpoely86
Copy link
Contributor

SUMMARY

This adds support for setting the certificates you need to use SAML authentication with Zabbix: https://www.zabbix.com/documentation/current/manual/web_interface/frontend_sections/administration/authentication#saml_authentication

These certificates need to be set up in the config file, the rest can be done from the web UI.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

Add support for certificates for SAML authentication

@dj-wasabi
Copy link
Contributor

Hi @wpoely86

Thank you! Looks very promising. Small change request: can you please add in the documentation a small note on that these files should already be present on the host and that this role is not responsible for placing these files?

@wpoely86
Copy link
Contributor Author

@dj-wasabi I've rephrased the documentation to make clear it's the path to XXX. It's not really mentioned anywhere else that you have to make sure yourself that these files are present on the host (like with the apache certificates)?

@dj-wasabi dj-wasabi merged commit 0273e33 into ansible-collections:main Jun 24, 2021
@dj-wasabi
Copy link
Contributor

Ah yes, looks good to me! Will merge the PR, thanks! 👍

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