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

OpenSSL no longer included in image #182

Closed
tdpearson opened this issue Jul 12, 2017 · 5 comments
Closed

OpenSSL no longer included in image #182

tdpearson opened this issue Jul 12, 2017 · 5 comments

Comments

@tdpearson
Copy link

It appears that the switch from base image Jessie to strech-slim has removed openssl from the container image. Any plans to add openssl back into the nginx image?

@thresheek
Copy link
Collaborator

That's correct - the idea is to keep nginx image as minimal as it can be, without shipping things not directly used by nginx. What's the case for openssl in your setup?

@tdpearson
Copy link
Author

We were using it to generate self-signed certs directly in the container.

@thresheek
Copy link
Collaborator

Right, so a plain apt install openssl will work right after FROM: nginx. I'll keep this issue open to collect the feedback from the users that might want to +1 it and we'll see if it makes sense to provide openssl in the base image. Thanks!

@tdpearson
Copy link
Author

@thresheek Thanks for your quick responses. We have moved openssl into its own container and this is no longer an issue for us.

@eigood
Copy link

eigood commented Mar 13, 2018

My vote would be to never add it to the nginx container; it's simple enough to mount a volume from $HOST or elsewhere, and isolate certification creation away from the network-exposed service. You can(should) even mount said volume as read-only in nginx, for further security/protection.

sitnik added a commit to devicehive/devicehive-proxy that referenced this issue Jun 4, 2018
- remove generation of self-cigned certificates from start script
- remove openssl package installation
$
This only causes problems and isn't recommended to do by nginx developers (nginxinc/docker-nginx#182).
sitnik added a commit to devicehive/devicehive-proxy that referenced this issue Jun 4, 2018
- remove generation of self-cigned certificates from start script
- remove openssl package installation

This only causes problems and isn't recommended to do by nginx developers (nginxinc/docker-nginx#182).
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

No branches or pull requests

3 participants