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

Isolate containers from net, use proxy for access #10

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

oryjkov
Copy link

@oryjkov oryjkov commented Jul 24, 2024

The idea is to disallow outside access from any of the immich containers.
This won't work if you're running e.g. the ML container on a
separate machine.

This adds a new proxy container that runs nginx to provide access from
outside to the immich web port and the postgres db port (for backups).
Then we set the pod network to "none" meaning that containers in the pod
only have the loopback interfaces. They can talk to each other, but not
to the outside world.

The outside world can access it through a systemd socket and the proxy.

oryjkov added 3 commits July 24, 2024 08:29
The idea is to disallow outside access from any of the immich containers.
This won't work if you're running e.g. the ML container on a
separate machine.

This adds a new proxy container that runs nginx to provide access from
outside to the immich web port and the postgres db port (for backups).
Then we set the pod network to "none" meaning that containers in the pod
only have the loopback interfaces. They can talk to each other, but not
to the outside world.

The outside world can access it through a systemd socket and the proxy.
Copy link
Owner

@jbtrystram jbtrystram left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks you for your suggestion.
However this will breaks some setups if running the machine-learning container. Maybe move that into a isolated directory and add a README there explaining the reasonning ?

edit : sorry for my reaction time ! I was away from computers for a while then haven't had the time to look until now

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.

2 participants