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

--gen-certs option #668

Closed
wants to merge 1 commit into from
Closed

--gen-certs option #668

wants to merge 1 commit into from

Conversation

p-mongo
Copy link
Contributor

@p-mongo p-mongo commented Oct 1, 2018

This is a work in progress implementation of #127 / #667 .

Currently --gen-certs will 1) make one cert and 2) tell mongod to use it. The server must be launched with --sslAllowConnectionsWithoutCertificates and the client must be launched with --sslAllowInvalidCertificates (and --ssl).

TODO (which I may or may not be doing immediately):

  1. Make a client cert also, to be able to run the client without --sslAllowInvalidCertificates.
  2. Figure out how this plays with cluster auth and cert auth.

@p-mongo
Copy link
Contributor Author

p-mongo commented Mar 5, 2019

@rueckstiess Any thoughts on this PR?

@stennie
Copy link
Collaborator

stennie commented Mar 6, 2019

@p-mongo Apologies for the delay in feedback. We had some private discussion when you posted the original WIP but neglected to comment on the issue.

We thought generating self-signed certificates (with associated forced options) might be feature creep, but if this works cross-platform it could be handy. Can you add usage instructions to docs/mlaunch.rst?

Regards,
Stennie

@stennie stennie added this to the 1.5.4 milestone Mar 6, 2019
@stennie stennie added the mlaunch label Mar 6, 2019
@p-mongo
Copy link
Contributor Author

p-mongo commented Mar 7, 2019

I have since figured out how to use test certificates in the driver (https://github.com/mongodb/mongo-ruby-driver/tree/master/spec#tls-with-verification) therefore I am also fine with not making the change proposed in this PR.

@stennie
Copy link
Collaborator

stennie commented Mar 7, 2019

@p-mongo OK, will close this for now.

@stennie stennie closed this Mar 7, 2019
@stennie stennie removed this from the 1.5.4 milestone Mar 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants