-
Notifications
You must be signed in to change notification settings - Fork 27
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
support for SSL-/-TLS certificates #133
Comments
@bartgrefte - The typical and appropriate method to achieve TLS for docker containers is to use a webserver as reverse proxy such as nginx/traefik/caddy and proxy the connection to the container. You can do this within the same docker network and only expose the webserver container. As these are established and tested webservers they are usually better performing and more secure at implementing TLS. |
@needs-coffee I am aware of that, unfortunately, I have yet to find a pre-made reverse proxy container (like Nginx Proxy Manager) that supports adding pre-existing certificates through command line. Right now I've set up an Apache webserver and manually configured that to function as a reverse proxy, where I can use my own pre-existing certificates. Not quite user-friendly, fortunately I have worked with Apache config-files before, so it is doable. |
I use nginx but i dont use nginx proxy manager - i just edit the config files directly, more portable. services:
reverse_proxy:
container_name: reverse_proxy
image: nginx
ports:
- 80:80
- 443:443
volumes:
- ./config/default.conf:/etc/nginx/conf.d/default.conf
- ./certs:/etc/nginx/certs
networks:
- adsb
depends_on:
- readsb and in the docker-compose directory -
Certs are added in a certs in the docker-compose folder with your chain, key and root ca |
Just out of curiosity, why doesn't there seem to be support for SSL-/TLS-certificates?
I have yet to find even óne docker container for ADS-B where the webinterface to display the flights can be used with those.
Consider this a request to implement support for usage of certificates.
The text was updated successfully, but these errors were encountered: