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
SSL support for activate_control_app #2015
Labels
Comments
composerinteralia
added a commit
to composerinteralia/puma
that referenced
this issue
Oct 20, 2019
This commit extracts the `MiniSSL::Context` creation into its own `MiniSSL::ContextBuilder` class along the same lines as in [puma#1989]. This will allow us to reuse this code for adding SSL support to the control app (issue [puma#2015]). Since we will need the `MiniSSL` require and check in both places, I moved that into the `ContextBuilder` class as well. [puma#1989]: puma#1989 [puma#2015]: puma#2015
nateberkopec
pushed a commit
that referenced
this issue
Oct 21, 2019
* Extract class for building SSL context This commit extracts the `MiniSSL::Context` creation into its own `MiniSSL::ContextBuilder` class along the same lines as in [#1989]. This will allow us to reuse this code for adding SSL support to the control app (issue [#2015]). Since we will need the `MiniSSL` require and check in both places, I moved that into the `ContextBuilder` class as well. [#1989]: #1989 [#2015]: #2015 * Add SSL support for the control app This starts to address [#2015]. I think we will need to add SSL support to the control cli as well. [#2015]: #2015
Closed by #2046 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Valid:
Also valid:
Invalid:
It would be great for Puma to allow control app to be run behind a TLS certificate to ensure confidentiality of the token.
I know I can TLS-ify it using nginx but so we could use the same argument against
ssl_bind
:-)The text was updated successfully, but these errors were encountered: