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

enabling client certs #870

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sebasmannem
Copy link

@sebasmannem sebasmannem commented Mar 16, 2022

Trying to have stolon work with client certificates.
Please see #869 for more info.

@sebasmannem
Copy link
Author

Fixes #869

@sebasmannem sebasmannem force-pushed the clientcerts branch 4 times, most recently from cf1361c to c3967ed Compare May 15, 2022 06:26
@sebasmannem
Copy link
Author

sebasmannem commented May 15, 2022

@sgotti I tested this and it works. Team is running larger tests, just to see if all works as expected.
The implementation adds a lot of options, like other auth methods (peer, ident and cert auth), setting hostssl and the option to have different local auth and different remote auth options.
This implementation is very directly configurable, and with that highly flexible.
We could add documentation how to deploy a client cert authorised setup.
Alternatively we could wrap all in one or 2 options which would be easier to setup and document, but would not be as flexible.
Let me know your thoughts please...

@sebasmannem
Copy link
Author

@sgotti I intend to finalise this and my other commit over the next few weeks.
Let me know if you have any comments, ok?

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.

1 participant