Skip to content

add CA for https#33

Open
legioner0 wants to merge 3 commits into
lrsmith:masterfrom
legioner0:add-ca-cert
Open

add CA for https#33
legioner0 wants to merge 3 commits into
lrsmith:masterfrom
legioner0:add-ca-cert

Conversation

@legioner0

Copy link
Copy Markdown
Contributor

instead of skip tls verify

@attachmentgenie

Copy link
Copy Markdown
Collaborator

@legioner0 would it be possible to add a test for this new behavior?

@legioner0

Copy link
Copy Markdown
Contributor Author

@legioner0 would it be possible to add a test for this new behavior?

Sure, next week

@legioner0

Copy link
Copy Markdown
Contributor Author

@attachmentgenie, done.
Implementation use a bit tricky way to make test work correctly - replace server certificate after initial procedure in entrypoint of icinga2 container, but before icinga2 server started. New certificate has IP SAN with 127.0.0.1 value, required for correct connection to https://127.0.0.1:5665/v1.

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