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

Bring toomuchio config into the 21st century #58

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

brettpetch
Copy link

Applied a number of optimizations and best-practices for current webstandards, added ssl info.

@toomuchio
Copy link
Owner

There's some good stuff in here, I have an updated copy that's very similar I just haven't pushed as I forgot.

Your cfg is one that'll replace the base nginx.conf as well mind you, this was just intended to be dropped into conf.d/sites-enabled.

@brettpetch
Copy link
Author

brettpetch commented Feb 15, 2022

Interestingly enough I actually saw better IO/Buffers for clients when using the nginx proxy buffers rather than having them off

nginx.conf Show resolved Hide resolved
@brettpetch
Copy link
Author

proxy_set_header X-Plex-Container-Size $http_x_plex_container_size;
is required because of this log warning from Plex

Jul 11, 2022 17:41:19.341 [0x7f106b0e0b38] WARN - [Req#c3e9] Missing X-Plex-Container-Size header. This will fail with status code 400 in the future.


#Upstream to Plex
upstream plex_backend {
#Set this to the IP address that appears in `ifconfig` (NATTED LAN IP or Public IP address) if you want the bandwidth meter in the server status page to work
Copy link

Choose a reason for hiding this comment

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

Why have you removed this comment? The bandwidth charts in dashboard (Plex Pass feature) no longer work when using your exact config. Changing it to point to my local LAN IP (192.168.1.222), the bandwidth charts work again perfectly. (And yes, I'm using the set_real_ip_from and real_ip_header and such correctly)

#add_header Strict-Transport-Security "max-age=63072000; includeSubDomains; preload" always;

#Plex has A LOT of javascript, xml and html. This helps a lot, but if it causes playback issues with devices turn it off. (Haven't encountered any yet)
gzip on;
Copy link

Choose a reason for hiding this comment

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

Did you encounter issues having gzip enabled, you accidentally removed it, or just didn't think it helps?

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.

None yet

3 participants