-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Adding Plex #4415
base: next
Are you sure you want to change the base?
Adding Plex #4415
Conversation
I am not very familiar with Plex, but why do we need all these port mappings? |
@peaklabs-dev Here's the documentation from Plex: https://support.plex.tv/articles/201543147-what-network-ports-do-i-need-to-allow-through-my-firewall/
|
I think the other ports are more optional and only needed in very special cases, so I think it makes sense to remove all ports since the required port 32400 is handled by the reverse proxy. |
Fair enough. If someone's got an issue in the future, I'll simply add the required ports one by one. |
Thanks. 2 more things, I just tried it and the healthcheck does not seem to work and also the devices section should be commented out by default as not everyone has a GPU on their server. |
Changes
Issues